<![CDATA[Vector.erase liefert unbeabsichtigte Ergebnisse]]>Hi,
ich habe einen float-Vector und möchte das größte und das kleinste element löschen,

Das funktioniert jedoch nicht, dass Programm löscht mir die Hälfte der Zahlen.

for (size_t i = 0; i < zufallszahlen.size(); ++i)
	{
		if (zufallszahlen[i] == min_E || max_E)
		{
			zufallszahlen.erase(zufallszahlen.begin() + i );

		}
	}

Das kleinste und größte Element findet es, dass haben mir meine Ausgaben bestätigt, aber es löscht wahllos Zahlen

Danke im Voraus für jegliche Hilfe

]]>https://www.c-plusplus.net/forum/topic/334090/vector-erase-liefert-unbeabsichtigte-ergebnisseRSS for NodeSun, 26 Apr 2026 01:02:12 GMTFri, 21 Aug 2015 08:44:30 GMT60<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 08:44:30 GMT]]>Hi,
ich habe einen float-Vector und möchte das größte und das kleinste element löschen,

Das funktioniert jedoch nicht, dass Programm löscht mir die Hälfte der Zahlen.

for (size_t i = 0; i < zufallszahlen.size(); ++i)
	{
		if (zufallszahlen[i] == min_E || max_E)
		{
			zufallszahlen.erase(zufallszahlen.begin() + i );

		}
	}

Das kleinste und größte Element findet es, dass haben mir meine Ausgaben bestätigt, aber es löscht wahllos Zahlen

Danke im Voraus für jegliche Hilfe

]]>https://www.c-plusplus.net/forum/post/2464990https://www.c-plusplus.net/forum/post/2464990Fri, 21 Aug 2015 08:44:30 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 08:49:48 GMT]]>Das ist immer wahr:

if (zufallszahlen[i] == min_E || max_E)
]]>https://www.c-plusplus.net/forum/post/2464991https://www.c-plusplus.net/forum/post/2464991Fri, 21 Aug 2015 08:49:48 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:09:17 GMT]]>Das liegt daran weil deine if-Bedingung als

if ((zufallszahlen[i] == min_E) || max_E)

ausgewertet wird. Wenn max_E was anderes als 0 ist, dann ist die Bedingung immer wahr, wie manni66 schon sagte.

]]>https://www.c-plusplus.net/forum/post/2464996https://www.c-plusplus.net/forum/post/2464996Fri, 21 Aug 2015 09:09:17 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:20:38 GMT]]>ok danke an euch zwei und wie umgehe ich das?

]]>https://www.c-plusplus.net/forum/post/2464997https://www.c-plusplus.net/forum/post/2464997Fri, 21 Aug 2015 09:20:38 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:21:19 GMT]]>Zudem stimmt Dein Laufindex nach dem Löschen nicht mehr - Du überspringst u.U. ein Element.

Nimm std::remove_if() aus <algorithm> . Erase-Remove-Idiom

]]>https://www.c-plusplus.net/forum/post/2464998https://www.c-plusplus.net/forum/post/2464998Fri, 21 Aug 2015 09:21:19 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:26:53 GMT]]>C++Developer2000 schrieb:

ok danke an euch zwei und wie umgehe ich das?

Das Eine ist wahr, oder das Andere.
Was ist das Eine?
Was ist das Andere?

]]>https://www.c-plusplus.net/forum/post/2465000https://www.c-plusplus.net/forum/post/2465000Fri, 21 Aug 2015 09:26:53 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:46:05 GMT]]>manni66 schrieb:

C++Developer2000 schrieb:

ok danke an euch zwei und wie umgehe ich das?

Das Eine ist wahr, oder das Andere.
Was ist das Eine?
Was ist das Andere?

Das eine is die schlechteste note(max_E)
Das andere die bestte note (min_E)
und die muss ich streichen von 20 zuvor erzeugten zufallszahlen sucht er sich max_E und min_E und soll sie streichen

]]>https://www.c-plusplus.net/forum/post/2465005https://www.c-plusplus.net/forum/post/2465005Fri, 21 Aug 2015 09:46:05 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:51:38 GMT]]>C++Developer2000 schrieb:

manni66 schrieb:

C++Developer2000 schrieb:

ok danke an euch zwei und wie umgehe ich das?

