Problem: Variable wird nach gets 0 gesetzt.

Hallo, ich bin dabei ein Hangman Spiel zu programmieren.
Das Programm funktioniert bis auf den Lebenspunkte-Zähler.
Ich setze die Leben auf 10 und möchte, dass sie bei jedem Fehlversuch um 1 gesenkt werden. Sie haben den Wert 10 bis gets(input) Zeile 41 im Code. Dort werden sie auf 0 gesetzt. Ich kann mir dies leider nicht erklären und würde mich daher über Hilfe freuen.

Viele Grüße

#include <stdio.h>
#include <strings.h>

int main()
{

    unsigned long length;
    int i=0, k=0, n;
    int lifepoint;
    char input[1]; //creates string for the input
    char word [10]; //creats a string for the input word 10 chars maximum
    char sword [10];  //creats a string for the * word

    printf("Enter a word. (max 10 characters)\n");
    gets(word); //user input

    for(n=0; n<20; n++) //clears window
    {
        printf("\n\n\n\n\n\n\n\n\n\n\n\n\n");
    }

    length=strlen(word);  //counts the length of the word

    while (i<length)  //prints for every char of the word a star
    {
        sword[i]='*';
        i++;
    }

    printf("The word is %lu characters long. You got 10 attempts.\n",length); //prints the length of the word
    printf("\n");
    printf("%s\n",sword); //prints the unsolved star word
    printf("\n");

    lifepoint=10;
    while (strcmp(sword, word)!=0 && lifepoint>0)
    {
        printf("Enter a character:");
        gets(input); 
        printf("\n");

        for (k=0; k<length; k++)
        {
            if (input[0]==word[k])
            {
                sword[k]=input[0];
            }

            else if(input[0]!=word[k])
            {
                lifepoint--;
            }
        }

        printf("\n");
        puts(sword);
        printf("%d lifepoints left!\n",lifepoint);
    }

    printf("\n");
    printf("Great! You solved the word. =)\n");

    if (lifepoint==0)
    {
        printf("You are out of lifes!\n");
    }

    return 0;
}

SeppJ

Was ist denn, wenn gets mehr als 0 Zeichen einliest? Dein Eingabefeld hat nur Platz für ein Element und das braucht gets bereits für die Nullterminierung.

gets ist gefährlich und sollte generell niemals benutzt werden. Daher ist es auch im aktuellen Standard als "deprecated" markiert, d.h. eventuell wird es in Zukunft gar kein gets mehr geben. Der Grund: Genau das, was hier passiert ist: Es gibt keine Möglichkeit, gets auf eine Maximalgröße der Eingabe zu beschränken. Es wird stattdessen fröhlich Speicherplatz anderer Variablen überschrieben, wenn der Nutzer mehr eingibt, als man Platz vorgesehen hat.

gets ist hier sowieso fehl am Platz, da du doch nur ein Zeichen möchtest. Alleine schon ein Array mit nur einem Element sollte doch ein Warnzeichen beim Programmieren sein, dass in der Logik etwas nicht stimmt. Nimm getchar oder getc.

Ändere bei dem Text in Zeile 14 mal die 10 in 9 um. Dann bleibt noch platz für den Terminator

sword braucht noch einen Stringterminator '\0'

Für jeden falschen Buchstaben im Suchwort, wird dir ein Lebenspunkt abgezogen. (pro Eingabe)

if (input[0]==word[k])
            {  ....
            }
            else if(input[0]!=word[k])  // Das if (...) ist hier überflüssig

Denn du kommst an die Stelle nur, wenn input[0]!=word[k] wahr ist
(Das tritt dann ein, wenn input[0]==word[k] falsch ist)

Vielen Dank für eure schnelle Hilfe.
Ich habe eure Tips soweit angewendet. Leider macht der Lebenspunkte Zähler noch immer nicht das was er soll. Er zieht bei jeder, auch bei einer richtigen, Eingabe einen Punkt ab.

#include <stdio.h>
#include <strings.h>

int getchar(void);

