<![CDATA[findet jemand den Fehler?]]>#include <algorithm> #include <iomanip> #include <iostream> #include <istream> #include <iterator> #include <numeric> #include <ostream> #include <vector> int main() { std::vector<int> v; std::copy(std::istream_iterator<int>(std::cin), std::istream_iterator<int>(), std::back_inserter(v)); int avg = std::accumulate(v.begin(), v.end(), 0) / v.size(); std::cout << avg << std::endl; }

Das Programm soll den Mittelwert der eingegebenen Werte liefern. In den meisten Faellen funktioniert es, aber manchmal wird ein falsches Ergebnis geliefert. Weis jemand wieso?

]]>https://www.c-plusplus.net/forum/topic/209473/findet-jemand-den-fehlerRSS for NodeMon, 12 Oct 2026 20:17:46 GMTMon, 31 Mar 2008 09:54:14 GMT60<![CDATA[Reply to findet jemand den Fehler? on Mon, 31 Mar 2008 09:54:14 GMT]]>#include <algorithm> #include <iomanip> #include <iostream> #include <istream> #include <iterator> #include <numeric> #include <ostream> #include <vector> int main() { std::vector<int> v; std::copy(std::istream_iterator<int>(std::cin), std::istream_iterator<int>(), std::back_inserter(v)); int avg = std::accumulate(v.begin(), v.end(), 0) / v.size(); std::cout << avg << std::endl; }

Das Programm soll den Mittelwert der eingegebenen Werte liefern. In den meisten Faellen funktioniert es, aber manchmal wird ein falsches Ergebnis geliefert. Weis jemand wieso?

]]>https://www.c-plusplus.net/forum/post/1483626https://www.c-plusplus.net/forum/post/1483626Mon, 31 Mar 2008 09:54:14 GMT<![CDATA[Reply to findet jemand den Fehler? on Mon, 31 Mar 2008 10:04:02 GMT]]>Vermutlich ein Rundungsfehler durch "int avg" ?

Bei Division würde ich jedenfalls float oder double nehmen...

]]>https://www.c-plusplus.net/forum/post/1483631https://www.c-plusplus.net/forum/post/1483631Mon, 31 Mar 2008 10:04:02 GMT<![CDATA[Reply to findet jemand den Fehler? on Mon, 31 Mar 2008 10:09:39 GMT]]>Du dividierst nur integrale Zahlenwerte, d.h. der Nachkommaanteil wird ignoriert. Caste mal v.size() in einen double und mach aus avg auch einen double.

]]>https://www.c-plusplus.net/forum/post/1483636https://www.c-plusplus.net/forum/post/1483636Mon, 31 Mar 2008 10:09:39 GMT<![CDATA[Reply to findet jemand den Fehler? on Wed, 02 Apr 2008 04:58:28 GMT]]>Tyrdal schrieb:

Du dividierst nur integrale Zahlenwerte, d.h. der Nachkommaanteil wird ignoriert. Caste mal v.size() in einen double und mach aus avg auch einen double.

Vielleicht haette ich sagen sollen "Mittelwert ohne Nachkomma".

Nein, das ist nicht der Fehler. Tipp: Der Fehler tritt auf, wenn std::accumulate() eine negative Summe zurueckliefert. Dann koennte es passieren, dass der Mittelwert ganz und gar falsch ist.

]]>https://www.c-plusplus.net/forum/post/1484921https://www.c-plusplus.net/forum/post/1484921Wed, 02 Apr 2008 04:58:28 GMT<![CDATA[Reply to findet jemand den Fehler? on Wed, 02 Apr 2008 07:04:42 GMT]]>int avg = std::accumulate(v.begin(), v.end(), 0) / static_cast<int>(v.size()); ]]>https://www.c-plusplus.net/forum/post/1484961https://www.c-plusplus.net/forum/post/1484961Wed, 02 Apr 2008 07:04:42 GMT<![CDATA[Reply to findet jemand den Fehler? on Wed, 02 Apr 2008 08:17:10 GMT]]>if( !v.empty()) { int avg = std::accumulate(v.begin(), v.end(), 0) / static_cast<int>(v.size()); // weiter } ]]>https://www.c-plusplus.net/forum/post/1484998https://www.c-plusplus.net/forum/post/1484998Wed, 02 Apr 2008 08:17:10 GMT<![CDATA[Reply to findet jemand den Fehler? on Wed, 02 Apr 2008 09:15:49 GMT]]>bei den hier genannten Lösungen musst du aber dann noch aufpassen, dass v.size() einen bestimmten Wert nicht überschreitet(2^31? nicht ganz sicher), denn int nutzt das höchstwertigste bit als vorzeichenbit.

]]>https://www.c-plusplus.net/forum/post/1485039https://www.c-plusplus.net/forum/post/1485039Wed, 02 Apr 2008 09:15:49 GMT<![CDATA[Reply to findet jemand den Fehler? on Wed, 02 Apr 2008 17:02:21 GMT]]>So schnell wird er den std::vector wohl nicht mit 2 Milliarden Werten füllen können, denn das wären dann ja schon 8 GB Hauptspeicher...

]]>https://www.c-plusplus.net/forum/post/1485382https://www.c-plusplus.net/forum/post/1485382Wed, 02 Apr 2008 17:02:21 GMT