funkcja przegladajaca ciag liczb c++

Question

Cześć Wam

Pracuje nad programem który wyszukuje w ciągu liczb całkowitych pierwszego elementu którego suma cyfr o wartościach parzystych jest mniejsza od iloczynu cyfr o wartościach nieparzystych. Przekazuję wypełniona przez użytkownika tablice do funkcji void funkcja_szukaj w której kolejno rozbijam liczbę na cyfry i sumuję bądź mnożę jej cyfry, jednak cyfry nie wliczają mi się do sumy, ani do iloczynu. Poniżej zamieszczam fragment kodu odpowiedzialnego za odnalezienie szukanej liczby. Liczę na wskazówki w jaki sposób mogę poprawić instrukcje if lub umieścić druga pętle for.

void funkcja_szukaj(int tablica[], int LiczbaElementow)
{

    int jednosci, dziesiatki, setki, tysiace, d_tysiecy;
    int a, b, c, d, e; // cyfry parzyste
    int na, nb, nc, nd, ne; // cyfry nieparzyste
    int suma, iloczyn;

    do
    for (int i=0; i<=LiczbaElementow; i++)
    {
        jednosci = tablica[i]%10;
        dziesiatki = tablica[i]/10%10;
        setki = tablica[i]/100%10;
        tysiace = tablica[i]/1000%10;
        d_tysiecy = tablica[i]/10000%10;
        // cout  <<d_tysiecy<<" "<<tysiace<<" "<<setki<<" "<<dziesiatki<<" "<<jednosci << endl;
        //linijka wyzej dodana tylko po to aby sprawdzic czy rozbija liczbe na cyfry

         if (jednosci%2==0)
                jednosci = a;
            else
                jednosci = na;

        if (dziesiatki%2==0)
                dziesiatki = b;
            else
                dziesiatki = nb;

        if (setki%2==0)
                setki = c;
            else
                setki = nc;

        if (tysiace%2==0)
                tysiace = d;
            else
                tysiace = nd;

        if (d_tysiecy%2==0)
                d_tysiecy = e;
            else
                d_tysiecy = ne;

        suma = a+b+c+d+e;
        iloczyn = na*nb*nc*nd*ne;
    }
    while (suma > iloczyn);  
    cout <<"pierwsza liczba spelniajaca kryteria to: "<< tablica[i] << endl;
}

 

Answer 1

Za jedności przypisujesz a, zamiast za a przypisac jednosc, dlatego suma wychodzi niepoprawna.

Comments

dzieki, spodziewalem sie, ze blad jest w szczegolach

Answer 2

Dostałeś już odpowiedź, ale w kodzie widać inny problem. Gdy programujesz staraj się unikać powtarzania kodu (zasada DRY). Zobacz robisz 17 zmiennych na przechowanie różnych wartości i kod wcale nie jest uniwersalny. Teraz obsługuje tylko liczby do 99999. Jak będziesz chciał poszerzyć zasięg o np 2 cyfry będziesz musiał ręcznie dodać kolejne zmienne. Dodać kolejne ich obliczenia i kolejne ify z takim samym warunkiem i %2 i to nie koniec! Zostaje jeszcze edycja sumy i iloczynu. 

Zobacz taki kod:

#include <iostream>
using namespace std;

void funkcja_szukaj(int tablica[], int LiczbaElementow)
{
	for (int i = 0; i < LiczbaElementow; ++i)
	{
		int suma_parzystych = 0; // zmienna jest lokalna wewnatrz petli for wiec przy przy kazdym cyklu wartosc zmiennej bedzie 0
		int iloczyn_nieparzystych = 1; // element neutralny dla mnozenia
		int liczba = tablica[i]; // tak, aby nie modyfikowac elementow tablicy

		do
		{
			int cyfra = liczba % 10;
			if (cyfra % 2 == 0)
			{
				suma_parzystych += cyfra;
			}
			else
			{
				iloczyn_nieparzystych *= cyfra;
			}
			liczba /= 10;
		} while (liczba > 0);

		//cout << tablica[i] << "\t" << suma_parzystych << " " << iloczyn_nieparzystych << "\n";
		if (suma_parzystych < iloczyn_nieparzystych)
		{
			cout << "Pierwsza liczba spelniajaca kryteria to: " << tablica[i] << endl;
			return; // przerwanie dzialania funkcji
		}
	}
		
	cout << "Nie znaleziono takiej liczby \n";
}

int main()
{
	const int size = 5;
	int tab[size] = { 2211, 8835, 422222235, 0, 5252 };
	funkcja_szukaj(tab, size);
}

Wystarczyły tylko 4 zmienne, a kodu jest zdecydowanie mniej. 

Comments

dzięki wielkie, na początku chciałem żeby mój kod w tej funkcji był podobny do Twojej poprawki ale nie wiedziałem jak to zoptymalizować, bardzo mi pomogłeś