double numbers[ ] = { 1, 0.5 ,0.333333 ,0.25 ,0.2, 0.166667, 0.142857, 0.125,
0.111111, 0.1 } ;
std::vector<double> doublenumbers (numbers , numbers + 10) ;
std::cout << std::accumulate (doublenumbers.begin() , doublenumbers.end() , 0) ;
To daje 1, co jest ewidentnie błędne. Jakieś wyjaśnienia?C++ std :: accumulate nie daje oczekiwanej sumy
Należy napisać następujące:
std::cout <<
std::accumulate (doublenumbers.begin() , doublenumbers.end() , 0.0) ;
Ponieważ typ 0 jest int.
Po utworzeniu instancji std::accumulate z typem trzeciego argumentu to int, wówczas konwertuje prawą stronę sumy. np .:
result += *iter;
// int += double
Byłoby to wymusić konwersję double do int, zamiast tego, co pan myśli, które jest przeciwieństwem.
Co zabawne, nawet przy użyciu "0.0" nadal mogą występować pewne rozbieżności (w przypadku dużych sum) ze względu na przybliżoną reprezentację liczb zmiennoprzecinkowych. 'std :: vector
Po prostu miałem ten sam problem. Teraz zastanawiam się, jak należy to wiedzieć z dokumentacji (np. Http://pl.cppreference.com/w/cpp/algorithm/accumulate)? To poważne pytanie, a nie skarga. Naprawdę chciałbym bardziej przyzwyczaić się do C++. – user463035818
@MatthieuM. Nie jest to szczególny problem z 'std :: accumulate', prawda? – Isaac
std::accumulate (doublenumbers.begin() , doublenumbers.end() , .0) ;
lub
std::accumulate (doublenumbers.begin() , doublenumbers.end() , (double) 0) ;
Typ "akumulatora" zmienna jest typu ostatniego argumentu std::accumulate. Jako argument podano 0 - oznacza to, że akumulator będzie miał typ: int. "Kumulacja" jest dokonywana w akumulatorze int (to jest zaokrąglone do int po każdym pojedynczym podsumowaniu) i daje wynik int. W tym przypadku jest to, podobno, 1.
Nazywasz accumulate z 0 jako argumentem init, więc będzie się on kumulował przy użyciu liczb całkowitych. Zamiast tego użyj 0.0.
std::accumulate<double> (doublenumbers.begin(), doublenumbers.end(), 0); // also works
Podejrzewam, że niektóre parametry szablonu zostały pożarte. Załącz swój kod w tikki ('), aby tak się nie stało. –
Witamy w przepełnieniu stosu. Byłoby to lepszą odpowiedzią, gdybyś wspomniał o * dlaczego * to działa, a także gdybyś zwrócił uwagę na to, co dokładnie zmieniłeś. To różnica między dawaniem człowiekowi ryby a uczeniem go łowienia ryb. –
Ahh, jeśli to jest to, co zamierzałeś, to nie jestem pewien, czy to zadziała. 'std :: accumulate' pobiera w tym przypadku dwa parametry szablonu. Jeden do zdefiniowania typu iteratora, a drugi do zdefiniowania typu zwracanego. Jeśli podasz tylko jeden, dopasuje się do pierwszego parametru i nadal będzie odgadywany drugi. –
std::accumulate zacznie Podsumowując typ, który jest przekazywany jako 3rd argumentu, jeśli przejdą liczbę całkowitą, typ zwracany będzie int. I w tym przypadku niejawnie przekonwertowane na double.
w C++ 11 i C++ 14, jeśli chcesz, aby zapobiec zwężenie konwersji, można utworzyć obiekt przy bezpośredni wykazie inicjalizację:
double sum { std::accumulate(doublenumbers.begin(), doublenumbers.end(), 0) };
Kompilator będzie następnie daje ostrzeżenie mówiące, że próbujesz przekonwertować z int na podwójne, a także na której linii. Pozwoli to zaoszczędzić czas debugowania. I można łatwo naprawić, tak że staje się poprawne:
double sum { std::accumulate(doublenumbers.begin(), doublenumbers.end(), 0.0) };
Dla wyjścia pofałdowanego możemy użyć 0.0f. – user1436187
+1, jest to ważny haczyka, który jest ugryziony mnie kilka razy. –
również wektor nie jest konieczny; możesz używać wskaźników jako iteratorów: 'std :: accumulate (liczby, liczby + sizeof liczby/sizeof * liczby, 0.0);'. [W prawdziwym kodzie prawdopodobnie miałbyś stałą lub zmienną 'num_numbers' zamiast' sizeof numbers/sizeof * numbers'] –
Zapoznaj się z moją odpowiedzią, aby w łatwy sposób zapobiec temu bólowi głowy w przyszłości. –