Discussion :

[emykev22]

Bonjour,

merci pour votre aide....

voici la requête
Entrer un mot de plus de 10 caractères : extraordinaire
Lettre à rechercher : a
Résultat : extrAordinAire
la lettre a apparaît aux positions 5 et 11.

il me reste a intégrer cette partie a la saisie d’écran qui est terminer.

Le problème lorsque je mets un indice a mon tableau le résultat ne s'affiche pas de la même manière
que si je mets mon indice direct.

deuxième problème lorsque j’enlève la ligne cout << str << ' ' << pos[i] << '\n'; je n'est plus les majuscule que remplacer

voici mon 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
int main ()
{  string str = "extraordinaire";
 
  string lettre = "i";
  string var = "";
 
  int pos[20];
 
  for (int i=1;i < 20;i++)
   {
       size_t trouver = str.find(lettre);
       if (trouver!= string::npos)
          {var=putchar(toupper(str[trouver]));
           str.replace(str.find(lettre),lettre.length(),var);
           pos[i]= ("%d",trouver+1);
 
           cout << str << ' ' << pos[i] << '\n';
          }
    }
 cout << str,"%s";
 cout << "                                                                             ";
 cout << "La lettre ("<< lettre <<") apparait a la position: "<<(pos[1]);
 int ind2=1;
 int nbr;
 for (int ind=0;ind < 21;++ind)
  {if (pos[ind] <0)
        { cout<<" et "<<pos[ind];
        };
  }
  cout << ".";
  cout << "              return 0;
}

lorsque je mets mon ind à pos[ind] :

lorsque je mets 2 direct à pos[2] :


résultat attendu
";

[foetus]

Je te donnes un petit code qui fonctionne parce que je vois trop de trucs qui ne vont pas.
En vrac

• les indices de boucle commencent à 0 et non pas 1
• mauvais algo: pourquoi utiliser la méthode find avec une boucle for alors qu'il faut utiliser une boucle while. Édit: je ne connais pas trop les std::string mais la méthode find me semble inappropriée à utiliser dans ce cas
• Le tableau initialisé de façon pas précise. Ainsi que la fin des 2 boucles
• Les cout ne sont pas regroupés
• Trop de variables utilisées pour rien. Ainsi que la fonction putchar
• La classe string peut être utilisée comme un tableau C. Je pense que c'est quand même un super point.
• Avec les cout utiliser endl à la place de '\n'
• Tu as voulu faire quoi avec ton retour de main dans un cout

Après pour l'utilisation du tableau pos_array pour stocker les positions je n'ai rien contre, mais ce n'est pas optimal.

Et utilises la balise code signalée par le croisillon

Édit: Ton problème me semble venir de cette ligne pos[i]= ("%d",trouver+1); même si je ne savais pas qu'on pouvait faire cela ()
Mais à ce que comprend tu prends une variable size_t qui est un unsigned int ou un unsigned long que tu transformes en chaîne de caractères (même pas en char) avec "%d" et que cette dernière est castée en int pour être mise dans ton tableau.
Et après tu ne comprends pas pourquoi pos[i] = 2 lui fonctionne

Édit 1: Merci @the Hound et @leternel, même si je ne vois pas l'utilité du truc ou x = (a, b) <=> a; x = b;.

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
#include<iostream>
#include <string.h> // pour le memset
 
using namespace std;
 
int main ()
{
    string str = "extraordinaire";
 
    char letter = 'i';
 
    int str_len = str.size();
    int pos_array[str_len];
    int pos = 0, count = 0;
 
    memset(pos_array, 0, str_len); // Pas forcement utile
 
    for(pos = 0 /* En C/ C++ on commence tjs par 0*/; pos < str_len; ++pos) {
//      On teste 2 chars
        if (str[pos] == letter) {
            pos_array[pos] = 1;
            str[pos] += ('A' - 'a'); // Majuscule ;-)
            count++;
 
//          On peut regrouper les cout
//          On ajoute 1 a la position pour que ce soit plus humain
            cout << str << ' ' << pos + 1 << endl;
        }
    }
 
//  On peut regrouper les cout
    cout << "La nouvelle chaine est " << str << '.' << endl << "La lettre (" << letter << ") apparait " << count << " fois ";
 
    if (count > 0) {
        cout << ((count > 1)? "aux positions ": "a la position ");
 
//      Petite optimisation
//      On va decrementer count pour arreter plus vite
 
//      On peut reutiliser la variable pos
        for(pos = 0; ((pos < str_len) && (count > 0)); ++pos) {
            if (pos_array[pos] == 1) {
                cout << pos;
 
                --count;
 
                if (count > 0) { cout << " et "; }
            };
        }
    }
 
    cout << "." << endl;
 
    return 0;
}

[the Hound]

Pour info, x = (a, b, c); <=> a; b; x = c;
Donc pos[i] = ("%d", trouver + 1); <=> pos[i] = trouver + 1;
Non, on ne peut pas caster aussi facilement ou alors pas en C/C++ !

[ternel]

Quelques précisions complémentaires concernant la virgule.

l'opérateur virgule a la plus basse priorité (18), après l'affectation (16) et même après throw (17).
Il prends deux opérandes (comme l'addition), les évalue, et retourne le second.

Tout se passe comme s'il était ainsi défini: template <typename A, typename B> B operator,(A, B) {return B;}. aux subtilités de copie et déplacement près

Dans boost, il y a un cas de redéfinition (avec le =) pour simplifier la création de vector. C'est maintenant inutile avec C++, mais ca existe.