C programmering

Segmenteringsfejlen kommer fordi strcmp bliver bedt om at sammenligne en peger til en liste af karakterer med en liste af karakterer.

Du har erklæret

char *type[max_food];

hvilket betyder, at 'type' peger til en liste af karakterer. 'type' selv er ikke denne liste.

Når du dernæst i strcmp anvender 'type[0]' fås blot indholdet af variablen 'type' som ikke den list, du vil sammenligne med, men derimod adressen på den.

Brug istedet erklæringen:

char type[max_food];

og din strcmp skal så være:

strcmp(type,rest[i].food == 0){

Dette løser segmenteringsproblemet, men du får andre. F.eks. terminering scanf-konverteringen med 's' formatet så snart den støder ind i et mellemrum. Derfor vil inputtet 'Fast Food' blive formateret som 'Fast'. Der er også andre problemer, men prøv at boks lidt med dem. Ellers må du vende tilbage.

Se venligst gennem fingrene med diverse %"#-stavefjel og magler !

Forklaringen på segmenteringsfejlen i #1 gik vist lidt vel hurtigt. Problemet er, at du har erklæret en liste af pegere til chars fremfor en peger til en liste af chars. Erklæringen

char * type[10];

gør type til en liste af 10 pegere til char's. Således er type[i], 0=<i/>
Eftersom du endnu ikke har svaret gå jeg udfra du ikke har læst #1 og #2 og er gået videre. Så lad mig hjælpe dig med de øvrige problemer der er.

Det første problem er som nævnt i #1 at scanf terminerer formatering efter '%s' så snart den støder ind i white-space. Det gør det umuligt at læse flere ord, f.eks. 'Fast Food' og 'Bonne Cuisine'.

En løsning er at anvende a-flaget i scanf. Det bevirker at scanf benytter dynamisk strenginput, d.v.s. selv allokerer den til den læste streng nødvendige plads. Så er man fri for at bekymre sig om at allokere plads selv. Dernæst kan man eksplicit angive mængden af de karaketerer, scanf skal matche med. Når man anvender a-flaget skal scanf have en peger til en peger til en char, altså en variabel af typen char**, som argument. Eksempel:

char* type;
scanf("%a[ a-zA-Z0-9]",&type)

type er en peget til en char og dermed er &type - adressen af pegeren - en peger til en peger til en char. Da vil scanf matche enhver inputkarakter i den i klammerne [..] angivne mængde, allokere den nødvendige plads og returnere det formatterede input i type. I det konkrete tilfælde tillades input karakterer som er enten et mellemrum, et lille bogstav fra og med a til og med z, et stort bogstav fra og med A til og med Z eller et ciffer fra og med 0 til og med 9.

Hvis du vil bruge denne konstruktion skal der foretages følgende ændringer:

Erklæring:
----------
char *type[max_food];
->
char* type;

scanf:
------
scanf("%s", type);
->
scanf("%a[ a-zA-Z]",&type)

strncmp:
--------
if (strcmp(type[0],rest[i].food) == 0){
->
if (strcmp(type,rest[i].food) == 0){


Næste problem er din bubble() funktion. Den sorterer altid hele listen, d.v.s. samtlige 10 restauranter. Den burde blot sortere de j restauranter som matcher den type brugren valgte.

Du skal blot give j som heltalsparameter til bubble()

void bubble();
->
void bubble(int);


bubble()
->
bubble(j)

void buble(){
->
void bubble(int k){

og dernæst erstatte max_restaurants med k overalt i bubble.

Så burde det virke. Så kan du selvfølgelig overveje at indføre sikkerhedsfunktioner, der skal forhindre crash såfremt der indtastes et uventet valg. Du kunne f.eks. blot smide dit brugerinput i en while-løkke som ikke terminerer medmindre der er indtastet et brugbart valg.

Hej Fixer

Mange, mange tak for din lange udredning!

Du må undskylde det sene svar, men programmet voldte mig en del problemer - og jeg besluttede mig for, at jeg først ville skrive igen, når det virkede (for du er altid så venlig til at svare, så ville ikke stille et spørgsmål, før jeg virkelig havde forsøgt alt, hvad jeg kunne).

Så nu virker programmet, og hvor er jeg stolt ;)


Lige nu er jeg i gang med at kode vores eksamensopgave sammen med 2 andre. Vi skal kode et hangman-spil, og det er faktisk ganske sjovt :)