int main()
{

    unsigned long length;
    int i=0, k=0, l=0, n, t=0;
    int lifepoint=10;
    int input,input2;//creates string for the input
    char word [11]; //creats a string for the input word 10 chars maximum
    char sword [11];  //creats a string for the * word

    printf("Enter a word. (max 10 characters)\n");

    for (i = 0; i < 10; i++)
    {
        input = getchar();
        if (input == '\n')
        {
            for (i = i; i < 10; i++)
            {
                word[i] = '\0';
            }

            break;
        }
        word[i] = input;
    }

    for(n = 0; n < 20; n++) //clears window
    {
        printf("\n\n\n\n\n\n\n\n\n\n\n\n\n");
    }

    length=strlen(word);  //counts the length of the word

    while (l<length)  //prints for every char of the word a star
    {
        sword[l]='*';
        l++;
    }

    sword[length]='\0';

    printf("The word is %lu characters long. You got 10 attempts.\n",length); //prints the length of the word
    printf("\n");
    printf("%s\n",sword); //prints the unsolved star word
    printf("\n");

    while (strcmp(sword, word)!=0)
    {
        printf("Enter a character:");
        input2=getchar();
        printf("\n");

       for (k=0; k<=length-1; k++)
        {
            if (input2==word[k])
            {
                sword[k]=input2;
                t=1;
            }

            else
                t=0;
        }

        if (t==0)
        {
            lifepoint--;
        }
        t=0;
        printf("\n");
        puts(sword);
        printf("%d tries left!\n",lifepoint);
    }

    printf("\n");

    if (lifepoint>0)
    {
        printf("Great! You solved the word. =)\n");
    }

    else
    {
        printf("You are out of lifes!\n");
    }

}

rstdd schrieb:

Vielen Dank für eure schnelle Hilfe.
Ich habe eure Tips soweit angewendet. Leider macht der Lebenspunkte Zähler noch immer nicht das was er soll. Er zieht bei jeder, auch bei einer richtigen, Eingabe einen Punkt ab.

Was passiert denn, wenn ein Buchstabe richtig ist und die for-Schleife in Z. 61 noch nicht zu ende?

Ist <strings.h> von Dir, oder meinst Du <string.h> ?

Die for Schleife läuft nochmal durch und überspringt das getchar wodurch das t natürlich auf 0 gesetzt wird und somit ein leben abgezogen wird.

tdd schrieb:

Die for Schleife läuft nochmal durch und überspringt das getchar wodurch das t natürlich auf 0 gesetzt wird und somit ein leben abgezogen wird.

Das auch. ²
Aber das meinte Furble Wurble nicht.

Mal angenommen du suchst das Wort foo und gibst ein f ein.
Dann kommst du an erster Stelle stimmen die Buchstaben überein.
Aber dann überprüfst du die zweite Stelle und da stimmt es nicht mehr.

Du bekommst t = 0, obwohl das Zeichen richtig war.

Noch was:

Wofür ist den Zeile 23ff da?

for (i = i; i < 10; i++)
            {
                word[i] = '\0';
            }

Vor allem mit der umgebenden Schleife

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

²Das getchar wird nicht übersprungen, es wird das '\n' von der Entertaste eingelesen.

Stimmt, das else t=0 macht keinen Sinn.

So funktioniert es, allerdings muss ich den Buchstaben zwei mal eingeben:

while (strcmp(sword, word)!=0 && lifepoint>0)
    {
        printf("Enter a character:");
        while (getchar() != '\n' && getchar() != EOF)
        {
            input2 = getchar();

            printf("\n");

            for (k=0; k<=length-1; k++)
            {
                if (input2==word[k])
                {
                    sword[k]=input2;
                    t=1;
                }
            }
        }

        if (t==0)
        {
            lifepoint--;
        }
        t=0;
        printf("\n");
        puts(sword);
        printf("%d tries left!\n",lifepoint);
    }

    printf("\n");

Du rufst da jetzt bis Zeile 6 bis zu dreimal getchar() auf. Ist das sinnvoll?

Und ein Zeichen in einem String suchen gibt es auch schon in <string.h> : strchr() .

So ich möchte mich bei Allen für die freundliche Unterstützung danken.

Das Programm macht nun endlich das was es soll.

Viele Grüße

Ich habe es wie folgt gelöst:

while (strcmp(sword, word)!=0 && lifepoint>0)
    {
        printf("Enter a character:");

            scanf("%d", &input2);
            input2 = getchar();
            char *pos=strchr (word,input2);;
            if (pos)
            {
                while (pos!=NULL)
                {
                    sword[pos-word]=word[pos-word];
                    pos=strchr(pos+1,input2);
                }
            }

            else
            {
                lifepoint--;
            }

        printf("\n");
        puts(sword);
        printf("%d tries left!\n",lifepoint);
    }

SeppJ schrieb:

Daher ist es auch im aktuellen Standard als "deprecated" markiert, d.h. eventuell wird es in Zukunft gar kein gets mehr geben.

Der aktuelle Standard ist C11 und der sagt (zumindest der Committee Draft):

http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf schrieb:

— removed the gets function (<stdio.h>)

SeppJ

tddrs schrieb:

printf("Enter a character:");
        
            scanf("%d", &input2);
            input2 = getchar();

Abenteuerlich. Und damit meine ich nicht nur die Einrückung. Welche Art von Datentyp liest du da wie?

if (pos)
            {
                while (pos!=NULL)

Sind die beiden Zeilen vom gleichen Programmierer?