Discussion :

[GR3lh442kR]

salut,
je suis en train de creer un prog en c/c++ et j'ai un problemme au niveau de l'execution. je ne vous explique pas le but du projet ni la signification des variables c'est trop long...
je vous montre mon code, ce n'est que le debut du projet:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
#include <stdlib.h>
#include <stdio.h>
#include <SDL/SDL.h>
 
struct Couleur
{
    int r,g,b;
    SDL_Surface *rectangle;
};
 
struct Pixel
{
    int no_couleur;
    int est_physique;
};
 
int main(int argc, char *argv[])
{
    //initialisation du tableau de couleurs
    Couleur couleurs[100];
    for(int i=0;i<100;i++){
            couleurs[i].r=-1;
            couleurs[i].g=-1;
            couleurs[i].b=-1;
            couleurs[i].rectangle=NULL;
    }
 
    //initialisation du tableau de pixels ( le problemme est ici!!! )
    Pixel pixels[800][600];
    for(int i=0;i<800;i++){
            for(int j=0;j<600;j++){
                    (pixels[i][j]).no_couleur=-1;
                    (pixels[i][j]).est_physique=0;
            }
    }
 
    //ecriture dans un fichier pour tester si les valeurs ont été bien ecrite ( c'est juste un test ça ne restera pas )
    FILE* fichier = NULL;
    fichier = fopen("testvar.txt", "w");
    for(int i=0;i<800;i++){
            for(int j=0;j<60;j++){
                    fprintf(fichier,"%d,%d\n",(pixels[i][j]).no_couleur,(pixels[i][j]).est_physique);
            }
    }
    fclose(fichier);
 
}

le problemme est ici:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
 
    //initialisation du tableau de pixels
    Pixel pixels[800][600];
    for(int i=0;i<800;i++){
            for(int j=0;j<600;j++){
                    (pixels[i][j]).no_couleur=-1;
                    (pixels[i][j]).est_physique=0;
            }
    }

je creer un tableau de pixels de 800 par 600. tout compile parfaitement mais au moment de l'execution, ça plante!
j'ai testé avec un tableau de pixels de 800 par 60 et la ça ne plante plus!
j'ai aussi testé avec un tableau d'entier de 800 par 600 et pareil ça ne plante plus.

je cherche a savoir pourquoi ça plante, p-e un problemme de memoire? de taille du tableau?

merci de vos reponses

