[SOLUCIONADO] Problema con mi programa

[xignasix]

Hola, estoy empezando en el lenguaje c pero como ya me se un poco mas el lenguaje python he intentado hacer un programa para hacer de faciles a facilicimo pero cuando quiero poner datos desde pantalla me da errores
puedo poner valores en a y en b pero no puedo escoger una funcion desde pantalla,
he intentado mucho y seguire intentando pero si me ayudais os lo agradecedia
(el programa es para calcular densidad poblacion, taxas de natalidad, mortalidad, mortalidad infantil,etc).
el codigo es:
#include<stdio.h>

double DP(double poblacio_total, double superficie){
double DP;
DP=poblacio_total/superficie;
return DP;
}

double TN(double naixements,double poblacio_total){
double TN;
TN=(naixements/poblacio_total)*1000;
return TN;
}

double TM(double morts, double poblacio_total){
double TM;
TM=(morts/poblacio_total)*1000;
return TM;
}

double TMI(double morts_menors_1_any,double morts){
double TMI;
TMI=(morts_menors_1_any/morts)*1000;
return TMI;
}

double CV(double naixements,double morts){
double CV;
CV=naixements-morts;
return CV;
}

int main(void){
double a=0,b=0,resultado=0;//a=(nº)b=(nº).
printf("Valor1: ");
scanf("%g",&a);
printf("\nValor2: ");
scanf("%g",&b);
printf("DP, TN, TM, TMI o CV?");
scanf("%s",&resultado);//resultado=escrivir DP,TN,TM,TMI o CV y (a,b) dejar tal i como esta.//resultado=x(a,b); anterior
if (resultado=="DP"){
resultado=DP(a,b);
}
if (resultado=="TN"||"TM"){
resultado=TN(a,b);
}
if (resultado=="TMI"){
resultado=TMI(a,b);
}
if (resultado=="CV"){
resultado=CV(a,b);
}
if (resultado!="DP"||"TN"||"TM"||"TMI"||"CV"){
printf("ERROR");
}
printf("%g",resultado);
return 0;
}

y me marca error en los if.
Muchisimas gracias por la ayuda XD

[reethok]

Perdona, pero se nota que vienes de Python xP

Declaraste resultado como double, y con scanf lo usas como string, al igual que en los if's...

Te aconsejo, o hacer esto:

declarar la variable "opcion" de tipo int.

Código C:

printf("1 - DP, 2 - TN,  3 - TM,  4 -TMI, 5 - CV: ");
scanf(%d, &opcion);
 
if(opcion == 1) // DP
{ ...}
 

O usas un switch:

Código C:

switch(opcion)
{
   case 1:
      ...
       break;
   case 2:
      ...
      break;
}
 

Recuerda siempre usar tipos que tengan valor entero en las condiciones. Saludos.

[xignasix]

muchissimas gracias, ahora lo tengo de esta forma:

#include<stdio.h>

double DP(double poblacio_total, double superficie){
double DP;
DP=poblacio_total/superficie;
return DP;
}

double TN(double naixements,double poblacio_total){
double TN;
TN=(naixements/poblacio_total)*1000;
return TN;
}

double TM(double morts, double poblacio_total){
double TM;
TM=(morts/poblacio_total)*1000;
return TM;
}

double TMI(double morts_menors_1_any,double morts){
double TMI;
TMI=(morts_menors_1_any/morts)*1000;
return TMI;
}

double CV(double naixements,double morts){
double CV;
CV=naixements-morts;
return CV;
}

int main(void){
double a=0,b=0,resultado=0;//a=(nº)b=(nº).
int opcion;
printf("Valor1: ");
scanf("%g",&a);
printf("\nValor2: ");
scanf("%g",&b);
printf("\n1=DP, 2=TN, 3=TM, 4=TMI o 5=CV?");
scanf("%d",&opcion);//resultado=escrivir DP,TN,TM,TMI o CV y (a,b) dejar tal i como esta.//resultado=x(a,b); anterior
if (opcion == 1){
resultado=DP(a,b);
}

if (opcion == 2||3){
resultado=TN(a,b);
}

if (opcion == 4){
resultado=TMI(a,b);
}

if (opcion == 5){
resultado=CV(a,b);
}

while (opcion != 1||2||3||4||5){
printf("ERROR\n");
printf("\n1=DP, 2=TN, 3=TM, 4=TMI o 5=CV?");
scanf("%d",&opcion);
}

printf("%g",resultado);
return 0;
}

y no tengo errores en el codigo pero no me da ningun resultado y se encaia en el ciclo while
"!=" es = a: "si esto no pasa...". verdad?

[Fw190]

En las comparaciones, no puedes poner, por ejemplo,

