Fehlerabfrage!

Belli

Hinter "Eingegebenes Zeichen ist ....

solltest Du ein \n machen, der Übersichtlichkeit wegen.

bandchef

Sorry, mekr da grad nicht so was das soll...

bandchef

Ist die bestätigung mit Enter schuld?

bandchef

Das ganze sieht jetzt so aus:

#include<stdio.h>

int main()
{
    char zeichen;
	int zahl;

	zahl = 0;

	do
	{
		zeichen = getchar();
		//printf("\n*****************\nEingegeben: %d\n", zeichen);

		if((zeichen < 48) || (zeichen > 57))
		{
			printf("Eingegebenes Zeichen ist keine Zahl! \n");
		}
		else
		{
			zahl = zeichen - 48;
			printf("Eingegebenes Zeichen ist Zahl! \n");
		}

	}
	while(zahl != '\n');

return 0;
}

Wenn ich nun z.B. auf meinem Ziffernblock eine 5 drücke, bekomm ich als Ausgabe beide Meldungen! Ich verstehs echt nicht was da los ist...

Belli

Ja, Du kriegst doch nach Deinem eingegebenen Zeichen immer noch die 10 für die Enter-Taste.

Belli

Kannst ja mal anstelle von 5 eine Zahl mit mehr Ziffern eingeben, zB 5468 ... oder so.

Für (zeichen < 48) || (zeichen > 57) gibt es isdigit() aus der <ctype.h> bzw. die Negation davon.
Und das verlangt nicht mal ASCII.

bandchef

Ich hab mit nachfolgenden Code mal versucht, dass ich mir das Return, also die 10, rausfilter. Aber irgendwie ist es mir nicht so gelungen. Die Return-Taste ist immer noch mitdabei!

#include<stdio.h>

int main()
{
    char zeichen;
    int zahl;

    zahl = 0;

    do
    {
        zeichen = getchar();
        //printf("\n*****************\nEingegeben: %d\n", zeichen);

        if((zeichen < 48) || (zeichen > 57) || (zeichen == 10))
        {
            printf("Eingegebenes Zeichen ist keine Zahl! \n");
        }
        else
        {
            zahl = zeichen - 48;
            printf("Eingegebenes Zeichen ist Zahl! \n");
        }

    }
    while(zahl != '\n');

return 0;
}

Bei if((zeichen < 48) || (zeichen > 57) || (zeichen == 10)) liegt 10 doch schon bei < 48

Belli

#include<stdio.h>

int main()
{
    char zeichen;
    int zahl;

    zahl = 0;

    do
    {
        zeichen = getchar();
        //printf("\n*****************\nEingegeben: %d\n", zeichen);

        if(zeichen == 10)
            continue;

        if((zeichen < 48) || (zeichen > 57))
        {
            printf("Eingegebenes Zeichen ist keine Zahl! \n");
        }
        else
        {
            zahl = zeichen - 48;
            printf("Eingegebenes Zeichen ist Zahl! \n");
        }

    }
    while(zahl != '\n');

return 0;
}

orioon

getchar liefert auch eigentlich ein int und kein char zurück

pferdefreund

if ((zeichen < '0') || (zeichen > '9')) sollte es doch tun und hats in
Programmen von mir auch schon getan...