Eingabe nach If-Anweisung nicht möglich

Peter Viehweger

so gehts aber, oder nicht?

edit: irgendwas stimmt da nicht...

Joe_der_neue

Hallo und danke soweit!

Ich habe nun fgets mit scanf und die If-Anweisung mit einer switch-case Anweisung ersetzt.
Wenn ich nun wieder die 1 eintippe bekomme ich wieder den Text "Bitte Kontrolltext eingeben" im Terminal angezeigt, dann passiert für einen Bruchteil nichts, dann aber kommt folgende Fehlermeldung: "Speicherzugriffsfehler (Speicherabzug geschrieben)".

Ich verstehe hier meinen Logik- bzw. Programmierfehler nicht.

Könnte hier ein Profi ein Beispiel geben wie der Code richtigerweise lauten müsste? Danke!

wob

@Joe_der_neue dann zeig doch mal deinen neuen, fehlerhaften Code mit switch-case.

Beim Kompilieren kannst & solltest du außerdem Warnungen aktivieren (bei gcc mit mindestens -Wall -Wextra). Und außerdem könntest du deinen Code auch mal durch clang-tidy laufen lassen. Gut möglich, dass die dein Fehler da schon angezeigt wird.

Peter Viehweger

so?

#define _CRT_SECURE_NO_WARNINGS

#include <stdio.h>
#include <string.h>

void main()
{
	int option;
	char satz[100];

	printf("(1) Option A        (2) Option B \n > ");
	fgets(satz, sizeof(satz) / sizeof(*satz) - 1, stdin);
	sscanf(satz, "%i", &option);

	if(option == 1)
	{
		printf("\nBitte Kontrolltext eingeben:\n > ");
   	        fgets(satz, sizeof(satz) / sizeof(*satz) - 1, stdin);

		//while(satz[i]){
				//printf("%c",satz[i++]);

	}

	printf("\nZeichenanzahl: %lli\n", strlen(satz)); // %lli!
}

(1) Option A        (2) Option B
 > 1

Bitte Kontrolltext eingeben:
 > hallo dies ist ein test

Zeichenanzahl: 24

DirkB

@Peter-Viehweger sagte in Eingabe nach If-Anweisung nicht möglich:

fgets(satz, sizeof(satz) / sizeof(*satz) - 1, stdin);

Warum das -1 ?

Joe_der_neue

Hi und danke soweit.
Es funktioniert mit Peters Code so wie ich es will, ich habe aber noch Nachfragen zwecks Verständnis:

	printf("(1) Option A        (2) Option B \n > ");
	fgets(satz, sizeof(satz) / sizeof(*satz) - 1, stdin);
	sscanf(satz, "%i", &option);

Mein Verständnis ist, dass der Code Zeile für Zeile abgearbeitet wird. Warum wird fgets hier geschrieben aber erst nach sscanf ausgeführt?

printf("\nZeichenanzahl: %lli\n", strlen(satz)); // %lli!

Hier erscheint folgende Meldung:
"format ‘%lli’ expects argument of type ‘long long int’, but argument 2 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=]"

Ändere ich das auf "li" funktioniert alles bestens.

Joe_der_neue

@wob Mir werden die Warnungen im Terminal angezeigt sobald ich mit gcc compiliere.

Code::Blocks sollte doch auch einen eigenen Debuger haben?! Gibt es da eine Möglichkeit den Code Zeile für Zeile zu compilieren und bei Fehlern die entsprechende Meldung zu erhalten? Ich kenne das so ähnlich aus VBA.

SeppJ

@Joe_der_neue sagte in Eingabe nach If-Anweisung nicht möglich:

Mein Verständnis ist, dass der Code Zeile für Zeile abgearbeitet wird. Warum wird fgets hier geschrieben aber erst nach sscanf ausgeführt?

Wie kommst du darauf? Das passiert schon alles schön nacheinander. Beachte, dass Peter-Viehweger hier einen anderen Weg geht, als andere Antworten erklärt haben. Erst liest er eine ganze Zeile in einen String, und dann holt er aus diesem String eine Zahl heraus. sscanf ist was anderes als scanf.

printf("\nZeichenanzahl: %lli\n", strlen(satz)); // %lli!
Hier erscheint folgende Meldung:
"format ‘%lli’ expects argument of type ‘long long int’, but argument 2 has type ‘size_t’ {aka ‘long unsigned int’} [-Wformat=]"

Ändere ich das auf "li" funktioniert alles bestens.

Weil beides falsch ist Oder zumindest nicht richtig. Was genau ein size_t ist, hängt vom System ab, daher bekommt man solche Probleme, wenn man sich auf %li oder %lli festlegt. Zumal das auf jeden Fall ein unsigned Typ (%u) ist, daher ist alles mit %i sowieso die falschge Richtung. Der portable Formatspezifizierer für einen size_t ist %zu.

(Ganz uralte Compiler können evtl. kein %zu. In dem Fall nimmt man einen Typ, der garantiert groß genug ist, und castet dahin. Etwa printf("%llu", (unsigned long long) value);)

Peter Viehweger

@DirkB sagte in Eingabe nach If-Anweisung nicht möglich:

Warum das -1 ?

weil ich die dokumentation nicht gelesen / verstanden habe.

@SeppJ sagte in Eingabe nach If-Anweisung nicht möglich:

Der portable Formatspezifizierer für einen size_t ist %zu.

ja ich erinnere mich.

Joe_der_neue

Alles klar. Ich denke mein C- Büchlein ist da etwas zu alt oder ggf. kommen diese Funktionen und Platzhalter in den späteren Kapiteln...

Peter Viehweger

wahrscheinlich nicht, weil es irgendwie keine neuen bücher für c gibt. um das lesen der dokumentation dieser ganzen funktionen im internet, oder wo auch immer, wirst du nicht drum herumkommen. das buch ist nur quasi "starthilfe".

SeppJ

Dieser Beitrag wurde gelöscht!

DirkB

@Joe_der_neue sagte in Eingabe nach If-Anweisung nicht möglich:

Alles klar. Ich denke mein C- Büchlein ist da etwas zu alt oder ggf. kommen diese Funktionen und Platzhalter in den späteren Kapiteln...

HAst du die den schon mal die "Linkliste für Neuling" hier im Forum angesehen?
https://www.c-plusplus.net/forum/topic/300567/linkliste-für-neulinge

Auf Wikipedia sind die Änderungen von C zusammengefasst: https://de.wikipedia.org/wiki/Varianten_der_Programmiersprache_C

Und in den Referenzen wird sogar gezeigt, wann die Änderungen eingeführt wurden. Beispiel für printf, die gelben Zellen gab es mit C99: https://cplusplus.com/reference/cstdio/printf/
(Es sollte einem dabei klar sein, wie der Formatspecifier aufgebaut ist)