scanf - niezrozumiałe zachowanie (język c)

1

Problem wydaje się trywialny ale walczę z nim już cały dzień.
W jednym z programów potrzebuję aby użytkownik wprowadził dwa pojedyncze, dowolne znaki z klawiatury (char).
W sumie niby proste...scanf i wydaje się po temacie. Niestety ale ta funkcja zachowuje się nie tak jak bym sobie tego życzył - delikatnie mówiąc. :-)
Jak dla mnie program wariuje.
Po uruchomieniu program pyta o pierwszy znak. Wpisuję coś z klawiatury, najczęściej 1 i wciskam enter. Teoretycznie program powinien przejść do następnej linii pytając o drugi znak. Niestety program po wprowadzeniu 1 + enter jakby czeka na coś jeszcze, wpisuję więc np. 2 + enter i wtedy pada dopiero pytanie o drugi znak. Tu już wydaję się wszystko ok bo po wprowadzeniu 3 program pyta o kolejną zmienną ( w tym programie, dla testów używam trzech zmiennych) i tu podaję 4 + oczywiście enter. Program wypisuje co mu kazałem. Okazuje się że kolejne znaki przypisane do zmiennych to 1,2 i 3. Czwórka przepadła. Podczas różnych prób miałem inne zachowania. Np. program po wprowadzeniu pierwszego znaku jakby pomijał kolejne scamfy i od razu wyświetlał wyniki. Próby typu wstawienie spacji przed %c czy położenie jedynki %1c właściwie nie wnoszą niczego konkretnego. Próbowałem też scanf_s. I tyle z grubsza. W akcie desperacji piszę tutaj z prośbą o poradę :-)

scr2.png

#include <stdio.h>

char tab[3];
char zmienna_1;
char zmienna_2;
char zmienna_3;



int main(void)
{
	printf("Przypisz znak zmiennej_1= ");
	scanf("%1c\n", &zmienna_1);

	printf("Przypisz znak zmiennej_2= ");
	scanf("%1c\n", &zmienna_2);

	printf("Przypisz znak zmiennej_3= ");
	scanf ("%1c\n", &zmienna_3);

	printf("Zmienna 1= %c\n", zmienna_1);
	printf("Zmienna 2= %c\n", zmienna_2);
	printf("Zmienna 3= %c\n", zmienna_3);
	
	tab[0] = zmienna_1;
	tab[1] = zmienna_2;
	tab[2] = zmienna_3;

	printf(" Wartosc tablic 0,1,2= %c %c %c\n", tab[0], tab[1], tab[2] );
	if (tab[1] == tab[2])
		printf("Tab 1 i tab 2 to znaki zgodne= %c %c\n", tab[1], tab[2]);
	printf("Tab 1 i tab 2 to znaki rozne= %c %c\n", tab[1], tab[2]);
	
	if (tab[1] == '*')
		printf("Tab 1 to znak gwaiazdki %c\n", tab[1]);
	else
		printf("Tab 1 to inny znak niz gwiazdka= %c\n", tab[1]);
		

	return 0;
	}
1

Zapewne problemem jest buforowanie wyjścia. https://stackoverflow.com/a/1716621/2456565
Spróbuj:

    printf("Przypisz znak zmiennej_1= ");
    fflush(stdout);
    scanf("%1c\n", &zmienna_1);

    printf("Przypisz znak zmiennej_2= ");
    fflush(stdout);
    scanf("%1c\n", &zmienna_2);

    printf("Przypisz znak zmiennej_3= ");
    fflush(stdout);
    scanf ("%1c\n", &zmienna_3);
