Warunek "If" i zmienna z przedziału liczb...?

Question

Piszę kalkulator BMI i zastanawiam się czy jest jakiś krótszy sposób zapisu kodu gdzie instrukcją IF sprawdzamy czy zmienna mieście sięw jakimś przedziale liczb. Narazie wygląda to tak:

      else if((wynik>=17.0)&&(wynik<18.49))cout << "Niedowaga!"<<endl;

 

Poproszę o radę i pozdrawiam.

Answer 1

Możesz napisać sobie funkcję tego typu:

bool check(float checked, float lowerLimit, float upperLimit) {
     if(checked >= lowerLimit && checked < upperLimit) return true;
return false;
}

Następnie wywołujesz w ten sposób

if(check(wynik, 17.0f, 18.49f)) cout<<"Niedowaga!"<<endl;

Aha i nie nadużywaj nawiasów tam gdzie są niepotrzebne. Wewnętrzne nawiasy w twoim ifie są zbędne.

Comments

Moim zdaniem lepiej pisać za dużo nawiasów niż za mało. Co byś powiedział na przykład na taki przypadek:

#define add(a, b) a + b
.
.
.
int a = add(7, 5) * 4;

Jeśli nawiasy można wstawić, to lepiej pisać ich za dużo, niż potem mieć trudne do znalezienia błędy w kodzie.

Answer 2

Chyba niestety nie istnieje żaden sposób, ale jak coś wymyślę, to powiem.

Można by ewentualnie stworzyć tablicę tych wartości i tylko jednego if() w pętli.

Answer 3

Możesz pozbyć się ewentualnie nawiasów, w którzych masz np. (wynik>=17.0), bo akurat one są zbędnę, ale innego krótszego zapisu tutaj nie ma, ten i tak jest dość krótki. Gdy w przyszłości będziesz programować bardziej złożone rzeczy to przekonasz się, że taki sposób jest jednym z najkrótszych.

Answer 4

Jest ok. Dobrze, że się tym interesujesz, bo warunki należy pisać jak najprościej. If obsługuje przedziały liczb własnie w ten sposób. Następnym razem kod 

koloruj!

 

Comments

Bardziej pytanie wynika z mojego lenistwa niż ciekawości, ale na jedno wychodzi:)

---

                if (wynikbmi <= 16)
                    
                    $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "wyglodzenie");
                    
                    
                    else if (wynikbmi < 17)
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "Wychudzenie (spowodowane czesto przez ciezka chorobe lub anoreksje)");
                    
                    else if (wynikbmi  < 18.5)
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "niedowaga");
                    
                    else if (wynikbmi < 25)
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "wartosc prawidlowa");
                    
                    else if (wynikbmi < 30)
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "nadwaga");
                    else if (wynikbmi < 35)
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "1 stopien otylosci");
                
                    else if (wynikbmi < 40)
                    
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "2 stopien otylosci (otylosc kliniczna)");
                    
                    else if (wynikbmi >= 40)
                    
                    
                        $("#bmi").html("Twoje BMI:" + (" ") + wynikbmi + "3 stopien otylosci(otylosc skrajna)");
                    

U mnie ja to tak zrobiłem