Das Buch "C - Programmieren von Anfang an"

[XElectriCXxx]

Ja ich habe das Buch und bin bei Kapitel 3.1. Da soll ich eine Konsolenanwendung schreiben das mir dann x und y addiert,multipliziert ect.
Aber der zeigt mir einen Fehler an...

Mein Code:

 Spoiler

#include "stdafx.h"
#include <stdio.h>
#include <stdlib.h>


int main()
{
	int x,y;
	printf("\n\t1 . R E C H E N P R O G R A M M\n");
	printf("\nBitte x eingeben :");
	scanf("%i",&x);
	printf("\nBitte y eingeben :");
	scanf("%i",&y);
	printf("\n %i + %i ist %i",x,y,x+y);
	printf("\n %i - %i ist %i",x,y,x-y);
	printf("\n %i * %i ist %i",x,y,x*y);
	printf("\n %i / %i ist %i",x,y,x/y);
	printf("\n %i %% %i ist %i",x,y,x%y);
	system("pause");
}

Meine Fehlermeldung:

 Spoiler

Fehler	1	error C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.	c:\users\*******\desktop\*****\consoleapplication1\consoleapplication1\consoleapplication1.cpp	14	1	ConsoleApplication1

Kann mir jemand sagen was daran falsch ist??

[Jeoni]

Steht doch in der Fehlermeldung: die Funktion 'scanf' könnte unsicher sein. Es wird stattdessen 'scanf_s' empfohlen.
Also deine 'scanf's durch 'scanf_s's ersetzen, dann dürfte sich das wohl geklärt haben
Mit freundlichen Grüßen
Jeoni

[XElectriCXxx]

ah oke danke.. ja die fehlermeldung war wohl länger als angezeigt.. :P

[snow]

scanf ist halt unsicher wenn du z.B. einen String einliest, da kann es z.B. vorkommen, dass du mehr Speicher benötigst als reserviert ist. Nicht gerade gut für z.B. Systeme, die auf Sicherheit ausgelegt sind.
scanf_s dürfte klappen, alternativ kannst du auch was eigenes schreiben:

Auf die Schnelle:
int einlesen() {

int i = 0;
char tmp = 0;

while ((tmp = getchar()) != '\n') {

if (tmp >= '0' && tmp <= '9')
i = i * 10 + (tmp - '0');
}

return i;

}

Aber das kommt bestimmt noch.

[XElectriCXxx]

^^ scanf_s hat schon geklappt danke ^^

[.SkyneT.]

Btw. ist das keine Fehlermeldung, sondern eine Warnung. :P

[Rullx3]

Wenn du den code kleiner gestalten willst kannst du auch scanf("%i %i ",&zah1,&zahl2); benutzen dann musst nichts doppelt machn :P