[C++] Kod do oceny

Witam :slight_smile: Jako że od niedawna zaczołem coś więcej programować. Staram się pisać optymalne kody i przejrzyste.

Teraz Wam zaprezentuje kod programu i prosił bym Was o ocene tego kodu pod względem wydajności i optymalność.

Także bym prosił o rady/komentarza/co byście zmiennili aby przyśpieszyć działanie programu.

Program będe cały czas rozbudowywał i co jakiś czas będe nowe wersje na forum wrzucał :wink: Mam nadzieje że pomożecie :slight_smile:

Kod programu (271 linijek)

Z góry dzięki za wszelakie przejawy chęci pomocy :slight_smile:

Po bardzo pobieżnym przeglądnięciu:

→ pomyśl o bardziej czytelnym formatowaniu kodu

→ w kodzie wysokopoziomowym nie używa się goto, to nieeleganckie

→ czemu piszesz cały kod wewnątrz klasy?

Skoro dziękujesz za wszelakie :slight_smile: objawy chęci pomocy to dodam tylko :

–>Czemu zakładasz,że myszy nie zjadły pliku ,który chcesz otworzyć?

–>Wywal to wstrętne system(“pause”).

Nie rozumiem? Chyba klamerka gdzieś Ci umkneła :stuck_out_tongue:

Tutaj kod po “optymalizacji”. Jak by komuś się jeszcze chciało :stuck_out_tongue:

232 Linijki kodu

Dalej będe jeszcze konbinował jak funkcje szukaj_wpisow, nowy_wpis i edycja_wpisow przyśpieszyć…

  1. Takie rzeczy robi się za pomocą bazy danych.

  2. Wyszukiwanie liniowe jest nieefektywne dla dużej liczby rekordów.

  3. Nazywanie klasy “dane” jest bez sensu. Obiekty zwykle posiadają jakieś dane.

  4. Pisanie nazw skrótami (np. “mie”, “spr_costam”) powoduje, że kod jest mniej czytelny dla kogoś, kto go nie pisał.

  5. Niekonsekwentne nazwy: “data_podaj”, “pokaz_wpisy”. Zdecyduj się gdzie ma być orzeczenie a gdzie dopełnienie.