Ocena biblioteki statycznej do parsowania CMD

Odpowiedz Nowy wątek
2015-02-05 12:32
0

Witajcie,

Pisałem tę bibliotekę dawno. Ogólnie rzecz biorąc to był chyba pierwszy większy i o praktycznym zastosowaniu kod. Jeśli ktoś byłby zainteresowany to prosiłbym o ocenę kodu (szczególnie chodzi mi czy nie ma jakichś złych nawyków itd.). Na licencjach się nie znam. Chyba MIT może być do opensource, a nie tylko do binarek? Najbardziej odpowiadał właśnie MIT (to biblioteka statyczna więc gdyby była na GNU programy jej używające musiały by być także na GNU). Bibliotekę kompilowałem przez załączone Makefile i testowałem na GNU/Linux (da się pewnie to skompilować na Win bo właśnie tam była pierwotnie pisana, ponieważ używa tylko standardowych bibliotek C++). Link do GitHub: https://github.com/patrykbajos/CmdParser


128 postów [25.06.2015r. 21:03]
edytowany 1x, ostatnio: bajos, 2015-02-05 12:46

Pozostało 580 znaków

2015-02-05 13:01
2

Nie wczytałem się w logikę, więc tylko rzeczy które się rzucają w oczy:
1) Używanie nazw zaczynających się od _, np. _name
2) Przekazywanie obiektów przez wartość
3) Niepotrzebne używanie this->

Zamiast:

if (it == this->flags.end())
{
    return false;
}
else
{
    return true;
}

można

return it != flags.end();

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0