C error en una condicional

[erickggp]

Mi pregunta es porque esta condicional no funciona, se supone que recoje la cadena de caracteres y compara pero es como si nunca fueran iguales

Código PHP:

  #include <stdio.h> 
#include <conio.h> 
main(){char var; 
printf("Seleccione su sexo[Hm/Mj]:"); 
scanf("%s", &var); 
if(var=="Hm") 
           { 
           printf("Es hombre"); 
           } 
else if(var=="Mj") 
           { 
           printf("Es mujer"); 
           } 
getch(); 
}  
   

[Luciano9]

Mirá esto:
strcmp: -- no puedo poner links -- ve a google y busca strcmp conclase y te saldra la explicación.

No puedes hacer lo siguiente:

if (variable == cadenadecaracteres)

tienes que usar alguna función como strcmp

saludos y espero que te sirva

[JuanLVBR]

Cometes el error ya inicializando la variable.

Declaras un char en el que tu intención es guardar dos chars, deberías declarar un vector de dos chars, o bien una cadena de carácteres, o bien dos chars.

Y luego como bien dice Luciano9, los vectores (en este caso cadena de carácteres) no se pueden comparar con los operadores >, < o =, igual que no se pueden sumar ni concatenar así como así, hay que hacerlo índice a índice o bien utilizando las funciones declaradas en la librería string.h (que al fin y al cabo hacen lo mismo).

O bien:

Código:

#include <stdio.h>
#include <conio.h>

int main(){
	char var[2];
	printf("Seleccione su sexo[Hm/Mj]: ");
	scanf("%c", &var[0]);
	scanf("%c", &var[1]);
	if(var[0]=='H' && var[1]=='m')
		printf("Es hombre");
	else if(var[0]=='M' && var[1]=='j')
		printf("Es mujer");
	getch();
	return 0;
}

O bien:

Código:

#include <stdio.h>
#include <conio.h>
#include <string.h>

int main(){
	char var[2];
	printf("Seleccione su sexo[Hm/Mj]: ");
	scanf("%c", &var[0]);
	scanf("%c", &var[1]);
	if(strcmp("Hm", var)==0)
		printf("Es hombre");
	else if(strcmp("Mj", var)==0)
		printf("Es mujer");
	getch();
	return 0;
}

Te he cambiado también la entrada de datos, como son sólo dos carácteres no merce la pena un for para almacenar, y no lo hago en un solo scan o gets porque habría que controlar que al usuario le dé por meter más de 2 carácteres, de todos modos si quieres hacerlo así, tendrías que pasarle el puntero al primer elemento del vector al scanf, sería simplemente:

Código:

scanf("%s", var);

[erickggp]

Me acomodo asi:

Código PHP:

  #include <stdio.h> 
#include <string.h> 
 
int main() 
{  printf("Escribe [H/M]:"); 
   char s1; 
   int i, o; 
   scanf("%s", &s1); 
   i = strcmp( "H", &s1 ); 
   o = strcmp( "M", &s1 ); 
   if( i == 0 )  printf( "Es hombre" ); 
   else if( o == 0 )  printf( "Mujer" ); 
   else  printf( "NADA" ); 
   getch(); 
   return 0; 
}  
   

GRACIAS CUATES.

[JuanLVBR]

Te dará algún warning pero me parece una buena elección sin duda el olvidarse de que sean dos carácteres (Hm, Mj), sin complicarse la vida, como debe ser.

Ahora como es un sólo carácter no hace falta usar strcmp, te ahorras incluir la librería, y alguna que otra variable y comparación, yo lo haría así:

Código c:

#include<stdio.h>
 
int main(){
    char c;
    printf("Escribe [H/M]: "); 
    scanf("%c", &c);
    if(c=='H') printf("Es hombre");
    else if(c=='M') printf("Mujer");
        else printf("Nada");
    return 0;
}
 

Pero insisto, tu solución es totalmente válida y funciona estupendamente bien. Saludos.