Das Eine ist wahr, oder das Andere.
Was ist das Eine?
Was ist das Andere?

Das eine is die schlechteste note(max_E)
Das andere die bestte note (min_E)
und die muss ich streichen von 20 zuvor erzeugten zufallszahlen sucht er sich max_E und min_E und soll sie streichen

Also lautet die Bedingung:
Wenn 6 wahr ist oder wenn 18 wahr ist? Wirklich? Denk nach!

]]>https://www.c-plusplus.net/forum/post/2465007https://www.c-plusplus.net/forum/post/2465007Fri, 21 Aug 2015 09:51:38 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:54:51 GMT]]>Ich würde es so machen:

auto min_max = std::minmax_element(zufallszahlen.begin(), zufallszahlen.end);
zufallszahlen.erase(min_max.first);
zufallszahlen.erase(min_max.second);
]]>https://www.c-plusplus.net/forum/post/2465008https://www.c-plusplus.net/forum/post/2465008Fri, 21 Aug 2015 09:54:51 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 09:56:29 GMT]]>TNA schrieb:

Ich würde es so machen:

auto min_max = std::minmax_element(zufallszahlen.begin(), zufallszahlen.end);
zufallszahlen.erase(min_max.first);
zufallszahlen.erase(min_max.second);

Da kann das erste erase allerdings min_max.second invalidieren.

]]>https://www.c-plusplus.net/forum/post/2465010https://www.c-plusplus.net/forum/post/2465010Fri, 21 Aug 2015 09:56:29 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 10:02:38 GMT]]>auto mm = std::minmax_element(begin(v), end(v)); auto min = *mm.first, max = *mm.second; v.erase(std::remove_if(begin(v), end(v), [min,max](int x){ return x==min || x==max; }), end(v)); ]]>https://www.c-plusplus.net/forum/post/2465012https://www.c-plusplus.net/forum/post/2465012Fri, 21 Aug 2015 10:02:38 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 10:33:30 GMT]]>Danke @manni 66 für deinen Tipp nachzudenken

Lösung:

for (size_t i = 0; i < zufallszahlen.size(); ++i)
	{

		if (zufallszahlen[i] == min_E ||zufallszahlen[i]== max_E)
		{
			zufallszahlen.erase(zufallszahlen.begin() + i);

		}
	}
]]>https://www.c-plusplus.net/forum/post/2465021https://www.c-plusplus.net/forum/post/2465021Fri, 21 Aug 2015 10:33:30 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 10:45:09 GMT]]>Dein Code überspringt aber manche Zahlen, wenn diese direkt aufeinander folgen. Wenn du es weiterhin als Schleife machen möchtest dann besser so:

auto it = zufallszahlen.begin();
while (it != zufallszahlen.end())
{
	if (*it == min_E || *it == max_E)
		it = zufallszahlen.erase(it);
	else
		++it;
}
]]>https://www.c-plusplus.net/forum/post/2465023https://www.c-plusplus.net/forum/post/2465023Fri, 21 Aug 2015 10:45:09 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 10:46:27 GMT]]>Wenn du die Noten "2, 2, 2, 3, 4, 4" hast, willst du dann am Ende "2, 2, 3, 4" oder nur "3" haben? Weil deine Lösung macht letzteres, in der Schule ist es aber nur üblich, eine der schlechtesten Noten zu löschen.

]]>https://www.c-plusplus.net/forum/post/2465024https://www.c-plusplus.net/forum/post/2465024Fri, 21 Aug 2015 10:46:27 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 10:48:16 GMT]]>ok danke für den hinweis, schau es mir dann mal an

]]>https://www.c-plusplus.net/forum/post/2465026https://www.c-plusplus.net/forum/post/2465026Fri, 21 Aug 2015 10:48:16 GMT<![CDATA[Reply to Vector.erase liefert unbeabsichtigte Ergebnisse on Fri, 21 Aug 2015 11:09:17 GMT]]>@vorrausdenker, dessen bin ich mir bewusst und habe das im Programm auch eingebaut, nur ich hielt das für diesen Thread nicht für relevant

]]>https://www.c-plusplus.net/forum/post/2465029https://www.c-plusplus.net/forum/post/2465029Fri, 21 Aug 2015 11:09:17 GMT