Was ist falsch?

[Simon N. (ardi)]

Hab nach langem mal wieder C programmiert:

‪#‎include‬ <stdio.h>

int main(){
int ro;
int erste;
int zweite;
int ergebnis;
printf("Welche Rechenoperation moechten sie durchfuehren?\n");
printf("1 fuer + 2 fuer /:");
scanf("%c", ro);

if(ro==1){

printf("1. Zahl:");
scanf("%c", erste);
printf("2. Zahl:");
scanf("%c", zweite);
printf("Ergebnis: ",ergebnis);
ergebnis=erste+zweite;

}else {
printf("Hallo");
}
return 0;
}

Aber immer nach der ersten Eingabe stürzt das Programm ab.
Was ist da falsch?
Danke und Gruß

[Simon N. (ardi)]

habs: kaufmannsund vergessern und anstatt %c %d

[Amateur (Gast)]

Versuchs mal mit passenden Parametern. scanf("%c", ro);
Mit so was wie: "%d" und &ro. Muss aber nicht sein;-)

[Dr. Sommer (Gast)]

Falsch ist: Einrückung fehlt.

#‎include‬ <stdio.h>

int main () {
  int ro;
  int erste;
  int zweite;
  int ergebnis;
  printf ("Welche Rechenoperation moechten sie durchfuehren?\n");
  printf ("1 fuer + 2 fuer /:");
  scanf ("%c", &ro);

  if (ro == 1) {

    printf ("1. Zahl:");
    scanf ("%c", &erste);
    printf ("2. Zahl:");
    scanf ("%c", &zweite);
    printf ("Ergebnis: ", ergebnis);
    ergebnis = erste + zweite;

  } else {
    printf ("Hallo");
  }
  return 0;
}

[Amateur (Gast)]

@Dr. Sommer

Spielverderber!

So kann man das ja sogar vernünftig lesen.

Nee, nee.

[Simon N. (ardi)]

Und jetzt: Geht auch nicht beim "modifiezieren":

#include <stdio.h>

int main(){
int ro;
int erste;
int zweite;
int ergebnis=erste + zweite;
printf("Welche Rechenoperation moechten sie durchfuehren?\n");
printf("1 fuer + 2 fuer /:");
scanf("%d", ro);

if(ro==1){
    printf("1. Zahl: ");
    scanf("%c", &erste);
    printf("2. Zahl: ");
    scanf("%c", &zweite);
    printf("Ergebnis: ",ergebnis);

}else {
    printf("Hallo");
}
return 0;
}

[Amateur (Gast)]

Sticht Dir das nicht ins Auge?
scanf("%d", ro);        hole ASCII-Zahl
scanf("%c", &erste);    hole ASCII-Zahl
scanf("%c", &zweite);   hole ASCII-Zahl

Wie bereits geschrieben: Mein Favorit ist scanf ( "%d",&ro );
Musst Du aber nicht machen.

[Tom K. (ez81)]

1.c: In function ‘main’:
1.c:11: warning: format ‘%c’ expects type ‘char *’, but argument 2 has type ‘int’
1.c:16: warning: format ‘%c’ expects type ‘char *’, but argument 2 has type ‘int’
1.c:18: warning: format ‘%c’ expects type ‘char *’, but argument 2 has type ‘int’
1.c:19: warning: too many arguments for format
1.c:11: warning: ‘ro’ is used uninitialized in this function
1.c:16: warning: ‘erste’ may be used uninitialized in this function
1.c:18: warning: ‘zweite’ may be used uninitialized in this function
1.c:19: warning: ‘ergebnis’ may be used uninitialized in this function

Wer Compilerwarnungen nicht auf die höchste Stufe stellt, hat entweder 
einen sehr guten Grund dafür oder eben kein Interesse an 
funktionierendem Code.

[Karl H. (kbuchegg)]

Simon Name schrieb:
> Und jetzt: Geht auch nicht beim "modifiezieren":
>

> #include <stdio.h>
> 
> int main(){
> int ro;
> int erste;
> int zweite;
> int ergebnis=erste + zweite;
>

Würdest du mir mal erklären, wie du an dieser Stelle ein Ergebnis 
berechnen kannst, noch bevor die Werte überhaupt eingelesen wurden?
Das ist doch nicht Excel, das erst mal eine halbe Stunde darüber 
sinniert in welcher Reihenfolge welche Berechnungen gemacht werden 
müssen, damit dann hoffentlich das raus kommt, was du dir vorstellst!

>

> printf("Welche Rechenoperation moechten sie durchfuehren?\n");
> printf("1 fuer + 2 fuer /:");
> scanf("%d", ro);
>

Wo ist das & ?

>

> if(ro==1){
>     printf("1. Zahl: ");
>     scanf("%c", &erste);
>

%c ?
Echt?
erste ist ein char?

>

>     printf("2. Zahl: ");
>     scanf("%c", &zweite);
>

Selbiges nochmal

>

>     printf("Ergebnis: ",ergebnis);
>

Wo ist das Formatierzeichen für den auszugebenden Integer?
Schon angesprochen, was es mit ergebnis auf sich hat


Alles in allem: ein bischen mehr Sorgfalt!