Fehlerabfrage!

[Belli]

Wo hast Du denn da was geändert? Zeichen 47 und 58 werden immer noch als Ziffern interpretiert.

[bandchef]

#include<stdio.h>

int main()
{
    char zeichen;
    int zahl;

    zahl = 0;

    do
    {
        zeichen = getchar();

        if((zeichen < 48) || (zeichen > 57))
        {
            printf("Eingegebenes Zeichen ist keine Zahl!");

        }
                else
                {
                zahl = zeichen - 48;
                }
    }
    while(zahl != '\n');

return 0;
}

sorry, habs im VS 2010 geändert aber nicht im board...

[Belli]

Und bau mal Deinen else-Zweig so um:

else
{
   zahl = zeichen - 48;
   printf("\nEingegebenes Zeichen ist Zahl!");
}

[bandchef]

Jetzt sieht's so aus:

#include<stdio.h>

int main()
{
    char zeichen;
	int zahl;

	zahl = 0;

	do
	{
		zeichen = getchar();

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

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

return 0;
}

[Belli]

und dann mach mal hinter

zeichen = getchar();

die Anweisung:

printf("\n*****************\nEingegeben: %d\n", zeichen);

Merkst Du was?

[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.

[DirkB]

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

[DirkB]

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...