Discussion :

[Gilles57-H-G]

Je cherche à optimiser un "programme".

Ce programme combine des caractères, ainsi, sur cet exemple, je récupère trois pointeurs de type string, que je cherche à mettre directement dans un string.

Je ne veux pas rajouter de commande supplémentaire, ce qui, je pense ralentirait le code.

Pourquoi le string : mot ne s'affiche pas ? (première partie de l'exemple).

J'ai essayé une autre version dans la deuxième partie de l'exemple.
Cette foi c'est le string : word, qui n s'affiche pas correctement.

Merci.

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
 
#include <cstdlib>
#include <fstream>
#include "stdlib.h"
#include <stdio.h>
#include <sstream>
#include <string>    
#include <iostream>  
#include <cstring>
using namespace std;
 
int main(int argc, char *argv[])
{
 
 ofstream fichier("FileExit.txt", ios::out | ios::trunc); 
 ofstream fichier1("FileExit2.txt", ios::out | ios::trunc); 
 
 string mot;
 string str ("abcd");
 
 
  string::iterator it0;
    string::iterator it1;
      string::iterator it2;
 
 
  for ( it2=str.begin() ; it2 < str.end(); it2++ )
   {
    for ( it1=str.begin() ; it1 < str.end(); it1++ )
     {
      for ( it0=str.begin() ; it0 < str.end(); it0++ )
      {
 
         mot[0] = *it0;
          mot[1] = *it1;
           mot[2] = *it2;
 
 
    cout<<mot<<"  "<<*it2<<*it1<<*it0<<endl;
    fichier<<mot<<"   "<<*it2<<*it1<<*it0<<endl;
 
            }
          }
        }
    cout<<"------------Fin Premiere partie-----------------------"<<endl;
 
 
   string chaine = "acb";
   int taille = chaine.length();
 
   string word[taille];
 
 
  for (int i2=0; i2<taille; i2++)  
      { 
     for (int i1=0; i1<taille; i1++)  
      { 
       for (int i=0; i<taille; i++)   
          {
 
           word[0] = chaine.substr(i,1);
           word[1] = chaine.substr(i1,1);
           word[2] = chaine.substr(i2,1);
 
            cout<<word<<endl;
             fichier1<<word<<endl;
 
 
             }
           }
        }
 
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

[Steph_ng8]

Bonjour,

sur cet exemple, je récupère trois pointeurs de type string

Ah bon ?
Je dois avoir la vue qui baisse…

Pourquoi le string : mot ne s'affiche pas ? (première partie de l'exemple).

Voyons voir…
Tu déclares une variable de type string sans l'initialiser.
A priori, ça appelle le constructeur par défaut ; donc on va dire que cette variable représente la chaîne vide.
Ensuite tu essaies d'accéder aux 3 premiers caractères de cette chaîne.
La chaîne vide ayant une longueur nulle, ces caractères n'existent pas.
Bonne chance pour affecter lesdits caractères !

Si encore tu avais donné une taille à cette chaîne, par exemple en appelant ce constructeur qui crée une chaîne avec n fois le caractère c :

Code :

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

string::string(size_t n, char c)

ou, si tu veux simplement stocker des caractères sans faire d'opération dessus, et si tu connais à l'avance leur nombre :

Code :

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

1
2
3
4
5
6
7
8
char mot[NB_CHARS];
 
// Ou éventuellement :
char mot[NB_CHARS + 1];
mot[NB_CHARS] = '\0';
 
// Voire, s'il n'y a pas trop de caractères :
char mot[] = {'\0', '\0', ... , '\0'};

J'ai essayé une autre version dans la deuxième partie de l'exemple.
Cette foi c'est le string : word, qui n s'affiche pas correctement.

Voyons…
Tu crées un tableau de string, et tu n'en initialises aucun.
Ah bah en fait, c'est exactement le même cas que juste au-dessus.


Ceci dit, il y a quelque chose qui me perturbe.
Tu crées un tableau dont la taille est une variable.
En toute logique, ça devrait être interdit !!!
D'après ce que j'ai compris des tableaux alloués statiquement, leur taille doit être connue à la compilation.
Ce n'est clairement pas le cas ici !!!

[Gilles57-H-G]

tu as certes raison, je compris ce qui ce passe.

Bon j'ai mis une taille au string.

Code :

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

1
2
3
4
 string str ("abcd");
 int t = str.length();
 
 string mot[t]; ;

Cela ne change rien.

J'ai besoin de la valeur pour tester.

Cela fonctionne en passant par une variable intermédiaire.

Je voulais supprimer cette étape .
Je veux faire : 3 valeurs pointées rangées dans un string.

Ne me dis pas que ce n'es pas possible.

Bon pour lestableaux alloués statiquement, je vais faire attention.

[Gilles57-H-G]

Ceci dit.

Dans le dernier exemple la taille du tableau est connue à la compilation, puisque c'est le string chaine qui défini la taille via la fonction length.

Le string chaine contient un nombre de caractères fixe, non modifiable.

L'exemple en question

Code :

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

1
2
3
4
5
6
 
   string chaine = "acb";
 
   int taille = chaine.length();
 
   string word[taille];

Il eut été plus élégant de définir une taille via un entier , je conçoit.

[gl]

