No veo el error en programa

[IVICP3]

En este programilla sencillo para mirar si tienes anemia o no.

Tengo 2 problemillas.

1-No hace el bucle d0/while cuando quiero continuar.
2-Cuanto pongo mas de 15 años, da igual el sexo, y pongo que el nivel es mas de 15, me entra en el primer if, NO SE PORQUE, si tengo edad==1.


Algo pasa en estas dos cosas, algo muy tonto, pero no las veo, parece lo tipico que estas 1 hora sin poder compilar y te falta una comilla o un parentesis.

Código C:

#include <stdio.h>
#include <stdlib.h>
 
 
 
int comprobar(int edad, char sexo, float nivel){
    
    if(edad==1 && nivel<11 || nivel>15){
        return printf("\nTiene anemia, deberia estar entre 11 y 15");
    }
 
    else if(edad>1 && edad<=15 && nivel<11.5 || nivel>15.5){
        return printf("\nTiene anemia, deberia estar entre 11.5 y 15.5");
    }
 
    else if(sexo=='m' && edad>15 && nivel<12 || nivel>16){
        return printf("\nTiene anemia, deberia estar entre 12 y 16");
    }
 
    else if(sexo=='h' && edad>15 && nivel<14 || nivel>18){
        return printf("\nTiene anemia, deberia estar entre 14 y 18");
    }
 
    return printf("Usted no tiene anemia");
    
}
 
 
 
 
 
 
 
 
 
 
 
void main(){
    int edad;
    char sexo, cont;
    float nivel;
 
    do{
    printf("Introduzca su edad: ");
    scanf("%d", &edad);
    fflush(stdin);
 
    printf("Introduzca su sexo, h o m: ");
    scanf("%c", &sexo);
    fflush(stdin);
 
    printf("Introduzca su nivel de hemoglobina: ");
    scanf("%f", &nivel);
 
    comprobar(edad, sexo, nivel);
 
    printf("\n");
    printf("\nQuieres continuar s/n?: ");
    scanf("%c", &cont);
    getchar();
    }while(cont=!'n');
 
    
}
 

[vangodp]

Estas diciendo con if(edad==1 && nivel<11 || nivel>15) exactamente:

Si edad es igual a 1 y el nivel es mayor que 11 o.... todo lo demás importa un carajo si el nivel es mayor que 15 XDDD

Con el || quiero decir o.... Ejemplo: O pasa eso o pasa el otro. Lo mismo: O pasa eso || pasa el otro.

Entonces... Si edad==1 y nivel<11 o(||) nivel>15. Si al poner mas de 15 le va importar un pimiento que el nivel sea menor que 11 ya que va pasar que se va cumplir lo que esta después del operador ||. A menos que ambas expresiones sean verdaderas antes del ||.

No se exactamente que es lo que quieres hacer pero me parece que el programa hace lo que estas diciendo que haga, lo que falla es la lógica.

Ademas el do/while lo haría así:

Código C:

do {
        printf ( "Introduzca su edad: " );
        scanf ( "%d%*c", &edad );
 
        printf ( "Introduzca su sexo, h o m: " );
        scanf ( "%c%*c", &sexo );
 
        printf ( "Introduzca su nivel de hemoglobina: " );
        scanf ( "%f%*c", &nivel );
        
        comprobar ( edad, sexo, nivel );
 
        printf ( "\n\nQuieres continuar s/n?: " );
        scanf ( "%c%*c", &cont );
        
        getchar();
    } while ( cont != 'n' );
 

[lareto]

Hola; cuando una combinación de condiciones no se comparta como uno cree, lo más probable es que se deba a un error al interpretar el orden de precedencia. En este caso debes tener en cuenta que && tiene mayor precedencia que ||. Puedes consultar en:
http://en.cppreference.com/w/cpp/lan...tor_precedence

Por otro lado, poniendo nombres como "anemia" y "hemoglobina", yo aclararía que los datos son figurados, que no guardan ninguna relación con la "anemia" real. Nunca se sabe quién puede leer esto y creer que se trata de información real.