Sammenligne to strenge

Sammenlign 2 variable i een if-sætning med == (IKKE på hver side af == den er HELT gal!)

du skal vel have en

for(i=0;i<n;i++)

og testet skal være

if(s1[i] == s2[i])

Hvis jeg gerne vil bruge pointere, skal der i if sætningen så ikke stå *s1[i] og i det hele taget igennem funktionen?

Jeg har følgende:
Jeg ved godt at det ikke hænger helt sammen nu, sådan som jeg har brugt pointere. Men opgaven vil jeg meget gerne løse, så jeg til sidst i løkken 'flytter' pointeren med *s1++;

Jeg tilføjer først til sidst at det skrives ud, så det ser vi lige bort fra...


int  sammenlign_nchar(const char *s1, const char *s2, int n){       
    int i, comp_res;

    for ( i = 0; i <= n; i++){
        if(s1[n] == s2[n]){
                comp_res=0;
                }
               
        else if(s1[i] < s2[i]){
                comp_res=-1;                                         
                }
               
        else if(s1[i] > s2[i]){
                comp_res=1;                                         
                }
   
    *s1++;                                               
    }

}

med:

char *s

skal du bruge enten

*s

eller

s[i]

men ikke

*s[i]

Havde vist n i stedet for i ovenfor også!

Giver følgende ikke mening

int  sammenlign_nchar(const char *s1, const char *s2, int n){       
    int i, comp_res;
    for ( i = 0; i <= n; i++){
        if(*s1 == *s2){
                comp_res=0;
                }
               
        else if(*s1 < *s2){
                comp_res=-1;                                         
                }
               
        else if(*s1 > *s2){
                comp_res=1;                                         
                }
   
    *s1++;                                               
    *s2++;
    }

}

så løber jeg vel strengene s1 og s2 igennem med pointerne *s1 og *s2

i < n ikke i <= n bår du starter i 0

du skal returnere lige så snart du har en forskel

*s1++;                                             
    *s2++;

skal vel være

 
    s1++;                                             
    s2++;

Altså return comp_res efter hver if-del, eks.

  if(*s1 == *s2){
                comp_res=0;
return comp_res;
                }

skal jeg sætte en break ind efter return, så funktionen ikke kører videre eller sørger return for det?

Kun return hvis de er forskellige - ikke hvis de er ens

return returnerer fra funktionen med det samme

lige udenfor for-løkken har jeg tilføjet

    printf("\n%s", comp_res);

Er dette så ikke måden at kalde den på


    const char *s1="test";
    const char *s2="hest";
    int  n=2;
    sammenlign_nchar(s1, s2, n);

det ser rigtigt ud

Der sker absolut intet i programmet... Jeg kan trykke på en tast for at afslutte og det er det!

Eller den skriver (null) når de to strenge er ens, men ellers skrives der ingen ting!

Prøv lige og post hele koden

#include <stdio.h>
#include <stdlib.h>


int  sammenlign_nchar(const char *s1, const char *s2, int n){       
    int i, comp_res;
  //&& *s1 != '\0'
    for ( i = 0; i < n; i++){
        if(*s1 == *s2){
                comp_res = 0;
                }
               
        else if(*s1 < *s2){
                comp_res = -1;                                         
                return comp_res;
                }
               
        else if(*s1 > *s2){
                comp_res = 1;   
                return comp_res;                                     
                }
   
    *s1++;                                               
    *s2++;
    }
   
    printf("\n%s", comp_res);

}


int main(int argc, char *argv[])
{
    const char *s1="test";
    const char *s2="test";
    int  n=2;
    sammenlign_nchar(s1, s2, n);
 
  system("PAUSE");   
  return 0;
}

Prøv lige og erstat:

printf("\n%s", comp_res);

med:

comp_res = 0;
printf("%d\n", comp_res);

comp_res = 0;
printf("%d\n", comp_res);
return comp_res;

så skriver dden 0

Den skriver også 0 når jeg skriver:

printf("%d\n", comp_res);

og strengene er ens, men når de ikke er ens skrives den ingenting!

selvfølgelig

fordi der er jo ikke en printf før de return's

Helt sikkert, det virker, kanon :)

For lige at få det helt på plads.
hvorfor er det man skriver return comp_res; og ikke bare return. Jeg tænker på det som om at den returnerer værdien af comp_res, men den sørger vel bare for at den returnerer fra funktionen?

Du smider selvfølgelig bare et svar.

netop fordi du skal returne en værdi - funktionen returnerer int ikke void

og svar

aha... Mange tak for rigtig god hjælp.