Prośba o pomoc ze zwróceniem wyniku

Question

Hej,

chciałbym zwrócić vector wynik ale ciągle wyświetla mi 0,0. Wydaje mi się że sam algorytm funkcji jest okej. Gdzie mogłem popełnić błąd?

vector<int> breakingRecords(vector<int> scores) {
int pmax=0;
int pmin=0;
vector <int>wynik={pmax,pmin};

for(int i=0;i<scores.size();i++)
{
if (scores[i+1]>scores[i]) pmax++;
else if (scores[i+1]<scores[i]) pmin++;
}
return wynik; 
}

 

Comments

Stworzyłeś vector z 0,0 i nigdzie go nie modyfikujesz, więc nic dziwnego że wychodzi 0,0 ;)

---

Chciałbym żeby po zakonczonej pętli zwrócił inkrementowane wartosci pmax i pmin

---

I się inkrementują. Ale nie inkrementujesz liczb w vectorze ;) To je powinieneś modyfikować.

---

Podpowiesz? Chyba już dziś nic nie wymyślę. Nie umiem jakoś rozwiązać tego problemu.

---

#include <iostream>
#include <utility>
#include <vector>

using namespace std;
pair<int, int> breakingRecords(vector<int> scores) 
{
    int pmax=0;
    int pmin=0;
    auto wynik = make_pair(pmax, pmin);
    for(int i=0;i<scores.size();i++)
    {
        if (scores[i+1]>scores[i]) 
            wynik.first++;
        else if (scores[i+1]<scores[i]) 
            wynik.second++;
    }
    return wynik; 
}

int main()
{
    vector<int> temp{1,2,423,5,346,54,637,3567,5,2345,234,523,47,563,2,34,35,47,876,45,234,524,76,36234,56,2345,234,5234};
    auto [pmax,pmin] = breakingRecords(temp);
    cout<<pmax<<' '<<pmin;
}

I tak inkrementujesz dwie zmienne, więc nie wiem dlaczego wybrałeś vectora'a. 

Answer 1

Wypełnij vector już po wypełnieniu zmiennych poprawnymi wartościami? :)

vector<int> breakingRecords(vector<int> scores) {
  int pmax=0;
  int pmin=0;
  for(int i=0;i<scores.size();i++)
  {
    if (scores[i+1]>scores[i]) pmax++;
    else if (scores[i+1]<scores[i]) pmin++;
  }
  return {pmax, pmin}; 
}