[C++] array e stringhe

[minomic]

Ciao a tutti! Devo scrivere un programma in C++ che consideri due stringhe di testo e conti quante volte i caratteri della seconda sono presenti nella prima. Ho provato a usare gli array e a scrivere questo

#include <iostream>
#include <stdlib.h>
#include <string>
using namespace std;

int main () {
    int i , j , repeat;
    char p[10] , q[10];
    cout<<"Inserire il primo testo: ";
    cin>>p;
    cout<<"Inserire il secondo testo: ";
    cin>>q;
    for (i = 0 ; i < 10 ; ++i) {
        for (j = 0 ; j < 10 ; ++j) {
            while (p[i] == q[j]) {
                ++repeat;
            }
        }
    }
    cout<<repeat;
}

ma non funziona (dà dei risultati assurdi)...sapete aiutarmi? Grazie a tutti!

[ShaleN]

Ciao.
Prova a fare così:

for (i = 0 ; i < 10 ; ++i) {
for (j = 0 ; j < 10 ; ++j) {
if (p[i] == q[j]) {
++repeat;
}
}
}

[minomic]

grazie ma anche così continua a non funzionare
nessuna altra idea? Grazie!

[ShaleN]

Premetto che non ne so molto di C++, ma non si dovrebbe scrivere i++ al posto di ++i ?

EDIT:

Prova ad azzerare Repeat prima del ciclo for

[minomic]

niente neanche così...sto diventando matto!

[LeleFT]

Prendi visione del Regolamento interno. In particolar modo alle modalità di esposizione delle domande, dei titoli delle discussioni, dell'indicazione del linguaggio (obbligatoria) e dell'uso degli appositit ag CODE per il codice.

Questa discussione la sistemo io, ove possibile.


Ciao.

[minomic]

grazie delle indicazioni. comunque il mio problema l'ho già risolto.

[LeleFT]

Originariamente inviato da minomic
grazie delle indicazioni. comunque il mio problema l'ho già risolto.

E allora, per completezza, dovresti riportare la soluzione.
Il forum serve soprattutto a cercare soluzioni a problemi già affrontati.


Ciao.

[OninO]

codice corretto:

#include <iostream>
#include <stdlib.h>
#include <string>

using namespace std;

int main () {
    int i , j , repeat = 0;
    char p[10] , q[10];
    cout<<"Inserire il primo testo: "; p[0] = '\0';
    cin>>p;
    cout<<"Inserire il secondo testo: "; q[0] = '\0';
    cin>>q;
    for (i = 0 ; i < strlen(p) ; i++) {
        for (j = 0 ; j < strlen(q) ; j++) {
            if (p[i] == q[j]) {
                repeat++;
            }
        }
    }
    cout<< "Numero occorrenze: " << repeat;
}

errori:

1) stringhe non inizializzate
2) repeat non inizializzato, errore ancora più grave del primo. il "repeat" iniziava da un numero casuale.
3) nessun controllo sulla lunghezza delle stringhe. se introduci meno di 9 caratteri per le stringhe i due cicli continuano comunque a confrontare tra di loro le locazioni di memoria fino a i=9 e j=9.
4) il ciclo while va in loop infinito non appena si ha la situazione p[i] == q[j].

le notazioni postfisse mi piacciono di più

ciao

[minomic]

sì... io alla fine ho fatto così

#include <iostream>
#include <stdlib.h>
#include <string.h>
using namespace std;

int main () {
    int i , j , repeat = 0;
    cout<<"Inserire il primo testo: ";
    string p; getline (cin, p);
    cout<<"Inserire il secondo testo: ";
    string q; getline (cin , q);

    for (i = 0 ; i < p.size() ; ++i)    // scansiono le due stringhe con due contatori i e j
   {
        for (j = 0 ; j < q.size() ; ++j)
      {
            if (p[i] == q[j])
         {
                ++repeat;
            }
        }
    }
    cout<<"Il numero di ripetizioni fra le due stringhe e' " << repeat << endl;
    system ("PAUSE");
   return 0;
}

ma alla fine il principio è lo stesso.

Ciao a tutti