Komunikator TCP/IP

0

Dzień dobry

Ostatnio trochę siedziałem nad małym komunikatorkiem tcp/ip. Program napisany jest w języku C# (WPF). Poniżej zamieszczam kod do oceny z github'a:

https://github.com/NeuroXiq/ChatApp

Jeżeli chodzi o projekt, to jest on podzielony na trzy części : Klienta, serwer i common_resource które jest 'wspólne' dla obu części.
Z tego co sprawdzałem to wszystko powinno działać. Uruchomię do końca dnia server u siebie na kompie i jeżeli mógłby się ktoś zarejestrować i sprawdzić czy wszystko działa to byłbym wdzięczny. Tutaj sam .exe klienta:

http://www.filedropper.com/release_12

jeśli chodzi o dane to :

login/pass <- to dowolne i new account
IP: 77.254.30.4
port:1234

Zapraszam do oceny :)

0
  1. zapoznaj sie z czyms takim jak .gitignore i wywal niepotrzebne pliki/foldery
  2. Pod folderem Client tam nic nie ma
  3. w ChatApp/Chat_client/Program.cs
            Application a = new Application();
            LogInWindowViewModel VM = new LogInWindowViewModel();

oraz tona using
pokazuje, ze nie dbasz o szczegoly (powaznie takie zmienne?)
4) ogolnie widze nieporzadek w kodzie. Dborne sprawy, ale nadal...
5) LogInUser dla mnie za duze, jezeli jest success to bym tam wywolal funkcje z tym kodem co tam jest (zrobil prywatna funkcje)
6) formatowanie nie jest spojne

poza tym jezeli chodzi o kod to nie jest zle ;)

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