if (opcion == 2||3)

(en la que entiendo que lo que quieres es comprobar si opcion es igual a 2 ó opcion es igual a 3). Debes poner una comparación para cada valor:

if ( (opcion == 2) || (opcion == 3) )

y lo mismo ocurre con la condición del bucle while.

Para este caso, deberías utilizar la sentencia switch:

Código C:

switch (opcion)
{
    case 1:
        ...
        break;
    case 2:
    case 3:
        ...
        break;
    case 4:
        ...
        break;
    case 5:
        ...
        break;
    default:
        ...
        break;
}
 

(case 2 y case 3 van juntos porque hacen lo mismo).

Saludos,

[xignasix]

muchisimas gracias, al final lo dejo sin el while y tampoco el switch, que por haber empezado hace tan poco y tener la edad que tengo (14 años) creo que ya esta bien
#include<stdio.h>

double DP(double poblacio_total, double superficie){
double DP;
DP=poblacio_total/superficie;
return DP;
}

double TN(double naixements,double poblacio_total){
double TN;
TN=(naixements/poblacio_total)*1000;
return TN;
}

double TM(double morts, double poblacio_total){
double TM;
TM=(morts/poblacio_total)*1000;
return TM;
}

double TMI(double morts_menors_1_any,double morts){
double TMI;
TMI=(morts_menors_1_any/morts)*1000;
return TMI;
}

double CV(double naixements,double morts){
double CV;
CV=naixements-morts;
return CV;
}

int main(void){
double a=0,b=0,resultado=0;//a=(nº)b=(nº).
int opcion;
printf("Valor1: ");
scanf("%g",&a);
printf("\nValor2: ");
scanf("%g",&b);
printf("\n1=DP, 2=TN, 3=TM, 4=TMI o 5=CV?");
scanf("%d",&opcion);//resultado=escrivir DP,TN,TM,TMI o CV y (a,b) dejar tal i como esta.//resultado=x(a,b); anterior
if (opcion==1){
resultado=DP(a,b);
}

if (opcion==2){
resultado=TN(a,b);
}
if (opcion==3){
resultado=TM(a,b);
}

if (opcion==4){
resultado=TMI(a,b);
}

if (opcion==5){
resultado=CV(a,b);
}


printf("%g\n",resultado);
system("pause");
return 0;
}

ya lo mejorare otro dia

[xignasix]

Sigue el problema pero ahora no se donde esta, todo esta bien, pongo los datos, pongo la opcion pero cuando me ha de dar el resultado, no me lo da correcto.
Por ejemplo:
1-
Valor1: 12
Valor2: 12
Opcion: 1
Resultado: 1
en este caso correcto
2-
V1:12
V2:4
O:1
R:1,01163
Mal, Con la opcion 1: V1/V2 y todos sabemos que en decimal 12/4=3
3-
V1:12.3 (las comas se escriben con puntos, no con comas)
V2:3
O:1
R:1,01586
Mal

El codigo sigue siendo igual con micro variaciones (un par de printf):
#include<stdio.h>

double DP(double poblacio_total, double superficie){
double DP;
DP=poblacio_total/superficie;
return DP;
}

double TN(double naixements,double poblacio_total){
double TN;
TN=(naixements/poblacio_total)*1000;
return TN;
}

double TM(double morts, double poblacio_total){
double TM;
TM=(morts/poblacio_total)*1000;
return TM;
}

double TMI(double morts_menors_1_any,double morts){
double TMI;
TMI=(morts_menors_1_any/morts)*1000;
return TMI;
}

double CV(double naixements,double morts){
double CV;
CV=naixements-morts;
return CV;
}

int main(void){
double a=0,b=0,resultado=0;//a=(nº)b=(nº).
int opcion;
printf("Valor1(les comes es fan amb punts. EX. 12.3): \n");
scanf("%g",&a);
printf("\nValor2(les comes es fan amb punts. EX. 12.3): \n");
scanf("%g",&b);
printf("\n1=DP, 2=TN, 3=TM, 4=TMI o 5=CV?\n");
printf("escoje la opcion con los numeros (1,2,3,4 o 5)\n");
scanf("%d",&opcion);//resultado=escrivir DP,TN,TM,TMI o CV y (a,b) dejar tal i como esta.//resultado=x(a,b); anterior
if (opcion==1){
resultado=DP(a,b);
}

if (opcion==2){
resultado=TN(a,b);
}
if (opcion==3){
resultado=TM(a,b);
}

if (opcion==4){
resultado=TMI(a,b);
}

if (opcion==5){
resultado=CV(a,b);
}
printf("%g\n",resultado);
printf("Programa hecho por I***** N******** M****\n");
system("pause");
return 0;
}