Èt nummer til en ** position?

I koden i zip-filen skriver du flere gange til index 9, det går ikke da du kun har 9 pladser i arrayet, fra og med 0 til og med 8

Du har f.eks. i main:

	//Block 7
	myTempLevel.write(0, 6, 2);
	myTempLevel.write(2, 7, 6);
	myTempLevel.write(2, 9, 5);

Hvis du fixer det så vil programmet køre. Det var måske en ide at lave et check i funktionerne i Level der verificerede index.

Nååår ..Så er det derfor jeg først fik fejlen i run-time.

Har brugt "replace all" dum idé

simple fejl jeg sagtens kunne spildt min egen tid på at finde. Men tusind tak for hjælpen

Nååår ..Så er det derfor jeg først fik fejlen i run-time.

Har brugt "replace all" dum idé

simple fejl jeg sagtens kunne spildt min egen tid på at finde. Men tusind tak for hjælpen

Prøv at bruge defines i stedet for hardkodede tal.

Skidt:

int i, j;
for (i = 0; i < 9; i++) {
  for (j = 0; j < 9; j++) {
    matrix[i][j] = 0;
  }
}

Godt:

#define MATRIX_MAX 9
int i, j;
for (i = 0; i < MATRIX_MAX; i++) {
  for (j = 0; j < MATRIX_MAX; j++) {
    matrix[i][j] = 0;
  }
}

Så kan du senere omdefinere MATRIX_MAX og så slår ændringen igennem alle steder.

Ifölge Scott Meyers, skal man holde sig væk fra #define og bruge const i stedet:

const int MATRIX_MAX = 9

Ifölge Scott Meyers, skal man holde sig væk fra #define og bruge const i stedet:

Kode 

const int MATRIX_MAX = 9

yeah well...det afhænger af en fandens masse ting men i dette tilfælde vil en const int nok være bedst.

Jeg kan egentlig bedre lide enum metoden:
enum MyMagicNumbers
{
MATRIX_MAX = 9
};

Jeg kan egentlig bedre lide enum metoden:
enum MyMagicNumbers
{
MATRIX_MAX = 9
};

Hvad hvis flere skal have samme værdi ?

MATRIX_ROWS = 9,
MATRIX_COLS = 9

Det er ingen problem at have flere med samme værdi.
Med enum kan man også:

class MyClass
{
   enum MyMagicNumbers
   {
     MATRIX_ROWS = 9,
     MATRIX_COLS = 9
   };
public:
   int matrix[MATRIX_ROWS][MATRIX_COLS];
};

const int MATRIX_MAX = 9

Er ikke lige den beste løsning da jeg både køre loops til 9 i min level.cpp og main.cpp.

Og vil ikke lave det en global eller public. (Som efter min mening ikke er en god idé.)

Robert Larsen: Hvad er der galt med at bruge #define ?
.

Efter min menning, kan jeg ikke se nogen grund til at bruge enum. Jeg mener, hvad er fordelen hvis værdierne er de samme?

Robert Larsen: Hvad er der galt med at bruge #define ?

Defines kan være helt udemærkede at bruge, men mange forstår dem ikke og bruger dem forkert.

F.eks.:

#define MAX(a,b) ((a) > (b) ? (a) : (b))

Den kan bruges til at finde den største af to værdier:

int first = 7;
int second = 8;
printf("Det største af %d og %d er %d", first, second, MAX(first, second));

Men prøv at gætte hvad følgende vil skrive:

#include <stdio.h>

#define MAX(a,b) ((a) > (b) ? (a) : (b))

int main (int argc, char ** argv) {
    int numbers[] = {-1,7,2,3};
    int count;
    int max;

    for (count = 0; count < sizeof(numbers) / sizeof(int);) {
        max = MAX(count++, numbers[count]);
        printf("Max is %d\n", max);
    }
    return 0;
}

Derudover er const mere typesikker hvor defines kræver et cast.

Til gengæld er defines nødvendige til platform uafhængig kode. Specielt hvis du udvikler til Linux med autotools værkøjerne vil du se megen:

#ifdef SOME_DEFINE
...
#elif defined SOME_OTHER_DEFINE
...
#endif

...det kan ikke rigtig lade sig gøre på andre måder.