Discussion :

[schnito]

salut à tous!

En fait j'utilise une listbox ds laquelle je recois un message.
Suelement, si le message est trop long, la listbox n'est plus assez large.

Il faut donc passer des lignes pour que le texte ne soit pas manger.

Voici mo,n 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
 
        if (strlen(strTemp)>40)
		{
			int truc=strlen(strTemp);
			int compt=0;
			CString strTemp3="";
			do
			{
 
 
			strTemp3=strTemp.Mid (compt,compt+40);
			m_ctrlTexte2.AddString(strTemp3);
 
			compt=compt+40;
			truc=truc-40;
			}while (truc>0);
			//m_ctrlTexte2.AddString("\t");
			int k;
			k=strlen(strTemp)-truc;
			strTemp3=strTemp.Mid (k,strlen(strTemp));
			m_ctrlTexte2.AddString(strTemp3);
		    m_ctrlTexte2.SetTopIndex(m_iNumber);
		    m_iNumber++;
 
		}
		else
		{
      		m_ctrlTexte2.AddString(strTemp);
			m_ctrlTexte2.SetScrollPos(SB_VERT,m_iNumber,true);
			m_ctrlTexte2.SetTopIndex(m_iNumber);
			m_iNumber++;     
		}

Malheuresement, ca lé fait bien au 1er passage dans la boucle, mais ensuite, le mid prends plus de 40 caractères.
Ce n'est pas normal.
Y commence au bon endroit mais prends trop de caractères...

Merci.

@+.

[Fladnag]

je te propose ca comme code :

Code :

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

1
2
3
4
5
6
7
8
 
CString temp="";
while (strlen(strTemp)>40) {
     temp=strTemp.Mid(0, 40);        // les 40 premiers char.
     m_ctrlTexte2.AddString(temp);   // on les stocke dans le ctrl
     strTemp=strTemp.Mid(40);       // on reduit la chaine de 40
}
m_ctrlTexte2.AddString(strTemp);   // on stocke la fin de la chaine

c'est un peu moins performant sans doute, parce que strTemp est modifiée a chaque tour de boucle, mais c'est plus lisible et je pense que ca devrait fonctionner normalement.

Fladnag

[farscape]

salut,
mid extrait une chaine d'une longueur demande à partir d'un ofset.
le premier argument de mid c'est l'ofset dans la chaine ça c'est bon mais le
deuxieme c'est la longueur à partir de l'ofset.
dans ton exemple compte+40 pour la longueur c'est pas bon !
si tu veux des chaines de 40 mets simplement 40.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
         int compt=0;
         CString strTemp3="";
         do
         {         
              strTemp3=strTemp.Mid (compt,40);
             if(!strTemp3.IsEmpty())
             m_ctrlTexte2.AddString(strTemp3);         
             compt+=strTemp3.GetLength();
         }while (strTemp3.GetLength()==40);

essaye ça pour voir .
ceci dit on peux aussi rajouter un ascenceur horizontal à la liste box avec un peu de code (elle ne le fait pas nativement).
ou une hauteur de ligne correspondant à plusieurs et en reprenant la main sur l'ecriture de la chaine dans la listbox (j'ai deja donne une reponse dans ce sens sur le forum)
voir combiner les 2 techniques.

[schnito]

ok merci!

Je vais essyer tous ca tt de suite.

@+.

[schnito]

Bon ben ce marche niquel!
C'etais qd meme une erreur a la con!

Je vais essayer de voir un autre truc->essayer de laiiser le mot de la fin car les derniers mot des lignes st coupés et sont finis ds l'autre ligne , c dommage.

@+.

[farscape]

salut,
le pb de la cesure devrait pouvoir se regler comme ça simplement

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
          int compt=0;
         CString strTemp3="";
         do
         {         
              strTemp3=strTemp.Mid (compt,40);
              while(!strTemp3.IsEmpty() && strTemp3.Right(1)!=" ")              
                   strTemp3=strTemp3.Left(strTemp3.GetLength()-1);
             if(!strTemp3.IsEmpty())
                     m_ctrlTexte2.AddString(strTemp3); 
             compt+=strTemp3.GetLength();
         }
         while (compt<strTemp.GetLength());

a 2 chouias pres soit doit etre bon j'ai pas teste c'est du directe...

[schnito]

Bon ben je viens de tester le truc de la censure mais ca marche pas.
("Le progremme de repond pas").
Il doit rapasser indefinimment ds la boucle

Code :

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

while(!strTemp3.IsEmpty() && strTemp3.Right(1)!=" ")

car qd je met ceci en commentaire, c bon...

Je vais essyer de comprendre ce que tu as fais pr debugger...

Merci en tt cas.

@+.

[farscape]

salut ,
je pense qu'en rajoutant ce test ça devrait etre mieux:

Code :

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

1
2
3
4
5
6
 
if(strTemp3.GetLength()==40)
		  {
				while(!strTemp3.IsEmpty() && strTemp3.Right(1)!=" ")             
				   strTemp3=strTemp3.Left(strTemp3.GetLength()-1);
		  }

faire le traitement uniquement si la chaine fait 40 de long.
pour pas tomber dans le cas ou il restera un mot à la fin .

[schnito]

Alors la merci! c cool ca marche vraiment niquel!

Bon, je ne comprens pas trop car je ne connais pas ces fonctions mais je vais regarder.

Merci.

@+.

[schnito]

Au fait Farscape, je te remercie bien pour ta technique mais pourrais tu m'expliquer ce que tu fais la:

Code :

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

1
2
3
4
5
if(strTemp3.GetLength()==40) 
        { 
            while(!strTemp3.IsEmpty() && strTemp3.Right(1)!=" ")              
               strTemp3=strTemp3.Left(strTemp3.GetLength()-1); 
        }

Car je ne comprends pas tres bien, et je dois l'expliquer.

Merci beaucoup.

@+.

[farscape]

salut,
globalement on part de la droite tant que l'on est sur un mot et on cherche le premier blanc pour s'arreter la on est sure d'avoir une chaine se terminant par un blanc et donc pas de mot coupe

:le traitement est fait si la longueur de la chaîne extraite contient 40 caractères en dessous ça veut dire que c'est la dernière ligne et ça évite de tourner en rond dead lock (verrou mortel) quand il ne reste plus qu'un mot sans blanc.
la boucle est maintenue tant que la chaîne n'est pas vide c'est une sécurité pour sortir de la boucle au cas ou il n'y a plus rien.
et tant que le dernier caractère de la chaîne (right(1) en partant de la droite 1 caractère) est <> " " donc tant que je suis sur un mot .
strTemp3=strTemp3.Left(strTemp3.GetLength()-1);
tant que je suis sur un mot on affecte dans la chaîne son contenue -1 caractère :
left : extraction de la chaîne en partant de la gauche sur une longueur .
GetLength() longueur de la chaîne en cours sans le '\0'.
quand on trouve un blanc on s'arrête d'itérer donc je n'ai pas couper un mot.
la ligne est complète.

[schnito]

Ok merci!

Maintenant c clair...

@+.