[CSIE_Angel#5]

Je ne suis pas sûr mais tu as déclaré statiquement un tableau de 800*600 contenant une structure.

As tu essayé d'allouer dynamiquement ton tableau de pixels ?

De plus je te conseille de mettre des :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
#define WIDTH_SIZE 800
#define HEIGHT_SIZE 600

C'est plus facile pour changer ses valeurs par la suite plutot que de tout réécrire dans le code !

[Jan Rendek]

Pas de #define pour des constantes en C++. C'est villain et inutile.

En revanche, pour ce qui est de la cause première je pense qu'effectivement allouer 800x600 élément sur la pile en est probablement la source.

[CSIE_Angel#5]

Pas de #define mais il code en C/C++ car dans ce cas si il fait que du C++ je lui interdirait aussi les struct et lui conseillerais de faire des classes avec des constructeurs par défaut et de jolies données membres :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
Class Pixels
{
    public:
    Pixels();
 
    int mGetNoCouleur();
    int mGetIsPhysical();
 
private:
     int no_couleur;
     int est_physique;
};

Et je ferais une classe Ecran qui contiendrait des pixels ... 800*600 par exemple

[3DArchi]

Tu explose royalement ta pile. Par défaut, sous windows, je crois qu'elle est à 1MB et tu en demande 800x600x8=3 840 000!
Solution pas propre: /STACK:4000000 comme option de link.
Meilleur solution: new.

Plutôt que

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
#define WIDTH_SIZE 800
#define HEIGHT_SIZE 600

en C++, préférer:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
static const unsigned int g_uiWidth;
static const unsigned int g_uiHeight;

voire même encapsulé:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
struct stTailleEcran
{
static const stTailleEcran g_TailleEcran;
 
const unsigned int Width;
const unsigned int Height;
 
private:
stTailleEcran()
:Width(800),Height(600)
{}
stTailleEcran(const stTailleEcran&):Width(800),Height(600){}
 
}; 
const stTailleEcran stTailleEcran::g_TailleEcran;

[GR3lh442kR]

effectivement, a terme j'utiliserai des classes, pour le moment j'utilise des struct car je ne maitrise pas encore tout le c++ ( comme les classes par exemples ), ce programme est juste un "brouillon" je n'ai pas besoin qu'il soit codé a la perfection, il servira surtout de base pour une autre partie du projet et sera refait.

donc le problemme vien de la pile, new n'utilise pas la pile??

si je fait:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
 
    //initialisation du tableau de pixels
    Pixel *pixels = NULL;
    pixels = new Pixel[WIDTH_SIZE][HEIGHT_SIZE];
    for(int i=0;i<WIDTH_SIZE;i++){
            for(int j=0;j<HEIGHT_SIZE;j++){
                    (pixels[i][j]).no_couleur=-1;
                    (pixels[i][j]).est_physique=0;
            }
    }

(il y a peut etre des erreur?)
il commence a m'insulter:

cannot convert `Pixel (*)[600]' to `Pixel*' in assignment
no match for 'operator[]' in '*((+(((unsigned int)i) * 8u)) + pixels)[j]'

... ect

[CSIE_Angel#5]

new est utilisé quand tu veux instancier une classe. Or Pixels est une structure, tu doit donc utiliser la fonction malloc().

Déclare un Pixels** et regarde cela :http://www.developpez.net/forums/d28...sation-malloc/

[GR3lh442kR]

ok, je voi un peut mieu comment faire maintenant!

si je fait comme johnalias110, ça donne ça:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 
    //initialisation du tableau de pixels
    Pixel ** pixels = NULL;
    pixels = malloc (WIDTH_SIZE * sizeof(Pixel *));
    if (pixels != NULL){
        for (int i=0; i<WIDTH_SIZE; i++){
            pixels[i] = malloc (HEIGHT_SIZE * sizeof(Pixel));
        }
    }
    for(int i=0;i<WIDTH_SIZE;i++){
            for(int j=0;j<HEIGHT_SIZE;j++){
                    (pixels[i][j]).no_couleur=-1;
                    (pixels[i][j]).est_physique=0;
            }
    }

quand je compile ça me dit ça:

invalid conversion from `void*' to `Pixel**'

j'ai oublier quelque chose?

[Mat.M]

quand je compile ça me dit ça:
invalid conversion from `void*' to `Pixel**'

Il faut "caster" ou transtyper malloc en mettant (Pixel **) devant malloc.


Je n'arrive pas à comprendre ce que tu veux faire cela semble vraiment lourd comme méthode.
Avec SDL tu as des méthodes pour lire et écrire directement dans une SDL surface.
Ou veux-tu en venir ?

new est utilisé quand tu veux instancier une classe. Or Pixels est une structure, tu doit donc utiliser la fonction malloc().

je viens de tester sous Visual C++6 et une structure peut être allouée dynamiquement aussi avec new().

[GR3lh442kR]

SDL me permet d'afficher des surfaces avec une couleur de fond ou une image, mais moi j'ai besoin de pouvoir afficher les pixels 1 par 1 ( c'est une peu comme si je decomposait l'image en plein de petites images de la taille d'un pixel ) et que chaque pixels soit associé a une valeur ( a savoir est_phusique) qui definit si ont peut le traverser ou non...

merci a vous, ça marche tres bien.
voici le code final:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
 
    //initialisation du tableau de pixels
    Pixel ** pixels = NULL;
    pixels = (Pixel **)malloc (WIDTH_SIZE * sizeof(Pixel *));
    if (pixels != NULL){
        for (int i=0; i<WIDTH_SIZE; i++){
            pixels[i] = (Pixel *)malloc (HEIGHT_SIZE * sizeof(Pixel));
        }
    }
    for(int i=0;i<WIDTH_SIZE;i++){
            for(int j=0;j<HEIGHT_SIZE;j++){
                    (pixels[i][j]).no_couleur=-1;
                    (pixels[i][j]).est_physique=0;
            }
    }

[Mat.M]

mais moi j'ai besoin de pouvoir afficher les pixels 1 par 1 ( c'est une peu comme si je decomposait l'image en plein de petites images de la taille d'un pixel )

?? Il y a SDL_BlitSurface pour cela ! Il suffit de déclarer un rectangle de 1*1

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
SDL_Rect rcDest = { x_pos, y_pos, 1, 1 };
SDL_BlitSurface ( image, NULL, surface, &rcDest );

et que chaque pixels soit associé a une valeur ( a savoir est_phusique) qui definit si ont peut le traverser ou non...

Il y a bien plus souple que d'utiliser un tableau lourdingue de 800*600.
Il suffit comme je l'ai déja écris d'accéder directement à la SDL_surface et de tester le pixel à l'abscisse X et ordonnée Y voulus selon un décalage.
Selon la valeur du pixel tu peux déclarer ou non zone traversable ou non
http://www.libsdl.org/cgi/docwiki.cgi/SDL_LockSurface
Et puis si tu déclares un tableau de 800*600 ton programme risque de ralentir alors qu'en accédant directement à la SDL surface tu accèdes directement à la mémoire vidéo de la carte vidéo

[Médinoc]

new est utilisé quand tu veux instancier une classe. Or Pixels est une structure, tu doit donc utiliser la fonction malloc()

"Nonsense!"

new marche pour tous les types: structures, classes, int, etc.
Le malloc(), c'est une philosophie C. Et en plus, ça oblige à caster la valeur de retour.

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
	//initialisation du tableau de pixels
	Pixel ** pixels = new Pixel* [WIDTH_SIZE];
	//Selon le standard, inutile de tester la valeur de retour avec new,
	//puisqu'une exception C++ de type std::bad_alloc est lancée en cas d'échec.
	for (int i=0 ; i<WIDTH_SIZE ; i++) {
		pixels[i] = new Pixel[HEIGHT_SIZE];
        }
	for(int i=0 ; i<WIDTH_SIZE ; i++) {
		for(int j=0 ; j<HEIGHT_SIZE ; j++) {
			(pixels[i][j]).no_couleur=-1;
			 (pixels[i][j]).est_physique=0;
		}
	}

Ou mieux, juste une allocation par dimension:

Code C++ :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	//initialisation du tableau de pixels
	Pixel ** pixels = new Pixel* [WIDTH_SIZE];
	pixels[0] = new Pixel[WIDTH_SIZE*HEIGHT_SIZE];
	//Selon le standard, inutile de tester la valeur de retour avec new,
	//puisqu'une exception C++ de type std::bad_alloc est lancée en cas d'échec.
	for (int i=1 ; i<WIDTH_SIZE ; i++) {
		pixels[i] = pixels[0] + (i*HEIGHT_SIZE);
        }
	for(int i=0 ; i<WIDTH_SIZE ; i++) {
		for(int j=0 ; j<HEIGHT_SIZE ; j++) {
			(pixels[i][j]).no_couleur=-1;
			 (pixels[i][j]).est_physique=0;
		}
	}

[GR3lh442kR]

biensur qu'il y a blitsurface, mais pour faire ça il faut une RGBSurface crée a partir d'une couleur! et j'ai besoin que la couleur et la "physique" du pixel soient independantes.
donc ce que tu dit c'est exactement ce que je fait c'est juste que j'ai besoin de referencer la couleur du pixel et sa valeur "physique" dans un tableau.

[JolyLoic]

new est utilisé quand tu veux instancier une classe. Or Pixels est une structure, tu doit donc utiliser la fonction malloc().

Beaucoup d'erreurs dans cette phrase...

- new peut créer dynamiquement une classe, une structure, ou tout autre type
- malloc peut échouer à créer dynamiquement une structure :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
struct s
{
  string str;
};
s* ps = (s)malloc(sizeof(s));
ps->str = "toto";

Ce code est illégal. On ne peut utiliser directement une structure (ou une classe, d'ailleurs) allouée par malloc que si c'est un POD.

- De toute façon, structures et classes sont identiques, sauf pour les accès (privé, publique, protégé) par défaut.

[cynique]

(il y a peut etre des erreur?)
il commence a m'insulter:
cannot convert `Pixel (*)[600]' to `Pixel*' in assignment
no match for 'operator[]' in '*((+(((unsigned int)i) * 8u)) + pixels)[j]'
... ect

Tu dois utiliser:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
  Pixel (*pixels)[600] = new Pixel[800][600];

et puis "pixels[i][j]" marchera.

[CSIE_Angel#5]

Ah ben dis donc, j'ai toujours cru que new etait l'operateur d'instanciation des classes en C++. Il me serait jamais venu a l'idée de l'utiliser sur des structures. Merci !

[Sylvain Togni]

Structures et classes c'est la même chose, il n'y a que les droits d'accès par défaut qui diffèrent, public pour struct et private pour class. Une struct peut donc avoir des constructeurs, destructeur, fonctions virtuelles, ... et donc ne pas pouvoir être alloue par malloc.

[Médinoc]

Et comme je l'ai dit, new peut allouer n'importe quoi, y compris des int... Ce n'est pas "l'opérateur d'instanciation des classes", c'est l'opérateur d'allocation dynamique... tout court.

Et je rappelle qu'en C++, contrairement à Java, on peut instancier une classe sans new...