Problem z zliczeniem elementów tablicy, których suma cyfr osiąga podaną wartość

Question

Witam,

Mam takie zadanie: 

Losuje tablicę 20 elementową z liczbami z zakresu 50-99  

Zlicza elementy tablicy, których suma cyfr osiąga podaną wartość

I mam taki kod:

 

#include <iostream>
#include <cstdlib>
#include <ctime>

using namespace std;
int sum_cyfr(int tab[],int liczba);

int main()
{
 int tab[20],i,liczba;
    srand(time(NULL));
    for(i=0;i<20;i++)
    {
        tab[i]=rand()%50+50;
        cout<<tab[i]<<"  ";
    }
    cout<<endl;
    cout<<"Podaj Liczbe: ";
    cin>>liczba;
    cout<<"Podana sume cyfr zlicza tyle elemntow tablicy: "<<sum_cyfr(tab,liczba)<<endl;
    return 0;
}
int sum_cyfr(int tab[],int liczba)
{
    int i,wynik=0;
    for(i=0;i<20;i++)
    do
{
    wynik=wynik+liczba%10;
    liczba=liczba/10;
}
    while(liczba!=0);
    return wynik;
}

 

Problem jest taki, że nie zlicza poprawnie elementów tablicy, których suma cyfr osiąga podaną wartość

Przykładowo:

Podaj liczbę 10, czyli elementy tablicy,ktory się składa na to to liczba np. 82, 91 itd.

I nie wyświetla tej liczby poprawnie.

Co muszę zmienić w kodzie?

Answer 1

1. W funkcji sum_cyfr nie "rozbijasz" na cyfry wartości z tablicy tab[], tylko liczbę podaną przez użytkownika liczba
2. Po wyjściu z pętli do {...} while() - nie "zerujesz" zmiennej wynik

int sum_cyfr(int tab[],int liczba)
{
    if (liczba == 0) return 0;
    if (liczba < 0) liczba *= (-1); // Pozbywamy sie minusa -9 = 9  ;-)
    
    int wynik, ile=0, temp=0;
    
    for (int i=0; i<20; i++)
    {
        temp = tab[i];
        wynik=0;
        
        do
        {
            wynik+=temp%10;
            temp/=10;
        }
        while (temp !=0 );
        
        if (liczba == wynik) ile++;
    }
        
    return ile;
}

 

Comments

Dzięki za pomoc!

---

#include <iostream>
#include <ctime>
#include <vector>
 
using namespace std;

int sum_cyfr(int tab[], int liczba, vector<int> &test);
 
int main()
{
    int tab[20], i, liczba;
    vector<int> test;
    
    srand(time(NULL));
    for (i=0; i<20; i++)
        tab[i] = rand()%50+50;

    cout << endl
         << "Podaj Liczbe: ";
     cin >> liczba;
    cout << "Podana sume cyfr zlicza tyle elemntow tablicy: " 
         << sum_cyfr(tab, liczba, test) << endl << endl;
    
    for (int element: tab)
        cout << element << "  ";
            
    if (test.size() > 0) {
        cout << endl;

        for (i=0; i<20; i++) {
            bool arrow = false;

            for (int x=0; x<test.size(); x++)
                if (tab[i] == test[x]) {
                    arrow = true;
                    test.erase(test.begin() + x);
                    break;
                }
                else
                    arrow = false;
                    
            cout << ((arrow) ? "^^  " : "    ");
        }
    }

    return 0;
}

int sum_cyfr(int tab[],int liczba, vector<int> &test)
{
    if (liczba == 0) return 0;
    if (liczba < 0) liczba*=(-1); // Pozbywamy sie minusa -9 = 9  ;-)
    
    int wynik, ile=0, temp=0;
    
    for (int i=0; i<20; i++) {
        temp = tab[i];
        wynik=0;
        
        do {
            wynik+=temp%10;
            temp/=10;
        } while (temp !=0 );
        
        if (liczba == wynik) {
            ile++;
            test.push_back(tab[i]);
        }
    }
        
    return ile; // = test.size()
}

[ c++ foreach ]  [ c++ vector operations ]  [ vector::erase() ]