[Leinnan]

Mam sobie klasę Transaction. Ma ona jako pole QDateTime. Mam też klasę Account, a w niej std::Vector. Czy idzie posortować taką listę po dacie? Na pic rel widać co za cudo sobie tworzę, ma to być program do zarządzania wydatkami, na razie robię wersję konsolową, z czasem dojdzie apka desktopowa, a jak się uda to i mobilna.
#cpp #programowanie #leinnanblends

[that_ugly_dude]

@Leinnan:

in-place, o ile nie pokopałem składni i QDateTime jest porównywalne na pałę

std::sort(std::begin(transactions), std::end(transactions), [&](const Transaction& lhs, const Transation& rhs) { lhs.date() < rhs.date(); });

[Leinnan]

@that_ugly_dude: https://github.com/Leinnan/mybelka

[Leinnan]

@that_ugly_dude: Dzięki wielkie, ogółem std:sort musiałbym usiąść i zrozumieć co dokładnie robi, bo takie używanie "magicznych" kodów IMO na dłuższą metę jest złe.

[that_ugly_dude]

@Leinnan: w gruncie rzeczy ma być łatwe do zrozumienia - początek kontenera, koniec kontenera, funkcja porównująca, w bebachach sortuje w miejscu (quick sortem chyba)

[Leinnan]

@that_ugly_dude: Dzięki tobie w końcu usiadłem i zobaczyłem sobie jak z std::sort korzystać, okazuje się być prostsze niż myślałem. Musiałem w sumie do Transaction dodać gettera dla daty, ale już działa z taką funkcją dla sorta:
bool isDateLater(const la::Transaction& lhs, const la::Transaction& rhs){
return lhs.getDate() < rhs.getDate();
}

Gdy próbuje robić to z lambdą to mam błąd, w sumie nie wiem dlaczego, ale z lambd jeszcze nie umiem dobrze korzystać:

std::sort(std::begin(this->transactions),std::end(this->transactions),

[that_ugly_dude]

@Leinnan: nie ma to jak czytelny error z gcc ;p dodaj return przed komparator w ciele lambdy (ew. zerknij jeszcze tutaj: http://stackoverflow.com/questions/5122804/how-to-sort-with-a-lambda)

a co do lambd, to nie ma zbyt wiele filozofii w tym (przynajmniej w podstawowym użyciu)
[=] lub [&] -> jak łapane będą zmienne z zewnątrz do lambdy: czy przez kopię, czy przez referencję
(...) -> paramsy
{ ... } -> ciało

i drobne cechy lambd: każda lambda jest

[lionbest]

@Leinnan: Ta Lambda nic nie musi łapać, powinno być samo [] bez [&] ani [=].
Aha return jest konieczny, poprostu @that_ugly_dude zrobił literówkę.

[that_ugly_dude]

@lionbest: true, wbiło mi się, że puste nawiasy to domyślnie [=]