Foro de elhacker.net

Buenas, tengo este ejercicio :
escribir un programa que reciba dos números como parámetros y muestre por pantalla la syma y la resta de esos dos valores.

Y esto es lo que hice :

#include <stdio.h>
int operacion(int a, int b){
a=2;
b=3;
int s,r;
s=a+b;
r=a-b;
printf("\nLa suma es%d\n", s);
printf("\nLa resta es%d\n", r);
}
int main(void){
int x,y;
operacion(x,y);
}
 

Mi duda es simplemente si está bien hecho acorde a lo que pide el enunciado del ejercicio, es decir, si realmente lo hice bien y por parámetros tal como pide el problema o tiene otra forma de ser hecho.

UN saludo

Esta mal.

Lee esto sobre los argumentos del main.

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

Saludos.

Hola.

Lo correcto es inicializar los valores del parámetro antes de invocar a la función:

int x = 2,y = 3;
operacion(x,y);

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.

Le estais dando una respuesta erronea, la solución a su ejercicio es esta:

#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{
    if (argc != 3)
    {
        printf("Error, no introduciste los parametros.");
    }else{
        int n1 = atoi(argv[1]);
        int n2 = atoi(argv[2]);
 
        printf("La suma de los numeros es: %i\n", n1 + n2);
        printf("La resta de los numeros es: %i\n", n1 - n2);
    }
    getchar();
    return(0);
}

Salida del programa:


saludos.

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

Fair, ¿por qué es errónea? En todo caso no eres quién mandó el ejercicio, así que no conoces el criterio de lo que se está pidiendo o como se espera hacerlo.

Yo trato de dar la respuesta más amplia posible, no sólo la que encuadra con mi preferencia personal, o la forma como YO lo haría.

Se trata de pedir dos números, y devolver su suma y su resta. Es mi opinión se pueden pedir como argumentos de main(), o de cualquier función auxiliar dentro del programa.

En términos de eficiencia e implementación es diferente, pero estoy tratando de dar una respuesta EN LA MISMA LÍNEA QUE PLANTEÓ ORIGINALMENTE EL USUARIO, es decir, respetando su esfuerzo y criterio original, para que mi respuesta pueda ser realmente útil a él.

En vez de lucirme explicando la manera sensacional COMO YO LO HARÍA, en cuyo caso me estaría respondiendo a mí mismo y no a él.

Saludos, Yoel.

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.

Mafus, por eso dije le dije a Orb que tenía que modificar el cuerpo de operacion() de modo que pidiera los valores por teclado en lugar que establacerlos él mismo como valores fijos.

Y de acuerdo contigo en que si definió la función con retorno int, entonces es necesario poner el return.
Aunque la verdad lo veo innecesario, si fuera por mí la hubiera declarado void y no se requeriría retornar nada, lo cual por cierto estaría más de acuerdo con el propósito original del ejercicio que es devolver los datos por pantalla.

Saludos, Yoel.

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?

Yo con esta descripción no entiendo que el programa pida dos números, si no que se le deben pasar como argumento.

Para entender esto no me hace falta mandar yo el ejercicio.

Por otra parte si con ese enunciado lo que quiere el profesor es que el usuario ingrese dos numeros desde la consola que no es lo mismo que un programa reciba dos numeros como parametros estamos ante un profesor pésimo ó por lo menos su forma de expresarse.

saludos.

Orb, los argumentos de la función main, argc y argv, son para acceder a los parámetros que se le pasa al programa como información adicional.

argc es un entero que indica cuántos parámetros se han enviado. El nombre del programa es el primer parámetro y por siempre tendrá un mínimo valor de 1.

argv es un array de cadenas, cada una de ellas es un parámetro pasado al programa, así pues suponiendo la siguiente ejecución de un programa en línea de comandos:

miprogramasuma 1 2

argc valdrá 3, marcando que se han introducido tres parámetros por llamada.
argv[0] será la cadena miprogramasuma
argv[1] será la cadena 1
argv[2] será la cadena 2

Hay que recordar que todo son cadenas, incluso los números que se han pasado como argumentos son cadenas y no se pueden operar matemáticamente sin antes haberlos convertidos a su valor numérico.
También hay que tener en cuenta que, como son argumentos de main, solo son visibles a esta función.
Por último, argc y argv son nombres de variables que pueden ser llamadas de otra forma (p. ej. nargs y argum) pero que por convención tienen estos nombres.