[C++] Stringhe: posizione della prima occorrenza

[paranoid85]

cosa cè che non va?(tante cose lo so siate buoni..)


#include<iostream>
using namespace std;

int main()
{
char s1[]="abcdefgh";
char s2[]="ilmnohcder";
int indice=0;
bool occorrenza=false;

while(!occorrenza)
{
for(int i=0;i!='\0';i++)
{
while(!occorrenza)
{
for(int j=0;j!='\0';j++)
{
if(s1[i]==s2[j])
{
occorrenza=true;
indice=i;}
}
}
}
}

if(occorrenza!=true)
cout<<"non ci sono occorrenze"<<endl;
else
cout<<"la posizione della prima occorrenza di s2 in s1 e': "<<indice<<endl;
system("pause");
}


grazie!!

[infinitejustice]

Originariamente inviato da paranoid85
cosa cè che non va?

c'è che nn usi la STL...

[paranoid85]

purtroppo l'esercizio è questo..
nn so bene o male cosa cè ke nn va

[Michele Facchin]

Puoi anche dirci cosa dovrebbe fare?
Cercare se s1 è contenuta in s2?

[paranoid85]

il testo dell'esercizio è:"scrivere un programma che date due stringhe di caratteri s1 e s2 restituisce la posizione del primo carattere della prima occorrenza di s2 in s1"

[paranoid85]

qualcuno m sa dire cosa cè ke nn va?

[pprllo]

Guarda, nel tuo codice ci ho visto parecchi errori. Così è come l'avrei fatto io:

int FindStr(char *s1, char *s2)
{
    for(int i = 0; s1[i] != '\0'; i++)
    {
        if (s1[i] == s2[0])
        {
            bool ok = true;
            for(int j = 1; s2[j] != '\0'; j++)
            {
                if (s1[i + j] != s2[j])
                {
                    ok = false;
                    break;
                }
            }
            if (ok) return i;
        }
    }
    return -1;
}

[paranoid85]

scusa se t disturbo ma almeno csi m rendo conto.. m potresti dire gli errori ke c sono?grazie!

[pprllo]

Ci sono diversi errori, sia di sintassi che di logica. Vediamo:

int main() 
{ 
    char s1[]="abcdefgh"; 
    char s2[]="ilmnohcder"; 
    int indice=0; 
    bool occorrenza=false; 

    while(!occorrenza) //Logica: cosa fai se non trovi occorrenze ? Vai in loop !
    { 
        for(int i=0;s1[i]!='\0';i++) //Sintassi. E' s1[i], non i. 
        { 
            while(!occorrenza)  //Logica: cosa fai se non ti viene data occorrenza = true al primo ciclo ? Vai in loop !
            {
                for(int j=0;s2[j]!='\0';j++) //Sintassi. E' E' s1[j], non j 
                { 
                    if(s1[i]==s2[j]) 
                    { 
                        occorrenza=true; 
                        indice=i;
                    } 
                    /*Logica. Il codice che hai messo qui praticamente fa questo:
                      confronta carattere dopo carattere TUTTA la stringa s2 con il singolo carattere
                      della stringa s1. Io non intendo questo con trovare occorrenze di s2 in s1, ma
                      di solito intendo trovare l'intera stringa s2 in s1. Per fare questo dovresti fare
                      una cosa del tutto diversa.*/
                } 
            } 
        } 
    } 

    if(occorrenza!=true) 
    cout<<"non ci sono occorrenze"<<endl; 
    else 
    cout<<"la posizione della prima occorrenza di s2 in s1 e': "<<indice<<endl; 
    system("pause"); 
}

[pprllo]

Includo una versione modificata:

#include<iostream> 
using namespace std; 

int main() 
{ 
    char s1[]="abcdefgh"; 
    char s2[]="def"; 
    int indice=0; 
    bool occorrenza=false; 
    
    for(int i=0;s1[i]!='\0';i++)
        if(!occorrenza) 
        {
            occorrenza = true;
            indice = i;
            for(int j=0;s2[j]!='\0';j++)
                if(s1[i + j]!=s2[j]) 
                    occorrenza=false; 
        }  

    if(occorrenza!=true) 
    cout<<"non ci sono occorrenze"<<endl; 
    else 
    cout<<"la posizione della prima occorrenza di s2 in s1 e': "<<indice<<endl; 
    system("pause"); 
}