[sylwke3100]

Jest jakieś narzędzie do poukładania method i zmiennych w klasach (C++) alfabetycznie dla Linuksa?

#programowanie #cpp

[japer]

@sylwke3100: Doxygen jedyny słuszny dokumentator,

korzystaj z dokumentatorów, i nie będziesz musiał sortować :D

[sylwke3100]

@japer: No chyba nie. Ja nie chce generować dokumentacji tylko posortować kod dla własnej pzyjemności

[a.....r]

@sylwke3100: Musisz jeszcze pamiętać o jednej rzeczy. Takiego sortowania nie należy wykonywać jeśli jesteś w jakimś projekcie gdzie nad kodem pracuje więcej niż jeden człowiek. Jedna taka akcja sortowania albo formatowania i osobie postronnej, a nawet Tobie po jakimś czasie ciężko będzie stwierdzić jakie zmiany wystąpiły w konkretnej rewizji kodu.

No chyba, że program jest malutki lub pracujesz nad nim sam.

[anonim1133]

@sylwke3100: ide powinien gdzies miec liste metod i ja mozesz sobie sortowac dowolnie.

[sylwke3100]

@anonim1133: Lista jest posortowana ale kod nie ;/

@archlinuxuser: Przeiceż nawet jeśli ktoś w najgorszym wypadku takie corż zrobił to duży projekt raczej bez Gita czy w ostateczności SVNa nie powinien wg pracować a wtedy odnalezienie rewizji było by pewnie łatwiejsze/

[a.....r]

@sylwke3100: Odnalezienie rewizji nie jest problemem. Problemem jest porównanie co się zmieniło między wersjami. Narzędzia typu diff w żaden sposób się nie połapią, że poprzenoszone zostały linijki. W przypadku plików o nietrywialnym rozmiarze kod "przed" i "po" będzie wyglądał jakby został napisany od nowa i nie da się zauważyć rzeczywistych zmian.

[Marmite]

@archlinuxuser: Ciekawe czy są jakieś cwane diffy, które zamiast standardowego algorytmu znajdowania najdluższego wspólnego podciągu porownują rzeczywiste fragmenty kodu, takie jak konkretne zmiany w klasie i jej metodach czy tym podobne.

[a.....r]

@Marmite: Słyszałem o jakimś narzędziu .NET które coś takiego robi, ale nie znam szczegółów, bo nie jestem .NET-owcem.

[Yahoo_]

@sylwke3100: nie lepiej miec posortowane metody "tematycznie", a nie alfabetycznie?

[tell_me_more]

@sylwke3100: nie rób tego, potem jak zrobisz blame-a to 99% linijek będzie miało ostatnią zmianę w tej rewizji, w której poprawiałeś kolejność, numery linijek będą zupełnie inne, więc połapanie się, kto kiedy zmienił logikę w tej linijce i w związku z jakim zadaniem będzie bardzo utrudnione. A to jeden z lepszych sposobów komentowania kodu - pisanie numeru zadania w messagu przy commicie i można za 5 lat zrobić blame i widać

[sylwke3100]

@tell_me_more: Mam takie coś. Ale chciałbym żeby kod był w miarę poukładany bo teraz to tak nie zabardzo

[tell_me_more]

@sylwke3100: to refaktoring zrób - podziel na mniejsze klasy/funkcje, spróbuj coś uogólnić, żeby kod się nie powtarzał, albo żeby 2 metody zastąpić 1 z parametrem, a nie bawisz się pierdołami :)

Jak masz tyle metod, że potrzebujesz je sortować, to fakt, że nie są posortowane to najmniejszy problem.

[sylwke3100]

@tell_me_more: https://github.com/sylwke3100/cristallparser to ten kod. Co do niego proponujesz?

[tell_me_more]

@sylwke3100: Tak na szybko to nie widzę potrzeby - masz maksymalnie 8 metod w klasie, zwykle koło 5, nie ma w czym się pogubić.

Jeśli już coś refaktoryzować, to IMHO można by podzielić metodę

CristallParser::parseData(string RawData)

bo jest dość długa - można by zrobić prywatne metody boolowskie na warunki w tej metodzie i nazwać je jakoś opisowo, i prywatne metody na kod w gałęziach tego if then elsa.

Jeszcze zastanawiam się,

[sylwke3100]

@tell_me_more: searchinvoke() będzie wywalona bo po pewnych zmianach wcześniej utraciła swoje zastosowanie i będą te brzydkie informacje zamienione na ładne (bo już mam napisane typy)

Co do podzielenia na mniejsze metody to sam nie mam pomysłu ani na nazwy ani na jako tako organizacje tego choć fukcja parseData sama się rozrosłą tak jakoś)

[tell_me_more]

@sylwke3100: może np tak

if (isSingleRule(element) && ruleStartsAt(element, RawData, pos)) {

     handleSingleRule(element, RawData, pos);

} else if (isMultiRule(element) && ruleStartsAt(element, RawData, pos)) {

     handleMultiRule(element, RawData, pos);

} else if (isSpecialRule(element, pos) && jakToNazwac(element, RawData, pos)) {

     handleSpecialRule(element, RawData, pos);

}

Albo można w ogóle pójść we wzorzec obiektowy i utworzyć klasy SingleRule, MultiRule i SpecialRule, którę dziedziczą po abstrakcyjnej Rule, i mają wirtualne metody

bool Rule::matches(element, RawData, pos) // sprawdzającą warunek

[sylwke3100]

@tell_me_more: RawData to jest zwykły string tak swoją drogą, element to kolejny Obiekt w vectorze Operacji. pos to aktualna pozycja stringa. Poprawiłem wcześniej o czym mówiłeś i zarazem jeszcze jeden błąd który przy okazji znazłem z tym że jak ktoś by się ustawił zamiast kropi przecinek to funkcja wykrywająca liczby zmiennoprzecinkowe by mu tego ie wykryła

[tell_me_more]

@sylwke3100: aha, pos jest zmieniane w handleXXX , więc będzie trzeba przekazywać je przez referencję

[sylwke3100]

@tell_me_more: To pomógł byc mi to wprowadzać do kodu (dołaczyć do projektu)?