Nullen aus std::vector< std::vector< int > > löschen

for_each( MasterVector.begin(), MasterVector.end(),
        bind( &vector< int >::erase, _1,
            bind( &remove< vector< int >::iterator, int >, bind( &vector< int >::begin, _1 ), bind( &vector< int >::end, _1 ), 0 ),
            bind( &vector< int >::end, _1 )
        )
    );

So proggen nur totale Vollhupen.

hustbaer

Werner Salomon schrieb:

Für die Profis - sowas geht dann auch:

for_each( MasterVector.begin(), MasterVector.end(), 
        bind( &vector< int >::erase, _1,
            bind( &remove< vector< int >::iterator, int >, bind( &vector< int >::begin, _1 ), bind( &vector< int >::end, _1 ), 0 ),
            bind( &vector< int >::end, _1 )
        )
    );

Man kann sich das Leben auch unnötig schwer machen...
Ich würde das so schreiben:

#include <vector>
#include <algorithm>
#include <boost/foreach.hpp>

int main()
{
	std::vector<std::vector<int> > masterVector;

	BOOST_FOREACH(std::vector<int>& v, masterVector)
		v.erase(std::remove(v.begin(), v.end(), 0), v.end());
}

Finde ich etwas ... leserlicher

hustbaer schrieb:

Werner Salomon schrieb:

Für die Profis - sowas geht dann auch:

for_each( MasterVector.begin(), MasterVector.end(), 
        bind( &vector< int >::erase, _1,
            bind( &remove< vector< int >::iterator, int >, bind( &vector< int >::begin, _1 ), bind( &vector< int >::end, _1 ), 0 ),
            bind( &vector< int >::end, _1 )
        )
    );

Man kann sich das Leben auch unnötig schwer machen...
Ich würde das so schreiben:

#include <vector>
#include <algorithm>
#include <boost/foreach.hpp>

int main()
{
	std::vector<std::vector<int> > masterVector;

	BOOST_FOREACH(std::vector<int>& v, masterVector)
		v.erase(std::remove(v.begin(), v.end(), 0), v.end());
}

Finde ich etwas ... leserlicher

Hab ich was verpasst? Wo ist der SlaveVector hin?

volkard

JezMalButterBeiDieFische schrieb:

SlaveVector hin?

der heißt jetzt v.

Vorher wurde doch aus zwei Vektoren gelöscht und jetzt doch nur aus einem, oder?

volkard

JezMalButterBeiDieFische schrieb:

Vorher wurde doch aus zwei Vektoren gelöscht und jetzt doch nur aus einem, oder?

aus jedem v alle nullen.

std::vector<std::vector<int> > masterVector; 
BOOST_FOREACH(std::vector<int>& v, masterVector) 
   v.erase(std::remove(v.begin(), v.end(), 0), v.end());

ist

std::vector<std::vector<int> > masterVector; 
for(auto i=masterVector.begin();i!=masterVector.end();++i)) 
   std::vector<int>& v=*i;
   v.erase(std::remove(v.begin(), v.end(), 0), v.end());

std::remove(v.begin(), v.end(), 0) schmeißt sozusagen alle nullen raus, naja, löschen kann die funktion nicht. was sie aber kann, ist die nullen ans ende schieben und die richtigen werte an den anfang. und sie gibt zurück, wo nach dem umverschieben die nullen anfangen.
v.erase(woDieNullenANfangen); löscht dann die ganzen nullen.

aber das ist doch immer nur ein vector pro for-schleife, vorher waren es zwei in einer schleife.

volkard

JezMalButterBeiDieFische schrieb:

aber das ist doch immer nur ein vector pro for-schleife, vorher waren es zwei in einer schleife.

in remove steckt innendrin eine schleife.

hustbaer

Ne er hat schon Recht - vorher gab's zwei Schleifen UND zwei std::vector.

Der "slave Vector" is weg, weil ... den irgendwer mal weggelassen hat.
Mein Code bezog sich bloss auf das "bind-Monster" von Werner -- dass man das halt wesentlich einfacher (und kürzer) schreiben kann.

BTW: in C++0x wird's ja gottseidank lambda-expressions geben, dann muss man kaum mehr "binden".

Nexus

Wie würde das denn mit Lambda-Ausdrücken aussehen? Wäre es dann tatsächlich einfacher bzw. übersichtlicher?

hustbaer

ca. gleich wie die BOOST_FOREACH variante, nur ohne den "Hack" üder das Marko eben:

for_each(MasterVector.begin(), MasterVector.end(), [](std::vector<int>& v) {
    v.erase(std::remove(v.begin(), v.end(), 0), v.end());
});

Nullen aus std::vector&lt; std::vector&lt; int &gt; &gt; löschen

Nullen aus std::vector< std::vector< int > > löschen