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

scanf - niezrozumiałe zachowanie (język c)
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
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

Kopiuj
#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;
	}

edytowany 3x, ostatnio: Waran3
Miang
a kod mamy se zgadnąć?
Waran3
No już...nie jestem taki prędki :-)
kq
Moderator C/C++
  • Rejestracja:prawie 12 lat
  • Ostatnio:minuta
  • Lokalizacja:Szczecin
1

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

Kopiuj
    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);
```

Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
0

@kq: Bez tytułu.png

Niestety bez zmian.


edytowany 2x, ostatnio: Waran3
Miang
  • Rejestracja:około 7 lat
  • Ostatnio:3 minuty
  • Postów:1674
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()


dzisiaj programiści uwielbiają przepisywać kod z jednego języka do drugiego, tylko po to by z projektem nadal stać w miejscu ale na nowej technologii
edytowany 1x, ostatnio: kq
Miang
z strumieniem czyli możne to być klawiatura getc(stdin);
Waran3
Czy za pomocą getc można wprowadzać znaki z klawiatury? Wszędzie jest opis czytania znaków z pliku.
Waran3
Kurcze tu są te same problemy. ;-)
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
0

Bez tytułu.png

Jest super, dziękuję! :-)


edytowany 1x, ostatnio: Waran3
MarekR22
Moderator C/C++
  • Rejestracja:około 17 lat
  • Ostatnio:9 minut
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.


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.
edytowany 1x, ostatnio: MarekR22
Waran3
Dzięki za wyjaśnienie. :-)
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
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

Kopiuj
    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");

edytowany 1x, ostatnio: Waran3
Barstok
  • Rejestracja:ponad 4 lata
  • Ostatnio:20 dni
  • Postów:9
0

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

Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
0

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


Barstok
  • Rejestracja:ponad 4 lata
  • Ostatnio:20 dni
  • Postów:9
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

edytowany 1x, ostatnio: Barstok
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
0

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

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

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

edytowany 2x, ostatnio: Waran3
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
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.

Kopiuj
    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:

Kopiuj
//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;
}



Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:prawie 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
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.

"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 2x, ostatnio: Shalom
Waran3
Dzięki za wskazówki i uwagi. :-) Mieszam bo ćwiczę :-)
MarekR22
Moderator C/C++
  • Rejestracja:około 17 lat
  • Ostatnio:9 minut
3

Komentarze w stylu:

Kopiuj
//////////////////////////// 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ę.


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.
edytowany 2x, ostatnio: MarekR22
Waran3
Dzięki za wskazówki. :-)
Waran3
  • Rejestracja:około 4 lata
  • Ostatnio:ponad rok
  • Lokalizacja:Zielonka
  • Postów:103
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 :-)

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

edytowany 1x, ostatnio: Shalom

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.