Discussion :

[Altal]

Bonjour tout le monde.

Voilà, je viens d'ajouter ce code à mon 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
 
string chaine = var0;
	char* chaine_coupe;
	char *p = (char*)chaine.c_str();
	CDC *dc = m_List_Column.GetDC();
	int i;
 
 
	for( i = 0; i < Compteur; i++)
	{
		chaine_coupe = strtok( p, ";\n" );
		p=0;
		if(!chaine_coupe) break;
		CSize Size = dc -> GetTextExtent( chaine_coupe, strlen(chaine_coupe) );
		m_List_Column.InsertColumn ( i, chaine_coupe, LVCFMT_LEFT, Size.cx+2, -1 );
	}
 
	ListView_SetExtendedListViewStyle( m_List_Column.m_hWnd ,LVS_EX_FULLROWSELECT|LVS_EX_GRIDLINES );

et depuis j'ai un soucis d'interface.
En effet, avant d'ajouter ce code j'avais cette interface:
http://pics.imagup.com/ano1/1275947868.JPG

et depuis, mon interface ressemble à cela:
http://pics.imagup.com/ano1/1275947922.JPG

Changement incompréhensible :s.

Ce n'est pas très important me direz vous et je le conçois mais je voudrais juste comprendre pourquoi c'est devenu comme ca.

J'ai tenté de mettre en commentaire le code que j'avais ajouté et malgré cela, l'interface reste inchangé.

Merci d'avance pour vos petites infos

[Invité]

Bonjour,
A mon avis vous avez un écrasement de mémoire.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
// essayez de remplacer
	char *p = (char*)chaine.c_str();
// par
        char p[64];  //c'est à dire suffisamment grand
        strcpy(p, chaine.c_str();
// ou
        char* p = new char[ chaine.Length() + 1 ];
        strcpy(p, chaine.c_str();
// et il faudra penser à faire
        delete p;
// mais, si p a évolué, ça risque de poser des problèmes
// c'est pourquoi, je préfère la déclaration d'un tableau

Ce type de question est très à la mode ces temps-ci.
Un pointeur contient l'adresse mémoire du début d'une zone.
Lorsqu'on travaille sur une chaine de caractères, ou travaille sur un tableau.
La classe string est (je crois) une fausse classe. Elle contient un mécanisme interne qui permet de faire des opérations diverses sur des chaines.
Quant on fait S.c_str(), on renvoie un pointeur, mais si on veut travailler sur la chaine, il faut en faire une copie pour réserver de la place.
La place utilisé par string n'a pas physiquement été allouée, à aucun moment on ne doit la libérer.

Cordialement.

[JolyLoic]

La classe string est (je crois) une fausse classe.

??

Elle contient un mécanisme interne qui permet de faire des opérations diverses sur des chaines.
Quant on fait S.c_str(), on renvoie un pointeur, mais si on veut travailler sur la chaine, il faut en faire une copie pour réserver de la place.

Ce qui est vrai, c'est qu'on n'a pas le droit de modifier ce qui est retourné par c_str, et surtout pas d'en changer la taille. Dans la prochaine version du langage, on aura l'assurance que c_str retourne le pointeur interne, et on devrait pouvoir en changer le contenu (mais pas la taille) sans trop de soucis.
Pour découper simplement une chaîne, je conseille boost::string_algo, ou pour plus évolué, des regex.

La place utilisé par string n'a pas physiquement été allouée, à aucun moment on ne doit la libérer.

??

[JolyLoic]

Changement incompréhensible :s.

Et si tu remodifies ton code, tu retrouve la première version ? Je suis prêt à parier que non. Tu as probablement fait une autre modif (mise à jour du compilateur, d'une bibliothèque, de l'os ???).

[Invité]

@JolyLoic

Je suppose que les points d'interrogation que vous tapez veulent dire que vous ne comprenez pas. En tout cas, moi je crois avoir compris ça.
Cela fait deux fois depuis huit jours que je lis que la prochaine version de C++ précisera l'utilisation de c_str().
Moi, vous savez, je utilise c_str() depuis 10 ans, alors, s'il avait fallu que j'attende la prochaine version ??? ou !!!

[Jean-Marc.Bourguet]

Quant on fait S.c_str(), on renvoie un pointeur, mais si on veut travailler sur la chaine, il faut en faire une copie pour réserver de la place.

Moi, vous savez, je utilise c_str() depuis 10 ans, alors, s'il avait fallu que j'attende la prochaine version ??? ou !!!

La copie ne sera plus necessaire. Tant qu'on ne modifera pas la taille.

[Invité]

Oui, mais pour l'instant elle l'est.
Ceci dit, je suis bien content, mais je ne vais certainement pas racheter un compilateur, modifier mon programme etc.
Et actuellement il u a des gens qui se posent la question, que leur répondez-vous?

[JolyLoic]

La copie ne sera plus necessaire. Tant qu'on ne modifera pas la taille.

Et en pratique, aujourd'hui, il n'y a qu'une version de bibliothèque pour laquelle la copie est nécessaire (roguewave ?). Ça marche déjà sans ça avec les autres. Dans le futur, plus besoin de copies pour personne.

[3DArchi]

Salut,
Pour le calcul de la taille de ta colonne, est-ce que CListCtrl::SetColumnWidth ne te simplifierait pas le travail ? Pour découper les chaines, si boost ne te convient pas tu dois pouvoir t'en sortir soit en jouant avec la STL et les std::strings, soit il me semble me souvenir que les CString ont des choses aussi, non ?
Pour ton changement d'IHM, je ne dois pas bien y voir car la seule différence est l'affichage 'old' style windows standard (Comment appliquer le style XP aux fenêtres ?)

[Altal]

Alors rebonjour à tous (et toutes).

Tout d'abord désolé de ne répondre que mtn mais j'étais en exam toute la semaine^^. Je viens de lire toutes vos réponses et je vais tester tout ca pour voir ce qu'il en est. Je peux juste répondre à JolyLoic pour lui confirmer que effectivement, même en remodifiant mon code, je ne re tombe pas sur mon interface que j'avais avant. Mais pour autant, je n'ai rien modifié ailleurs :s.
J'ai même crée un nouveau projet, et arrivé à l'endroit ou je dois gérer les colonnes etc... je place le code et paf mon interface change.

Donc bon j'essai de mettre en oeuvre tout ce que vous avez dit et je vous tient au courant .

Merci encore

Altal

[JolyLoic]

Et actuellement il u a des gens qui se posent la question, que leur répondez-vous?

Que ce n'est pas l'idéal, que, sauf contraintes spécifiques (perfs...), j'écrirais probablement le code différemment (par exemple, avec boost::string_algo::split, comme indiqué dans mon premier message), mais qu'il est peu probable que ce soit la cause d'erreur.