Problem z if

[masher]

Mam taki problem:

Probuje napisac program gdzie trzeba podac liczbe calkowita i on ja przydziela do 4 kategorii: liczby ujemne, jednocyfrowe, dwucyfrowe i pozostale z pomoca "if".

To moj kod:

#include <iostream>
#include <conio.h>

using namespace std;
void main()
{
    int a;
    std::cout << "Podaj dowolna liczbe calkowita: "; 
    std::cin >> a;  
    
    if(a<0)
    {
        cout << "Liczna jest ujemna" << endl;
    }
    else
    {
        if(1<=a<=9)
        {
            cout << "Liczba jest jednocyfrowa" << endl;
        }
        else 
        {
            if(10<=a<=99)
            {
                cout << "Liczba jest dwucyfrowa" << endl;
            }
            else 
            {
                cout << "Liczba zalicza sie do pozostalych" << endl;
            }
        }
    }
getch();
}

Dobrze dopasowuje tylko liczby ujemne, a inne zalicza do jednocyfrowych.

Prosilbym o pomoc,

Z gory dziekuje.

[kryniak]

Spróbuj teraz:

#include <iostream>
#include <conio.h>

using namespace std;
void main()
{
   int a;
   std::cout << "Podaj dowolna liczbe calkowita: ";
   std::cin >> a;  

   if(a<0)
   {
       cout << "Liczna jest ujemna" << endl;
   }
   else if(1<=a && a<=9)
   {
       cout << "Liczba jest jednocyfrowa" << endl;
   }
   else if(10<=a && a<=99)
   {
       cout << "Liczba jest dwucyfrowa" << endl;
   }
   else
   {
       cout << "Liczba zalicza sie do pozostalych" << endl;
   }
getch();
}

[masher]

Dzieki kryniak, teraz dziala.

Mam tylko jezcze jedno pytanie:

Czy zawsze nalezy oddzielac takie znaki (1<=a<=9) &&?

[Vardi]

To zależy co chcesz osiągnąć. Sam powinieneś zdecydować, przyglądając się tamtemu wyrażeniu można wywnioskować dlaczego nie działa tak jak chciałeś.

[Sylwester]

Jeśli napisałeś:

using namespace std;

to nie musisz później pisac:

std::cout