Teilen Code in Funktionen in C

Question

Ich komme zu C nach langer Zeit nicht zu verwenden. Ich habe gerade geübt und bin auf dieses Problem gestoßen. Ich versuche zu:

1. nehmen 2 Variablen von scanf()
2. die beiden Eingänge multiplizieren
3. dann gibt das Problem

Ich habe versucht, die verschiedenen Teile in Funktionen aufgeteilt, aber das Programm gibt verrückte hohe Zahlen (denke, dass sie die Speicheradressen sind). Es liegt wahrscheinlich an meinem Mangel an Missverständnissen.

#include <stdio.h> 

int input(); 
int mult (int x, int y); 

int main() 
{ 
int x; 
int y; 

x, y = input(); 

printf("In Main: x, y: %d, %d\n", (x, y)); 
z = mult(x,y); 
printf("The product of your two numbers is %d\n", z); 

getchar(); 
getchar(); 
} 


int input() 
{ 
int i_x; 
int i_y;  

printf("Please input two numbers to be multiplied: "); 
scanf("%d", &i_x); 
scanf("%d", &i_y); 
printf("In Input: x, y: %d, %d\n", i_x, i_y); 
return i_x, i_y; 
} 


int mult (int x, int y) 
{ 
int a; 
int b; 
int c; 

a = x; 
b = y; 

printf("In Multi: x, y: %d, %d\n", a, b); 

c = a*b; 

return c; 
} 

Answer 1

x, y = input(); nicht tun, was Sie erwarten. Sie können nicht zwei Werte von einer Funktion zurückgeben. Sie sollten über comma operator lesen.

Ich würde eher vorschlagen, die Adresse x und y an die Funktion input übergeben. Ändern Prototyp erste

void input(int *, int *);  

dann nennen es als

input(&x, &y); 

und ändern Sie die Definition

void input(int *i_x, int *i_y) 
{ 
    printf("Please input two numbers to be multiplied: "); 
    scanf("%d", i_x); 
    scanf("%d", i_y); 
    printf("In Input: x, y: %d, %d\n", *i_x, *i_y); 
} 

Answer 2

main sollte einen Wert zurückgeben, da es als int definiert ist.
return i_x, i_y; wird nur i_y durch die Definition der [comma operator] zurückgegeben. Mit derselben Definition wird x, y = input(); nur y zugewiesen.

Answer 3

Wie von @hacks gezeigt, funktioniert die Zuweisung von x & y durch Aufruf von input() nicht richtig. Die Variable x wird nur einen unitären Junk-Wert haben und die Ausgabe entsprechend beeinflussen. Um zu verifizieren, wenn Sie x auf 1 initialisieren, sollte die Ausgabe dann wie die zweite Eingabe (i_y) sein.

Answer 4

Wie @hackcks sagte, Sie können nicht zwei Werte von einer Funktion zurückgeben. Definieren Sie stattdessen die Argumente für input() als Zeigervariablen und machen Sie input() und geben Sie void zurück.

so ur endgültige Code kann wie folgt aussehen:

#include <stdio.h> 

void input(int *i_x, int *i_y); 
int mult (int x, int y); 

int main() 
{ 
int x; 
int y; 
int z; 

input (&x, &y); 

printf("In Main: x, y: %d, %d\n", x, y); 
z = mult(x,y); 
printf("The product of your two numbers is %d\n", z); 

getchar(); 
getchar(); 
} 


void input(int *i_x, int *i_y) 
{  

    printf("Please input two numbers to be multiplied: "); 
    scanf("%d", i_x); 
    scanf("%d", i_y); 
    printf("In Input: x, y: %d, %d\n", *i_x, *i_y); 

} 

int mult (int x, int y) 
{ 
int a; 
int b; 
int c; 

a = x; 
b = y; 

printf("In Multi: x, y: %d, %d\n", a, b); 

c = a*b; 

return c; 
}