Koło fortuny

Martyna Michalska
  • Rejestracja:ponad 5 lat
  • Ostatnio:około 4 lata
  • Postów:10
0

Mam z poniższym kodem jeden problem (i zapewne jeszcze trochę o których nawet nie wiem, a Was zabolą oczy), a mianowicie - ustawiłam ilość prób na 25, każde wpisanie litery skutkuje dwukrotną inkrementacją licznika prób w pętli, jakby odczytywane i sprawdzane zostawały dwa znaki, program kończy się po połowie zadeklarowanej ilości prób a prośba o podanie litery wyświetla się za każdym razem podwójnie, jeśli podam dwie litery potrójnie itd. Gdzie leży problem?

Kopiuj
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
int main ()
{
    char **zbior = (char**) malloc (5 * sizeof(char*));
    for(int i=0;i<5;i++)
    {
        zbior[i] = (char*) malloc (30 * sizeof(char*));
    }
    zbior[0] = "co w trawie piszczy";
    zbior[1] = "marchewkowe pole";
    zbior[2] = "gibraltar";
    zbior[3] = "stany zjednoczone";
    zbior[4] = "rwa kulszowa";

    char word[30];
    char input[30];
    char litera;

//Losowanie hasla
    int losowana;
    srand(time(NULL));
    losowana = rand() % 5;
    if (losowana==0) strcpy(word,zbior[0]);
    else if (losowana==1) strcpy(word,zbior[1]);
    else if (losowana==2) strcpy(word,zbior[2]);
    else if (losowana==3) strcpy(word,zbior[3]);
    else if (losowana==4) strcpy(word,zbior[4]);

    strcpy(input, word);

//Szyfrowanie pola do odgadywania
    for(int i=0;word[i]!=NULL;i++)
    {
        if(word[i]!=' ')
        input[i]='_';
        else
        input[i]=' ';
    }
    printf("slowo: %s\n", input);

    int licznikProb=0;
    while(licznikProb<=25)
    {


        printf("Podaj litere: ");
        scanf("%c", &litera);
        licznikProb++;
        for (int i=0; word[i]!=NULL; i++)
        {
            if(litera==word[i])
                input[i]=litera;
        }

        printf("slowo: %s", input);
        if(strcmp(word,input)==0)
        {
            printf("\nWygrales!"); break;
        }

    }
    if (licznikProb>25) puts("\nWykorzystano wszystkie proby. Koniec gry.");

//Zwalnianie pamieci
    for (int i=0;i<5;i++)
    {
        free (zbior[i]);
        zbior[i]=NULL;
    }
    free(zbior);
    return 0;



}

edytowany 2x, ostatnio: kq
kq
Moderator C/C++
  • Rejestracja:prawie 12 lat
  • Ostatnio:około 4 godziny
  • Lokalizacja:Szczecin
3
Kopiuj
scanf("%c", &litera);

Jeśli wpisujesz np. x<enter> to <enter> jest traktowane jako osobny znak - \n

Kopiuj
scanf("%c", &litera);
if(litera == '\n')
    continue;

Martyna Michalska
tak właśnie myślałam że to ten enter, dzięki wielkie!
Martyna Michalska
wydaje mi się jednak że dalej nie naprawia to problemu naliczania się kolejnych iteracji, można zrobić tak żeby w ogóle nie łapało tego entera?
kq
Możesz spróbować " %c" (spacja przed procentem), ale nie jestem pewien czy to dla znaku będzie działało
Martyna Michalska
zadziałało, dzięki ;)
AK
  • Rejestracja:prawie 7 lat
  • Ostatnio:około miesiąc
  • Postów:3561
3

Skoro alokujesz bufory, czyli są zdolne do przyjęcia danych. Robi się to przez strcpy().

*( W tym prostym programie nie występuje pewien konflikt. Tu można zapytać, czy alokowanie jest konieczne, skoro przypisujesz stałe stringi. Można by było to zrobić "zamiast", tzn od razu przypisać.
Jednak w ogólnym przypadku alokowanie to jest rozwiązanie OK, na przykład w miarę rozwoju programu stringi by pochodziły z pliku, czy w inny sposób "dynamiczne". )
*

Kopiuj

for(int i=0;i<5;i++)
    {
        zbior[i] = (char*) malloc (30 * sizeof(char*));
    }
    strcpy(zbior[0], "co w trawie piszczy");
    strcpy(zbior[1], "marchewkowe pole");
// itd

W dłuższych sekwencjach wyrażeń jak ta, używa się switch ... case, to jest uważane za lepsze pod względem stylu (poczytaj).
Tu jednak warto zauważyć, że ta sama liczba występuje dwa razy, jest taka sama.

Więc

Kopiuj

if (losowana==0) strcpy(word,zbior[0]);
    else if (losowana==1) strcpy(word,zbior[1]);
    else if (losowana==2) strcpy(word,zbior[2]);
    else if (losowana==3) strcpy(word,zbior[3]);
    else if (losowana==4) strcpy(word,zbior[4]);

można zmienić na

Kopiuj
   // mamy pewność, ze 'losowana jest 0,,4 - jak nie, trzeba by sprawdzić
  strcpy(word,zbior[losowana]);

W konsekwencji zagadnienia pierwszego tracisz alokowane wskaźniki (zamazujesz je adresami stałymi). I zwalniając śmieci jest UB - Udefined Behavbiour czyli rozumieć jako błąd, który może się nie ujawnić. Więc w nie poprawionym kodzie to by było źle - ale przecież poprawiamy :)

Kopiuj
//Zwalnianie pamieci
    for (int i=0;i<5;i++)
    {
        free (zbior[i]);
        zbior[i]=NULL;
    }

Bo C to najlepszy język, każdy uczeń ci to powie
edytowany 1x, ostatnio: AnyKtokolwiek
Zobacz pozostałe 4 komentarze
_13th_Dragon
... Przez podejście "pro" przez początkujących ...
AK
Widzę "pro" dyskusję, ale wszyscy "zapomnieli" podać nazwę funkcji strcpy_s https://en.cppreference.com/w/c/string/byte/strcpy
Martyna Michalska
to w sumie co zamiast? próbowałam zamiast używania tej funkcji po prostu to przypisywać za pomocą "=" ale działało jakoś pokracznie albo wcale, już nie pamiętam (niby można napisać własne strcpy, które sprawdza rozmiar bufora, ale na pewno można to zrobić jeszcze inaczej)
AK
Masz linijkę wyżej
_13th_Dragon
to nie jest standard C tylko taka odskocznia od M$

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.