Discussion :

[wilfryjules]

Bonjour à tous !!

Alors voilà, je cherche à ouvrir d'un seul coup plusieurs images notées pic1, pic2, pic3, ....., pic16 pour ensuite faire des manip dessus en analyse d'images, avec OpenCV.

Alors j'ai pensé faire une concaténation avec le terme constant "pic" et les différents numéros 1, 2, ... 16 le truc classique que j'ai déjà fait maintes fois mais là je coince !!



VOICI L'ERREUR :

Exception non gérée à 0x518ab840 (msvcr100d.dll) dans OpenCV.exe*: 0xC0000005: Violation d'accès lors de la lecture de l'emplacement 0x00000010cccccccc.

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


int main(int argc, char** argv)
	{
	    char* names[16];
		
		for(int i=0; i<16; i++)
		{
			names[i] =  new char[5];
		}

		

		const char a[] = "pic"; 


		for(int i=0; i<16; i++)
		{
			strcpy (names[i], a);  // L'ERREUR EST ICI
			strcat (names[i], convertInt(i+1).c_str());  // char * strcat ( char * destination, const char * source ); 
		}
	

	    namedWindow( wndname, 1 );
	    vector<vector<Point> > squares;
	   
	    for( int i = 0; names[i] != 0; i++ )
	    {
	        Mat image = imread(names[i], 1);

	        if( image.empty() )
	        {
	            cout << "Couldn't load " << names[i] << endl;
	            continue;
	        }
	       
	        findSquares(image, squares);
	        drawSquares(image, squares);
	
	        int c = waitKey();
	        
			//break;
	    }
	
	    return 0;
	}

Et en sortie j'obtiens :

Couldn't load pic1
Couldn't load pic2
.......................
Couldn't load pic16

J'ai cherché pas mal mais je suis débutant, et je n'ai point trouvé !! Votre aide me serait fort utile !!

[jouana]

Bonjour,

ensuite un new de 1000 c'est pas un peu excessif pour écrire une chaîne de 5 caractère max.

ce type de déclaration fonctionne bien dans le cadre de chaine qui ne bougeront pas.

Code :

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

1
2
3
 
char names[][] = {"pic1", "pic2", ....
}

Le contenu de tes 2 boucles tient dans une seule.

Et pq tu utilise des char* alors que tu es en C++ et que l'objet string t'éviterais qlq pb

[r0d]

Bonjour,

je ne peux que te conseiller avec insistance de:
. utiliser la classe string à la place des char*
. utiliser la classe vector à la place des tableaux[]

Par exemple, un tableau de chaines de caracteres, en c++, ne sera pas char* names[16], mais vector<string> names( 16 );

En utilisant des string à la place des char*, plus de strcopy ni de strcat (avec tous les problèmes qui vont avec), mais des opérations entre string (= et +).

[Bousk]

Le code est moche mais semble correct.
Preuve en est ta sortie "couldn't load pic1" ... si tu avais vraiment une exception, le code n'irait pas aussi loin, à fortiori si elle se trouvait là où tu l'indiques.

Code :

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

for( int i = 0; names[i] != 0; i++ )

Que signifie cette condition de sortie ?! (en dehors d'un bon gros crash bien évident)

[jblecanard]

Code :

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

names[i] != 0

var arriver jusqu'à l'indice 16 et taper une zone de mémoire qui ne lui est pas réservée ! Le pointeur n'est pas nul mais n'est pas valide pour autant. J'imagine que tu as fait ça pour pouvoir changer le nombre d'images sans avoir à modifier ta boucle (curieux car on peut utiliser une variable contenant le nombre d'images ^^) ?

L'utilisation de string, de vector et des itérateurs règlera tous ces problèmes.

[wilfryjules]

VOILA MON NOUVEAU CODE :

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
49
50
51
52
53
54
 
 
 
	int main(int argc, char** argv)
	{
	    vector <string> names;
 
 
		string png = ".png";
 
		string pic = "pic";
 
		string total; 	 
 
 
 
		for(int i=0; i<16; i++)
		{
			// créer un flux de sortie
			std::ostringstream oss;
			// écrire un nombre dans le flux
			oss << i+1;
			// récupérer une chaîne de caractères
			std::string result = oss.str();
 
			total = pic + result + png;
 
			names.push_back(total);
		}
 
 
	    namedWindow( wndname, 1 );
	    vector<vector<Point> > squares;
 
	    for( int i = 0; i<16 ; i++ )
	    {
	        Mat image = imread(names.at(i), 1);
 
	        if( image.empty() )
	        {
	            cout << "Couldn't load " << names[i] << endl;
	            continue;
	        }
 
	        findSquares(image, squares);
	        drawSquares(image, squares);
 
	        int c = waitKey();
 
			//break;
	    }
 
	    return 0;
	}

ça marche !!

Merci de vos indications !!

Je déteste le c ++ !!

[jblecanard]

VOILA MON NOUVEAU CODE :Je déteste le c ++ !!

Ben pourquoi, c'est pas beaucoup plus clair une fois écrit plus proprement ?

[jouana]

tu as encore 2 boucles avec la même condition qui ont leurs traitements qui pourraient tenir dans une seule

Après je dit ça je dit rien