char vergleichen

Michael B schrieb:
> scanf("%d", &choice);
Wartet auf Enter, soll aber ohne sein "(without pressing enter key)"
getch ist zwar nicht standardkonform, aber wäre eine Lösung

1

#include <stdio.h>

2

#include <conio.h>

3

4

int main()

5

{

6

    char c;

7

    while(1){

8

        printf("Choice: ");

9

        c = getch();

10

11

        switch(c){

12

        case 'a':

13

            printf("\nGood morning.\n");

14

            break;

15

        case 'b':

16

            printf("\nGood evening.\n");

17

            break;

18

        case 'c':

19

            printf("\nExit Program.\n");

20

            return 0;

21

            break;

22

        default:

23

            printf("\nInvalid Input: %c\n",c);

24

        }

25

    }

26

    return 0;

27

}

Michael B schrieb:
> char choice;
>
> scanf("%d", &choice);

Bei %d liest scanf eine Dezimalzahl (Ziffern) ein und speichert sie als 
int ab.

Ein int benötigt aber meist mehr Byte als ein char.
Da wird dann Speicher unzulässig überschrieben.

Der Compiler erkennt das mittlerweile und gibt dazu auch eine Warnung 
aus (wenn sie aktiviert sind)

Also Warnlevel hoch setzen (alle Warnungen aktivieren), die Warn7ngen 
beachten und deren Ursache beheben.

DPA schrieb:
> getchar hingegen ist standardkonform. Eventuell muss man dann aber das
> line buffering auf stdin ausschalten:
> http://www.cplusplus.com/reference/cstdio/setbuf/

Wenn denn das Buffering nur innerhalb des Programms stattfindet und 
nicht schon im Terminal.

zitter_ned_aso schrieb:
> lieber getchar()/putchar() nehmen. Wobei diese Funktionen eine
> int-Variable erwarten ;-)

Die geben ein int-Wert zurück. DerCompiler stutzt das dann - wenn nötig 
- bei der Zuweisung zurecht.
Das kann Probleme mit dem Vorzeichen geben, wenn man einen erweiterten 
Code benutzt (was bei Umlauten der Fall ist)


Beim scanf(" %c", &choice); sollte man vor dem %c ein Leerzeichen 
setzen.
Dies überliest mögliche Whitespace (z.B. die Entertaste einer vorherigen 
Eingabe)

Dies ist aber nur bei %c und %[ nötig, da alle anderen Formatspecifier 
dies automatisch tun.

DPA schrieb:
>> Das kann Probleme mit dem Vorzeichen geben, wenn man einen erweiterten
>> Code benutzt (was bei Umlauten der Fall ist)
>
> Blödsinn,

Nein.

> das gibt immer nur ein byte oder EOF zurück. Bei Zeichencodierungen die
> mehr bytes brauchen muss man es halt entsprechend oft aufrufen. Oder
> anders gesagt, die kodierung ist dem ding egal.

Die Kodierung an sich schon, allerdings kann es, falls char 
vorzeichenbehaftet ist, bei negativen Werten (also in der Regel alles 
außerhalb des reinen ASCII-Wertebereichs) zu Problemen kommen. Dann 
liefert getchar in dem int nämlich die negativen Werte als positiv 
zurück. Das heißt, getchar erzeugt intern keinen char, sondern einen 
unsigned char und konvertiert den dann für die Rückgabe nach int. Und 
putchar erwartet es auch wieder in dieser Form. Wenn ich den Wert also 
zwischendurch nach char konvertiere, muss ich das berücksichtigen. Das 
gilt übrigens auch für printf mit %c.

Oder in Standard(C99)-Sprech:

"If the end-of-file indicator for the input stream pointed to by stream 
is not set and a next character is present, the fgetc function obtains 
that character as an unsigned char converted to an int and advances the
associated file position indicator for the stream (if defined)."

(getchar verweist auf getc und das wiederum auf fgetc)

Das ist eins von den Dingen, bei denen ich mich wirklich frage, was K&R 
damals geraucht haben, bevor sie das so entschieden haben.

Bei Binary Data soll man ja jedes Bitmuster von EOF unterscheiden.

Dirk B. schrieb:
> Bei Binary Data soll man ja jedes Bitmuster von EOF unterscheiden.

Das ist in der Praxis zwar üblicherweise gegeben, aber C lässt auch 
Implementationen zu, wo das nicht gilt. Deshalb gibt's die Funktion 
feof().