🚀 go-pugleaf

> #include <stdio.h>
> #include <string.h>
>
>
> int isin(char *string1,char *string2) /* if (string1 isin string2) return
1
> else return 0 */

Waarom programmeer je "isin" eigenlijk uit? Wat is het (bedoelde) verschil
tussen "isin" en de standaard-functie "strstr"?

De artistieke impressie is dat "isin" 1 oplevert als het eerste woord van
string1 in string2 voorkomt. Of zie ik het verkeerd?

> {
>  char *word1;
>  word1 = strtok (string1, " "); // returns a char *
>  string2 = strstr (string2,word1); // returns a char *  ///

Vaststelling: "string2" kan nu van alles zijn (inclusief NULL).

>  if (string1 == '\0')

string1 is van het type "char *". '\0' is van het type "char". Da's dus niet
goed. Probeer je te testen of string1 de lege string is? Gebruik dan
"*strings1 == '\0'".
(Mag natuurlijk alleen als je zeker weet dat string1 != NULL)

>  {
>   /*it isn't in it */
>   return 0;
>  }
>  if(strncmp (string1, string2, strlen(string1)) == 0)

Huuu! string2 kan op dit punt ook gelijk aan NULL zijn. Zou de oorzaak van
een core-dump kunnen zijn...

>  {
>   /*its in it*/
>   return 1;
>  }
>  else
>  {
>   /*its not in it */
>   return 0;
>  }
> }
>
>
>
> int main()
> {
>    char *a="test test";
>    char *b="lala test tetsttt123";
>  if (isin(&a,&b) == 1) // werkt in C niet in CPP en moet CPP zijn dus :(

Alweer een fout in de typering. "&a" en "&b" zijn van het type "char**". De
functie isin
verwacht "char*". Met andere woorden: haal die "&" weg.

>  {
>   printf("it works");
>  }
>  else
>  {
>   printf("it doesn't work");
>  }
>  return 0;
> }
> ---------------- einde code -------------
>
> alle hulp is welkom
> alvast bedankt :)

Ik zou zeggen: probeer eerst een scherp te krijgen wat je nu eigenlijk wil
berekenen (vergeet de gevallen string1 == NULL en/of string2 == NULL daarbij
niet). Dan netjes uitprogrammeren:
1) Zorg dat de types kloppen. Je lijkt nog wat moeilijkheden te hebben met
pointers. Lees
K&R (of je favoriete leesboek) er nog eens op na. Eigenlijk is het heel
simpel. Als je een parameter hebt van het type "T*" en je hebt een argument
van hetzelfde type, dan heb je geen "&" nodig.
2) Zorg dat je rekening houdt met ALLE mogelijke resultaten van functies die
je aanroept (zoals het geval dat "strstr" NULL oplevert.
3) Als het nog niet lukt, post hier weer een volgende versie.

Succes!

Duco