Ile jest liczb parzystych, ile nieparzystych?

Question

Czołem!

Treść zadania brzmi: Program z użyciem sad pętli while, który sprawdza czy więcej jest cyfr parzystych, czy nieparzystych we wczytanej liczbie. Mój program działa, ale niepoprawnie :D. Wpisując np "4455" wyświetla, że więcej jest cyfr nieparzystych. Z góry dziękuję za pomoc :)  Mój kod to:

#include <iostream>
#include <cstdlib>
using namespace std;
int a, b;
int i=0;
int j=0;
int main()
{
    cout << "Podaj liczbe calkowita: ";
    cin >> a;
    b=a%10;


while (a>0)
{
        if (b%2==0)
        {

            i++;
            a/=10;
        }
        else
        {
            j++;
            a/=10;
        }

}
   if (i>j)
        cout << "Wiecej jest cyfr parzystych niz nieparzystych." << endl;
    else if (i==j)
        cout << "Cyfr nieparzystych i parzystych jest tyle samo." << endl;
 else 
       cout << "Cyfr nieparzystych jest wiecej niz parzystych." << endl;
    return 0;
}

 

Answer 1

Pierwszym najważniejszym twoim błędem było utworzenie zmiennej b, która pozostawała nie zmieniona, a to na jej podstawie ustalałeś parzystość poszczególnych liczb. Ja bym to napisał tak: 

#include <iostream>
#include <cstdlib>

using namespace std;

int main()
{
    int a;
    int i=0;
    int j=0;
    cout << "Podaj liczbe calkowita: ";
    cin >> a;

    while (a>0)
    {
            if((a%10)%2==0){
                i++;
            }else{
                j++;
            }
            a/=10;
    }

    if (i>j)
        cout << "Wiecej jest cyfr parzystych niz nieparzystych." << endl;
    else if (i==j)
        cout << "Cyfr nieparzystych i parzystych jest tyle samo." << endl;
    else
       cout << "Cyfr nieparzystych jest wiecej niz parzystych." << endl;
    return 0;
}

A twoim sposobem tak:

#include <iostream>
#include <cstdlib>

using namespace std;

int a, b;
int i=0;
int j=0;
int main()
{
    cout << "Podaj liczbe calkowita: ";
    cin >> a;

    while (a>0)
    {
        b=a%10;
        if (b%2==0)
        {

            i++;
            a/=10;
        }
        else
        {
            j++;
            a/=10;
        }

    }
    if (i>j)
        cout << "Wiecej jest cyfr parzystych niz nieparzystych." << endl;
    else if (i==j)
        cout << "Cyfr nieparzystych i parzystych jest tyle samo." << endl;
    else
        cout << "Cyfr nieparzystych jest wiecej niz parzystych." << endl;
    return 0;
}

 

Comments

Dzięki, najpierw zrobiłem bez deklaracji b, podobnie do Twojego sposobu ale mi nie wychodziło dlatego próbowałem to rozwiązać metodą prób i błędów.

 

Dziękuję i pozdrawiam :)

Answer 2

Zakładając, że nie implikujesz własnej arytmetyki do programu, to najprościej jest sprawdzać całą liczbę, czy jest podzielna na 2, jak tak to jej ostatnia cyfra jest parzysta w przeciwnym wypadku jej ostatnia cyfra jest nieparzysta. Do jakiejś zmiennej pomocniczej zapisujesz ilość liczb parzystych w ten sposób i liczb nieparzystych. Następnie po prostu porównaj te dwie wartości i wypisz odpowiedź.

Answer 3

Wartość do b jest przypisywana tylko raz w linii 11

btw po co ci zmienne globalne?