Programmier Problem mit C

[BoomerBoss]

Hallo erstmal
Ich bin nämlich neu hier
KOmmen wir sofort zur Sache

Ich bin ein totaler Noob in programmieren und lerne jetz C.
Nun wollte ich einen DMark in Uero Umrechner programmieren.
Das Problem is nur er funnktioniert nich richtig.
Ich hab den Rechner mit Turbo C gemacht
Quellcode:
#include <stdio.h>
#include <conio.h>

int main()
{
int DMark; /*Deutsche Mark*/
long double Kurs = 1.95583; /*Euro Kurs*/


printf ("DMark zu Euro Umrechner \n");
printf ("Deutsche Mark eingeben... ");

scanf ("%d" ,DMark); /*Eingabe und in DMark Integer ablegen*/
printf ("%d Deutsche Mark sind %d Euro", DMark, DMark / Kurs); /*Rechnen und anzeigen*/

getch();
return 0;
}

So wennn ich jetz den Rechner anmache fragt er nach der DMark Zahl
Wenn ich die eingebe komm eine -Euro Zahl so -27000

Kann mir daa jemand helfen???

[roro]

scanf ("%d" ,DMark); /*Eingabe und in DMark Integer ablegen*/

ersetzen durch

scanf ("%d" ,&DMark); /*Eingabe und in DMark Integer ablegen*/

Andere Frage, wieso verwendest du int für DMark?

[BoomerBoss]

ähhhm weiss ich nich
ainfach so spontan
und warum komm dieses Zeichen vor DMArk????
jetz zeigt er die richtige DMark zahl an aber es kom ein völlig falsches ergebnis immer noch raus

[roro]

& steht für Pointer. Scanf erwartet einen Pointer und keinen Wert.

[BoomerBoss]

mein prob is immer noch net gelöst
als ergebnis kommt andauernd -23688 oder so raus

[kirschi]

Hi!
Bittesehr, so funktioniert es korrekt:

1) das &DMark beim scanf hat roro schon ausgebessert

scanf ("%f" ,&DMark); /*Eingabe und in DMark Integer ablegen*/

2) nimm für die Eingabe und den Kurs einen float und keinen double...
float hat gewöhnlich 4 Byte, double 8 Byte...4 reichen völlig aus

float DMark; /*Deutsche Mark*/
float Kurs = 1.95583; /*Euro Kurs*/

3) Du hast für den Kurs einen double gewählt, ihn aber als int angesprochen (mit dem %d), musst %f nehmen...

printf ("%0.2f Deutsche Mark sind %0.2f Euro", DMark, DMark / Kurs);

Ein paar Referenzen:
Allgemein:
http://www.cppreference.com/
IO-spezifisch
http://www.cppreference.com/stdio/printf.html
Für den Anfang empfehle ich 2 einfache Bücher für C und C++:
http://courses.iicm.edu/programmieren0/#d12e112
http://courses.iicm.edu/programmierpraktikum/skriptum/

LG, Kirschi

[ulrich]

& steht für Pointer.

nein, das & ist in dem fall der "adresse-von"-operator.
für pointer steht *, um es salopp auszudrücken.

Code: Alles auswählen

int i = 0;              //ganze zahl i auf 0 setzen
int* ip = &i;           //dem "zeiger auf int" ip die _adresse_von_ i zuweisen


[ulrich]

2) nimm für die Eingabe und den Kurs einen float und keinen double...

<haare sträub>
nie, nie, nie zum rechnen float verwenden! das ist ein absolutes no-no (wegen der bei float zu geringen anzahl gültiger stellen).
zum rechnen immer double: das sollte man sich zum prinzip machen, auch wenn es im vorliegenden beispiel möglicherweise nicht notwendig ist.

float hat gewöhnlich 4 Byte, double 8 Byte...4 reichen völlig aus

grmml.
stand der technik ist folgende empfehlung: zuerst schreibe man ein korrektes programm, dann optimiere man (falls überhaupt notwendig - ich rede hier von einem PC system, und nicht von beschränkten dingen, die auch in c programmiert werden (z.B. embedded systems und so...)).

[Avelyn]

Code: Alles auswählen

float DMark; /*Deutsche Mark*/
float Kurs = 1.95583; /*Euro Kurs*/

Außerdem ist es üblich *alle* Variablen zu initialisieren. Also

Code: Alles auswählen

float DMark = 0.0f;

[max_payne]

üblich schon, aber nicht notwendig, sofern du sie dann nicht abfragst, ohne ihnen vorher einen wert(zb aus einer rechnung) zuzuweisen

[ulrich]

üblich schon, aber nicht notwendig, sofern du sie dann nicht abfragst, ohne ihnen vorher einen wert(zb aus einer rechnung) zuzuweisen

aber sehr nützlich beim debuggen...
außerdem ist stand der technik das sogenannte RAII paradigma:
"resource aquisition is initialisation"
d.h. grob gesagt, nichts wird erzeugt, ohne sofort benutzt zu werden (und sollte auch nicht länger existieren als notwendig).

[Max O]

ich hätte da nen andren vorschlag:
einfach einen string bzw in c ein char array einlesen. dann mit den funktionen atoi(char*) oder mit atof(char *) den string umwandeln in das gewünschte format.
atoi -> array to integer
atof -> array to float

[max_payne]

und was tust du, wenn sich einer vertippt und n buchstaben, etc reinhaut?

[ulrich]

fehlerabfrage und erneute aufforderung zur eingabe, was sonst?

[Avelyn]

Also ich würde nicht die atoi(), atof() funktionen benutzen, da sie ehrlich gesagt schon etwas zu alt sind, und keine Möglichkeit lassen Fehler zu überprüfen. In C am besten mit scanf() arbeiten:

Code: Alles auswählen

const char* str_int = "777";
const char* str_float = "333.3";
int i;
float f;

if(EOF == sscanf(str_int, "%d", &i))
{
  //error
}

if(EOF == sscanf(str_float, "%f", &f))
{
  //error
}

Den atoi() (und die Familie) haben einen markanten Nachteil: Sie geben alle 0 zurück wenn ein Fehler passiert ist, oder wenn sie "0" konvertieren.

P.s.: Wahlweiße geht auch strtod, da es den String position zurück gibt, ab wo die Konvertierung zur Zahl fehlgeschlagen ist.

Mfg Avelyn