[C++] Sostituzione di un carattere in una stringa

[ross98]

salve
vi ringrazio in anticipo,potreste aiutarmi a risolvere questo programma?
la consegna è di sostituire le lettere"a" in una stringa con il carattere "@".
questo è il programma sbagliato che ho svolto

#include <iostream>
#include <string>
using namespace std;
int change ()
{
 string a;
 cout<<"inserisci una stringa di massimo 100 caratteri"<<endl;
 cin>>a;
 if(a.length()>100)
 {
  cout<<"errore la stringa doveva essere di 100 caratteri"<<endl;
  return 1;
 }
 int x=0;
    string c;
 for (int i=0;i<a.length();i++)
 {int y=i;
  if (a[i]=='@')
  { string b[y-a.length()];
   
   for (y;y<a.length();y++)
   {
            a[y+1]='\n';
   b[x]=a[y+1];
   x++;
   
   }
   a[i+1]='\n';
   
  }
     else 
     {   c[i]=a[i];
         
             }
         cout<<c[i];
 }
 
}
int main()
{
 change();
 system("pause");
 return 0;
}

[alka]

Comincia specificando cosa ha il tuo programma che non va, senza costringere gli utenti ad analizzarlo, compilarlo ed eseguire per scoprire l'errore.

[ross98]

Mi compila ma non mi fa la sostituzione.Nel ciclo for l'else funziona ma è l'if che non va perché non mi fa la sostituzione del carattere "@" e non capisco dove ho sbagliato

[Scara95]

Ma che stavi cercando di fare? A livello logico dico, come cercavi di risolvere. Perché basta per ogni carattere i fare:

 if(a[i]=='a')
  a[i]='@';

[ross98]

ciao volevo dirti che la tua soluzione è sbagliata perchè se scrivo @gg@ al posto di darmi agga mi scive gg questo comunque è il codice secondo la tua correzione

#include <iostream>#include <string>
using namespace std;
int change ()
{
 string a;
 cout<<"inserisci una stringa di massimo 100 caratteri"<<endl;
 cin>>a;
 if(a.length()>100)
 {
  cout<<"errore la stringa doveva essere di 100 caratteri"<<endl;
  return 1;
 }
 int x=0;
    string c;
 for (int i=0;i<a.length();i++)
 {
  if (a[i]=='a')
  { a[i]=='@';
  }
     else 
     {   c[i]=a[i];
         
             }
         cout<<c[i];
 }
 
}
int main()
{
 change();
 system("pause");
 return 0;
}

[oregon]

Guarda meglio la seconda riga perché l'hai copiata male (ed evidentemente non capita).

E poi butta tutta la parte relativa alla string c e utilizza e visualizza solo la string a