Szukanie tekstu w tablicy

[Ormi]

Funkcja porównuje dwie tablice. Jeżeli tekst zawarty w tej podanej jako drugi
argument znajduje się gdziekolwiek w tablicy podanej jako pierwszy argument, to funkcja ma zwracać 1. W innym wypadku zwraca 0. Problem jest taki. Jeżeli np. taki program:

//tutaj ta funkcja
main()
{
char buf[] = { "oabcaslelekoef" } ;
char buf2[] = { "abc" } ;
cout << tablewordsearch(buf, buf2) ;
}

To zwraca 1.
W przypadku kiedy do drugiej tablicy wrzucimy "asl", to wtedy zwraca 0, mimo że taki tekst występuje w tablicy 1 Ktoś wie, co tu jest "be"?
Oto funkcja:

int tablewordsearch(char buf1[], char buf2[]) {
long size1 = sizeof(buf1) / sizeof(char) ;
long size2 = sizeof(buf2) / sizeof(char) ;

for(int i = 0;i < (size1-1);i++) { // for1

for(int g=0;g<(size2-1);g++) { //for2


if(buf1[i+g] == buf2[g]) {

if(g == (size2-2)) {
return 1 ;
	}
continue ;
			}
else {
break ;
}
							
} // for2

	
} // for1

return 0 ;
}

[Mandr4ke]

Żeby Ci nie było za łatwo to wytłumacz mi dlaczego :


return 0 ;

[Ormi]

No return 0, dlatego, że jak główna pętla for się skończy, to znaczy, że nie ma w tablicy takiego tekstu. Gdyby był, to zgodnie z moim założeniem zostałoby wcześniej wykonane return 1... Tylko, że jak widać nie działa to tak "piknie"

[⌂⌂⌂]

abc oraz oab zwraca 1 ponieważ załapuje się do pierwszej pętli która..wykonuje się tylko 4 razy. size1 wynosi 4, pętla nie sprawdza więc całego ciągu, tylko 4 pierwsze znaki :>

[kaspsior]

..................... ...

[coefficient]

Troszkę skomplikowałeś sobie sprawę. Tak jak zostało to wyżej napisane, zmienne size1 oraz size2 mają wartość 4, przez co dalsza część traci sens. Poza tym sama funkcja jest napisana mało czytelnie, żeby nie powiedzieć dziwnie. Jeżeli rzeczywiście miałbym napisać taką funkcję, to zrobiłbym to w mniej więcej taki sposób:

int tablewordsearch(char *buf1, char *buf2) {
  int i;

  for(i = 0; i < strlen(buf1) - strlen(buf2) + 1; i++) {
      if (buf1[i] == buf2[0]) {
        int x;
        for (x = 1; x < strlen(buf2); x++) {
          if (buf1[i + x] != buf2[x]) break;
        }
        if (x == strlen(buf2)) return 1;
      }
  }
  return 0;
}

Jednak nie ma potrzeby pisać czegoś, co jest już zrobione:

cout<<(strstr(buf1, buf2) == NULL ? 0 : 1)<<endl;