Esta mal.

Lee esto sobre los argumentos del main.

http://decsai.ugr.es/~jfv/ed1/c/cdrom/cap6/cap64.htm

Saludos.

El problema que tienes es que en el cuerpo de la función operacion() no estás pidiendo los valores al usuario final, sino que los estás estableciendo tú mismo como a=2 y b=3.

Usando la función de entrada scanf() para pedir los valores, el programa quedaría como:

#include <stdio.h>
int operacion(int a, int b) {
 
	int s,r;
 
	s=a+b;
	r=a-b;
 
	printf("a = %d, b = %d\n", a, b);
 
	printf("\nLa suma es: %d\n", s);
	printf("\nLa resta es: %d\n", r);
 
	return 0;
}
 
int main(void){
 
	int x,y;
 
	printf("Intro primer valor: ");
	scanf("%d", &x);
	printf("Intro segundo valor: ");
	scanf("%d", &y);	
 
	operacion(x,y);
}
 

Otro detalle, no se si lo notaste es que faltó el valor de retorno de operacion() el cual se declaró como int. Por eso debes añadir la instrucción "return 0" al final del cuerpo de la función.

La salida en pantalla queda como

----------------------------------
Saludos, Yoel

---

Bueno 0xFer, yo diría antes de invocar a la función operacion(), o también sería válido dentro del cuerpo de ella.

Una tercera alternativa sería usando los argc y argv[] que sugirió fary.

Saludos, Yoel.

Si, la solución  que pone fary me parece correcta.

De todas formas tampoco tiene claro como funciona el paso de parámetros a una función.
Ya son unos cuántos que confunden parámetros de función con declaración de variables locales. Si su estudio viene del mismo sitio, en dónde sea que lo hagan, dan las clases muy mal.

Orb, cuándo tu función está declarada cómo

int operacion(int a, int b)

quiere decir que desde donde se la llame se le van a pasar dos enteros, el a y el b, para que dentro de la función se trabaje con ellos. Algo así como:

int operacion(int a, int b) {
    printf("La suma de a y b es %i.", a + b);
    ...
 

En cambio si quisieras que a y b formaran parte de la función pero no deben ser usadas para pasar parámetros en ella:

int operacion() {
    int a = 2;
    int b = 3;
    printf("La suma de a y b es %i.", a + b);
    ...
 

Hacer algo como lo que has puesto:

int operacion(int a, int b) {
    a = 2;
    b = 3;
    ...
 

Es una mala práctica ya que, sea lo que sea que has recibido por a y b, lo sustituyes por tus propios datos. Tu dirás: si así funciona, ¿porqué cambiar? La razón es que para llamar a la función debes pasarle tantos argumentos como así la hayas definido, además si la usa otra persona que no eres tú esperará que funcione de una forma cuándo se encontrará con resultados extraños porqué machacas lo que ella ha querido pasar.

Por otra parte tu función retorna un valor entero. Cuándo las funciones son declaradas con un valor de retorno éstas siempre deben terminar devolviendo un dato de ese tipo, aunqué después no lo uses, como ocurre casi siempre con printf (sí, printf devuelve un dato entero). Las únicas que están exentas de ello son las funciones con retorno tipo void.

Buenas!

Lo primero gracias por tomaros la molestia de ayudarme.

He estado haciendo todos los ejercicios que se han ido mandando la universidad, pero al llegar a los parámetros ha sido una confusión total.
YO considero a esto un parámetro no sé si estaré equivocado
void suma (int a, int b)
En los apuntes se ponen estos ejemplos, unas en las que es por ejemplo void suma(void) y demás.
También he estado mirando los usos de arc y argv por si pudieran servirme aquí u en otra ocasión pero no acabo de entender su uso.
Arriba declaré los datos porque no pedía el ejercicio que los datos fuesen introducidos por el usuario.

Gracias de nuevo, un saludo

P.D:El return no lo he usado en los 3 meses que llevo aprendiendo, lo que hago lo hago directamente en ubuntu, tendrá algo que ver?