Bon j'ai mis une taille au string.

Code :

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

1
2
3
4
 string str ("abcd");
 int t = str.length();
 
 string mot[t]; ;

Cela ne change rien.

Non tu n'as pas mis une taille au string, tu as créé un tableau de string.

Au passage, comme déjà signalé, ce code n'est pas correct en C++: on ne peut pas créer de taille statique avec une variable en guise de table. A priori tu utilises les extensions de ton compilateur.

Dans le dernier exemple la taille du tableau est connue à la compilation, puisque c'est le string chaine qui défini la taille via la fonction length.

Non, même si ici tu sais avec certitude la valeur qui sera contenue dans taille sans l'exécuter,taille n'en reste pas moins une variable, pas une constante connue à la compilation. On ne peut donc pas l'utiliser comme taille de tableau.

Pour illustrer l'explication, le code suivant n'est pas valide:

Code :

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

1
2
int t = 3;
int tab[t];

alors que celui-ci l'est:

Code :

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

1
2
const int t = 3;
int tab[t];

Le string chaine contient un nombre de caractères fixe, non modifiable.

Si le nombre de caractère (et le contenu) de la chaîne est modifiable.

Je pense que tu confonds le fonctionnement des chaînes C++ (std::string) et des char* pointant sur des chaînes littérales [1].





<HS>
[1] Au passage, on ne peut certes pas modifier la chaîne littérale dans ce cas, mais on peut modifier l'adresse contenue dans le pointeur. Le résultat de strlen(p) peut donc tout à fait évoluer même dans ce cas de figure
</HS>

[Gilles57-H-G]

@gl "Non tu n'as pas mis une taille au string, tu as créé un tableau de string."

C'est évident, je ne m'en était pas rendu compte, merci.

Mais j'ai lu qu'un string, est un tableau de caractère, qui sont de type string.

Donc, un string est un tableau de string par définition ?
-------------------------------------------------------------------------------Fin @ gl

Au départ, tout fonctionne, je veux simplement supprimer une commande.
La version que je cherche à rendre plus rapide.

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
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
  #include <cstdlib>
#include <iostream>
#include <fstream>
#include "stdlib.h"
#include <stdio.h>
#include <sstream>
#include <string>    
#include <cstring>
 
using namespace std;
 
int main(int argc, char *argv[])
{
 
 
    using namespace std;
     int a,b,c,d;
 
     string lt1,lt2,lt3;
     string mot;
 
     int taille;
 
 
     string chaine;
     cout<<"Entrez caracteres"<<endl;
     cin>> chaine;
 
     taille = chaine.length();
 
      string t1[taille];
         string t2[taille];
             string t3[taille];
 
 
         for (int ii=0; ii<taille; ii++)   
 
          {
 
                t1[ii] = chaine.substr(ii,1);
                t2[ii] = chaine.substr(ii,1);
                t3[ii] = chaine.substr(ii,1);
 
                                                       }
 
 
            for(c=0; c<taille; c++)
              {
                for(b=0; b<taille; b++)
                 {
                    for(a=0; a<taille; a++)
                     {
 
                     lt1 = t1[a];  
                     lt2 = t2[b];
                     lt3 = t3[c];
 
 
                     mot =    lt3 += lt2 += lt1  ;
 
                     cout<<mot<<endl;
 
 
                      }
                  }
              }
 
 
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

je voulais transformer le code ci dessus, résultat de l'application d'un algorithme basé sur des tableaux.

En ceci, utilisant les iterator de la classe string, en affichant directement les pointeurs dans le string mot, supprimant une commande .

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
#include <iostream>
#include <cstdlib>
#include <fstream>
#include "stdlib.h"
#include <stdio.h>
#include <sstream>
#include <string>    
#include <cstring>
 
using namespace std;
 
int main(int argc, char *argv[])
{
 
 
    using namespace std;
 
    string mot;
    string str;
 
    cout<<"Entrez caracteres"<<endl;
    cin>> str;
 
 
  string::iterator it0;
    string::iterator it1;
      string::iterator it2;
 
 
  for ( it2=str.begin() ; it2 < str.end(); it2++ )
   {
    for ( it1=str.begin() ; it1 < str.end(); it1++ )
     {
      for ( it0=str.begin() ; it0 < str.end(); it0++ )
      {
 
         mot[0] = *it0;
          mot[1] = *it1;
           mot[2] = *it2;
 
 
    cout<<mot<<"  "<<*it2<<*it1<<*it0<<endl;
 
 
            }
          }
        }
 
    system("PAUSE");
    return EXIT_SUCCESS;
}

Si je réussissais à récupérer directement la valeur pointée par les 3 pointeurs, rangés dans un string, le code serait plus rapide.

[3DArchi]

Salut,

ou, si tu veux simplement stocker des caractères sans faire d'opération dessus, et si tu connais à l'avance leur nombre :

Code :

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

1
2
3
4
5
6
7
8
char mot[NB_CHARS];
 
// Ou éventuellement :
char mot[NB_CHARS + 1];
mot[NB_CHARS] = '\0';
 
// Voire, s'il n'y a pas trop de caractères :
char mot[] = {'\0', '\0', ... , '\0'};

On reste sur des strings ! Pourquoi se mettre en danger avec des char