[ProGraMajster]

Cześć!
Od jakiegoś dłuższego czasu piszę w csharp i tak się zastanawiam jak u mnie z jakością kodu...
Byłby ktoś chętny przejrzeć mój projekt?
Chętnie bym przyjmie jakieś rady itp.

Link do projektu -> link

Z góry dzięki!
#programowanie #csharp #dotnet #codereview #programista15k

[a5f5c1]

@ProGraMajster: a gdzie jakieś pakiety? ( ͡° ʖ̯ ͡°)

[a5f5c1]

@ProGraMajster: i po co te metody na pół kilometra? ( ͡° ʖ̯ ͡°)

[n.....e]

@a5f5c1: a jakie pakiety np?

[a5f5c1]

@nietrolluje: odpowiednie do rozwiązywanego problemu ( ͡° ͜ʖ ͡°)

[ProGraMajster]

@a5f5c1: Wiem że są dość duże tylko średnio mam pomysł jak nie które robić na mniejsze. Między innymi przez deserializacje z plików tekstowych ( ͡° ʖ̯ ͡°).

[Przemosz]

@ProGraMajster: Ale masz linijek kodu. Na moje oko wygląda gites ale jakiś .Net developer pewnie coś znajdzie co nie jest gites. Mnie to bardziej commity po polsku rażą XD ale rozumiem ze to miało być na próbę a nie jako commity projektowe. Trzeba by się zagłębić mocno w projekt to pewnie coś wyjdzie. Jakieś testy projektowałeś czy bez testów?

[Endriu_]

@ProGraMajster: Poczytaj sobie o async/await vs Task.Run()

[ProGraMajster]

@Przemosz: Bez testów ( ͡° ͜ʖ ͡°)

[ProGraMajster]

@Endriu_: Dzięki!

[ProGraMajster]

Daj tu linki do kilku pobieranych plików przez program.
Lista tytułów -> link
Szczegółowe informacje o tytule -> link
Lista odcinków -> link

News'y -> link

[n.....e]

odpowiednie do rozwiązywanego problemu ( ͡° ͜ʖ ͡°)

@a5f5c1: jestes najgorszym typem wspolpracownika ( ͡° ͜ʖ ͡°)

[PotatoZiemniak]

@ProGraMajster: try z pustymi catch, a inne z jakimś dzikim wypisywaniem do konsoli, inne nazwy klasy a inne pliku (form1.cs), część nazw metod po polsku a część po angielsku, troche zduplikowanego kodu też masz.

[PotatoZiemniak]

@ProGraMajster: zahardkodowane ścieżki - PageSettings.cs. ogólnie sporo różnych mniejszych i większych baboli, ale bez tragedii.

[ProGraMajster]

@PotatoZiemniak: Dzięki za sprawdzenie!

[n.....e]

@ProGraMajster: tak na szybko scrollujac bo usiadlem do kompa
Jest troche bałagan w projekcie, ale to prosty refactor bo nie narobiłeś za dużo syfu. Nazewnictwo, literówki, inconsistencies itp. - do fixniecia.

Inne co przyfilowałem:

public static String Icon { get { return "C:\ContentLibrarys\OtherFiles\WMPOverlayApp\Icon\"; } }

Ogolnie wszystko "C:\" to nic dobrego moze uzywaj %AppData% bedziesz blizej cross platform ;)

Przenieś wszystkie ikonki itp. do Resources np.
Biblioteka nie powinna tez

[n.....e]

@ProGraMajster: Jakbys kiedys chcial zrobic logowanie to wszystkie Console.WriteLine Cie moga ugryzc w dupe, wiec mozesz na konsole wypisywac libka do logowania np. NLogiem.
Tu masz config, ktory kolorowo wypisuje nawet :)

Tak w kazdej klasie sobie go wolasz:
private static readonly Logger Logger = LogManager.GetCurrentClassLogger();

Logger.Trace(">>>>> You can disable these entries in NLog.config -> minlevel=\"Info\" ");
Trace/info/warn etc.

NLog.config

xml version="1.0" encoding="utf-8"?

[ProGraMajster]

@nietrolluje: Dziękuj bardzo za sprawdzenie i za rady! Co do SQLite to nie używam, wszystkie pliki/dane które są pobierane znajdują się na dysku google i onedrive...

[n.....e]

@ProGraMajster: Wiem, przeczytaj jeszcze raz co napisalem. Chodzilo o to, ze libka nie powinna sama sobie decydowac, ze dziala na dysku C.

[ProGraMajster]

@nietrolluje: Ok rozumiem.

[object]

@ProGraMajster: klasyczne gówno-kody ale nie przejmuj się i klep dalej. Zawsze będą gówno kody tylko troche lepsze albo gorsze.