```
0

@kq: Bez tytułu.png

Niestety bez zmian.

4

można też dodać spację i wyrzucić \n
printf("Przypisz znak zmiennej_1= ");
scanf(" %1c", &zmienna_1);

printf("Przypisz znak zmiennej_2= ");
scanf(" %1c", &zmienna_2);

printf("Przypisz znak zmiennej_3= ");
scanf (" %1c", &zmienna_3);

ale zazwyczaj to wolę getc()

0

Bez tytułu.png

Jest super, dziękuję! :-)

5

Fa

Miang napisał(a):

można też dodać spację i wyrzucić \n

printf("Przypisz znak zmiennej_1= ");
scanf(" %1c", &zmienna_1);

printf("Przypisz znak zmiennej_2= ");
scanf(" %1c", &zmienna_2);

printf("Przypisz znak zmiennej_3= ");
scanf (" %1c", &zmienna_3);

ale zazwyczaj to wolę getc()

Szkoda tylko, że nie ma wyjaśnienia czemu nie działa.
String formatujący "%1c\n" działa tak:

  • %1c wczytaj pojedynczy znak jakikolwiek on by był
  • \n - to jest biały znak, a wszystkie białe znaki w napisie formatującym, powoduje konsumpcję dowolnej ilości białych znaków, aż do napotkania nie białego znaku, który będzie czekał na następne wczytanie.

Efekt jest taki, że po wprowadzeniu pierwszego znaku, tak długo jak będziesz po nim wpisywał białe znaki (spacja, tabulator, koniec linii), funkcja scanf nie odda kontroli.

Teraz fix przesuwa biały znak \n z końca pierwszego napisu formatującego, na początek drugiego napisu formatującego (jako spacje), więc to drugie scanf teraz zjada wszystkie białe znaki po wpisanym pierwszym znaku.

0

Postanowiłem pozwolić programowy wstawić spację jako znak z klawiatury.
Jak rozumiem scanf() tego nie umożliwia więc posłuchałem się @Miang i użyłem getc().
No i problem się niejako powtórzył.
Pierwsza linia z getc() działa jak należy za to druga już nie. Program
wyświetla pytanie o drugi znak ale nie czeka na jego wprowadzenie tylko wyświetla natępna linię
kodu, czyli pytanie o ilość wierszy.
screenshot-20210308201810.png

    printf("Wpisz dowolny znak z klawiatury= ");
    jedynka = getc(stdin);
   
    printf("Wpisz drugi dowolny znak z klawiatury= ");
    zero = getc(stdin);

    
    // Pobranie danych
    printf("Podaj ilosc wierszy(y): ");
    scanf("%d", &wiersze);
    printf("Podaj ilosc kolumn(x): ");
    scanf("%d", &kolumny);
    printf("\n");
0

W buforze ci zostaje '\n'. Musisz to wyczyścić ;)

0

@Barstok: Niby wstawiłem fflush(stdout) ale bez skutku :-)

1

No nie dziwię się, bo tu chodzi o stdin xD Tak czy siak nie powinno się używać fflush(stdin), najlepiej to pochłonąć getcharem, między pobraniem tych dwóch znaków ;)
Tzn. while(getchar()!='\n') To ci wyczyści cały nadmiarowy input

0

@Barstok: No tak, znalazłem też to rozwiązanie ;-)
Zrobiłem tak ( dodałem getchar(); ) i jest ok:

    printf("Wpisz dowolny znak z klawiatury= ");
    jedynka = getc(stdin);
    getchar();

    printf("Wpisz drugi dowolny znak z klawiatury= ");
    zero = getc(stdin);
    getchar();
    
0

Wszystko działa tak jak powinno z małym wyjątkiem. Czyli właściwie nie wszystko. ;-)
Odpaliłem program docelowy, który rysuje węża na zadeklarowanej dynamicznej podwójnej tablicy .
Program, podczas rysowania może korzystać z dowolnych znaków, które podaje się na początku.

    printf("Wpisz dowolny znak z klawiatury dla węża= ");
    jedynka = getc(stdin); //zamiana numeru znaku klawiatury na znak klawiatury
    getchar();
    printf("jedynka= %c\n",jedynka);

    printf("Wpisz dowolny znak z klawiatury dla pola wokół węża= ");
    zero = getc(stdin);
    getchar();
    printf("zero= %c\n", zero);

Potem program pyta o ilość wierszy i kolumn i zaczyna działać.
Na końcu, po narysowaniu węża pada pytanie czy powtórzyć operację.
Jak T to wracamy do pierwszego pytania o dowolny znak z klawiatury dla węża.
I tu pojawia się problem bo po wpisaniu znaku, zmienna jedynka jest pusta, tak samo zmienna zero.
Robiłem testy zapętlając tylko te pytania i wszystko śmigało.
Myślałem, że może czyszczenie pamięci coś mąci ale po wyłaczeniu tego fragmentu kodu jest ciągle źle.
No i znowu proszę o radę...:-)

Tu cały program:

//Wąż ver 1.1 

#include <stdio.h>
#include <malloc.h>
#include <locale.h>
#include <conio.h>
#include <stdlib.h>



int i = 0, j = 0, a=0;

int main(void)
{

    // Deklaracja zmiennych
    int wiersze;
    int  kolumny;
    int** tab;
    char jedynka;
    char zero ;
    char key;
    

    setlocale(LC_ALL, "polish");// polskie znaki

poczatek:
    i = 0, j = 0;

    printf("Wpisz dowolny znak z klawiatury dla węża= ");
    jedynka = getc(stdin); //zamiana numeru znaku klawiatury na znak klawiatury
    getchar();
    printf("jedynka= %c\n",jedynka);


    printf("Wpisz dowolny znak z klawiatury dla pola wokół węża= ");
    zero = getc(stdin);
    getchar();
    printf("zero= %c\n", zero);
   
    
    // Pobranie danych
    printf("Podaj ilość wierszy: ");
    scanf(" %d", &wiersze);
    printf("Podaj ilość kolumn: ");
    scanf(" %d", &kolumny);
    printf("\n");

           

    // Alokacja pamięci dla tablicy dwuwymiarowej
    tab = (int**)malloc(wiersze * sizeof(int*)); // alokacja pamięci dla wierszy
    for (int i = 0; i < wiersze; i++)
        tab[i] = (int*)malloc(kolumny * sizeof(int)); // alokacja pamięci dla kolumn
        
    for (i = 0; i < wiersze; i++) // wypełnienie tablicy znakiem pierwszym (0) - początek
        for (j = 0; j < kolumny; j++)
            tab[i][j] = zero; // wypełnienie tablicy znakiem pierwszym (0) - koniec

    
  
    i = 0;
    j = 0;
   
    printf("i= %d, j= %d kolumny= %d wiersze= %d tab[0][0]= %c", i, j, kolumny, wiersze, tab[0][0]);
    system("pause");
           
            ///////////////////////////// w prawo ////////////////////////////
    for (j; j < kolumny && tab[i][j] == zero ; j++)
    {

        //printf("W petli w prawo_1 i= %d j= %d tab= %d\n", i, j, tab[i][j]);
        tab[i][j] = jedynka;
    }
            //printf("Po wyjsciu z petli w prawo i= %d j= %d \n", i, j) ;

            j = j - 1;
            tab[i][j] = zero;
                        
            //printf("Koniec bloku w prawo_1   i= %d j= %d\n", i, j);
            //printf("\n");
               
         
            /////////////////////////// w dół ////////////////////////////////

            for (i; i < wiersze && tab[i][j] == zero ; i++)

            {
                //printf("W petli w dol_1 i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                tab[i][j] = jedynka;
            }
            //printf("Po wyjsciu z petli w dol_1 i= %d j= %d \n", i, j );

            i = i - 1;
            tab[i][j] = zero;

            //printf("Koniec bloku w dol_1   i= %d j= %d\n", i, j);
            //printf("\n");

            
                /////////////////////////// w lewo ////////////////////////////////

                for (j; j >= 0 && tab[i][j] == zero; j--)
            {
                //printf("W petli w lewo_1 i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                tab[i][j] = jedynka;
            }
            //printf("Po wyjsciu z petli w lewo_1 i= %d j= %d \n", i, j);

            j = j + 1;
            tab[i][j] = zero;

            //printf("Koniec bloku w lewo_1   i= %d j= %d\n", i, j);
            //printf("\n");

             

            while (a==a) //pętlna nieskończona 
            {
                a++;
                //printf("a= %d wiersze= %d kolumny= %d\n", a, wiersze, kolumny);
                /////////////////////////// w gore_2 ////////////////////////////////
                if (tab[i][j] == zero && tab[i - 1][j] == zero && tab[i - 2][j] == jedynka)
                    goto skok;

                int g = i;
                for (i; tab[i][j] == zero && i >= 0; i--)
                    if (tab[i - 1][j] == jedynka)
                    {
                        //printf("W petli w gore_1  i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                        tab[i][j] = zero;

                    }
                    else
                    {
                        tab[i][j] = jedynka;
                    }
                i = i + 2;
                tab[i][j] = zero;
                if (g - i == 1)
                    goto skok;

                //printf("Po wyjsciu z petli w gore_2 i= %d j= %d \n", i, j);
                //printf("\n");




                /////////////////////////// w prawo2 ////////////////////////////////
                if (tab[i][j] == zero && tab[i][j + 1] == zero && tab[i][j + 2] == jedynka)
                    goto skok;

                int p = j;
                for (j; tab[i][j] == zero && j < kolumny; j++)
                    if (tab[i][j + 1] == jedynka)
                    {
                       // printf("W petli w prawo_2  i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                        tab[i][j] = zero;

                    }
                    else
                    {
                        tab[i][j] = jedynka;
                    }
                j = j - 2;
                tab[i][j] = zero;
                if (j - p == 1)
                    goto skok;


               //printf("Po wyjsciu z petli w prawo_2 i= %d j= %d \n", i, j);
                //printf("\n");

                //////////////////////////// w dol_2 //////////////////////////////////////////////////

                if (tab[i][j] == zero && tab[i + 1][j] == zero && tab[i + 2][j] == jedynka)
                    goto skok;

                int d = i;
                for (i; tab[i][j] == zero && i <= wiersze; i++)
                    if (tab[i + 1][j] == jedynka)
                    {
                       // printf("W petli w dol_2  i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                        tab[i][j] = zero;

                    }
                    else
                    {
                        tab[i][j] = jedynka;
                    }
                i = i - 2;
                tab[i][j] = zero;
                if (i - d == 1)
                    goto skok;


                //printf("Po wyjsciu z petli w dol_2 i= %d j= %d \n", i, j);
                //printf("\n");

                /////////////////////////// w lewo_2 ////////////////////////////////

                if (tab[i][j] == zero && tab[i][j - 1] == zero && tab[i][j - 2] == jedynka)
                    goto skok;

                int l = j;
                for (j; tab[i][j] == zero && j > 0; j--)
                    if (tab[i][j - 1] == jedynka)
                    {
                        //printf("W petli w prawo_2  i= %d j= %d tab= %d\n", i, j, tab[i][j]);
                        tab[i][j] = zero;

                    }
                    else
                    {
                        tab[i][j] = jedynka;
                    }
                j = j + 2;
                tab[i][j] = zero;
                if (l - j == 1)
                    goto skok;

                //printf("Po wyjsciu z petli w lewo_2 i= %d j= %d \n", i, j);
                //printf("\n");

            }



        skok:

            tab[i][j] = jedynka;

           
            
    // Wypisywanie wartości
    for (int i = 0; i < wiersze; i++)
    {
        for (int j = 0; j < kolumny; j++)
        {
            printf("%1c", tab[i][j]);
        }
        printf("\n");
    }
        // Czyszczenie pamięci
    for (int i = 0; i < wiersze; i++)
        free(tab[i]);
        free(tab);
        

        printf("\n");
        printf("Chcesz powtórzyć? (T/N)\n");
        key = _getch(); 
        if (key == 't')
        {
            system("cls");
            goto poczatek;
        }
        
        if (key == 'n');
        {
            system("cls");
            printf("Program zokończył działanie\n");
        }

        
       

    return 0;
}


5
  1. goto... to się nie może udać
  2. Mieszanie scanf, getc, getchar i _getch w jednym kodzie to proszenie się o kłopoty :)
  3. Stawiam ze po scanf(" %d", &kolumny); w buforze wisi znak nowej linii (bo wciskasz enter!) i potem twoj _getch() go zjada, albo jakaś podobna kombinacja.
3

Komentarze w stylu:

//////////////////////////// w dol_2 //////////////////////////////////////////////////

są wyraźna wskazówką, że kod poniżej powinien być zamknięty w osobnej funkcji.
Ba, ten komentarz nawet sugeruje nazwę tej funkcji.

Każda funkcja, która ma więcej niż 10 linii lub więcej niż 2 instrukcje sterujące (if for while switch), powinna byc podzielona na mniejszą funkcję.

0
  1. Stawiam ze po scanf(" %d", &kolumny); w buforze wisi znak nowej linii (bo wciskasz enter!)...

Dobrze obstawiłeś. :-)
Tak jak poprzednio dodałem linię z getchar() i po kłopocie :-)

// Pobranie danych
    printf("Podaj ilość wierszy: ");
    scanf(" %d", &wiersze);
    printf("Podaj ilość kolumn: ");
    scanf(" %d", &kolumny);
    printf("\n");
    getchar(); // dodana linia

1 użytkowników online, w tym zalogowanych: 0, gości: 1