#programowanie #csharp #uwp #programista15k
tldr; #naukaprogramowania i proszę o sprawdzenie kodu, czy jest w miarę okej
Mirki, ostatnio zamieszczałem kod i prosiłem, byście zerknęli, czy czy w miarę jest okej, parę błędów wytknęliście, więc zabrałem się za inny mini projekcik, który jest prawie ukończony, jedynie można dodawać do niego różne bajery. Nie wykryłem żadnych błędów. (Mam nadzieję, że prawo murphiego nie działa ^^)
Czy ktoś miałby troszkę czasu i przejrzenie kodu pod kątem różnych rzeczy, które mogłyby zostać napisane lepiej?
Projekt to wrapper planu lekcji do szkoły, pisany w UWP w języku C#,
czyli pobieram źródło strony, formatuję, zapisuję do klasy, wyświetlam dane w programie. Postawiłem też na wersję offine, czyli zapis do XML (nie opłacało mi się bawić w sql)
Rzeczywisty plan: http://www.zs-1.pl/plany_nauczyciel
A jako obrazek daję zrzut z mojego programu :)
Link do GitHub: https://github.com/gtteamamxx/ZS1Plan/tree/master/ZS1Plan
tldr; #naukaprogramowania i proszę o sprawdzenie kodu, czy jest w miarę okej
Mirki, ostatnio zamieszczałem kod i prosiłem, byście zerknęli, czy czy w miarę jest okej, parę błędów wytknęliście, więc zabrałem się za inny mini projekcik, który jest prawie ukończony, jedynie można dodawać do niego różne bajery. Nie wykryłem żadnych błędów. (Mam nadzieję, że prawo murphiego nie działa ^^)
Czy ktoś miałby troszkę czasu i przejrzenie kodu pod kątem różnych rzeczy, które mogłyby zostać napisane lepiej?
Projekt to wrapper planu lekcji do szkoły, pisany w UWP w języku C#,
czyli pobieram źródło strony, formatuję, zapisuję do klasy, wyświetlam dane w programie. Postawiłem też na wersję offine, czyli zapis do XML (nie opłacało mi się bawić w sql)
Rzeczywisty plan: http://www.zs-1.pl/plany_nauczyciel
A jako obrazek daję zrzut z mojego programu :)
Link do GitHub: https://github.com/gtteamamxx/ZS1Plan/tree/master/ZS1Plan
- Najlepsze
- Wszystkie komentarze
@MroczekBall3D: xD
@MroczekBall3D: Ta, to samo mówiła dziwczyna, która pisałą kalkulator wyborczy xD
Zablokuj aplikacji dostęp do neta to pewnie się wysypie.
1. Czemu nie korzystasz z konwencji nazewniczej stosowanej w c# (click)?
2. 1 klasa, 1 plik.
3. metoda HTMLServices::getData ma prawie 200 linii... Na pewno da się to jakoś ładniej zrobić.
4.
// 2, because 0 is h'our number, 1 is a ring time (eg. 7:10 -> xxx )<-- od tego są enumy.5.
if (item.InnerHtml == @" "). Magiczny string. Daj zmienną, która powie co to w@player11one: @MroczekBall3D: żadnych przedrostków! Visual Studio pokazuje takie rzeczy i dopisywać jakieś cudowne prefixy w nazwach zmiennych/funkcji to przeżytek
popatrz na ciekawą zależność -> wszystkie twoje metody są publiczne (a na pewno wszystkie w Connections.cs). Zrobienie prywatnej metody często może sie opłacić, zwiększa to znacznie czytelnośc kodu -> dajesz opisowe nazwy i czytasz kod jak książke, a nie zastanawiasz się co ty sie dzieje. Ogólna zasada - krótkie funkcje, opisowe nazwy, mało (najlepiej brak) argumentów przyjmowanych przez funkcje!
zahardcodowana nazwa pliku z planem w xml
@MroczekBall3D: Zainstaluj sobie resharper ;). Tylko ostrzegam - uzależnia, potem nie chce się robić bez :D.
Jeszcze muszę zrobić użytek z HasInternetConnection, mysle, ze tylko to oraz dzialania na plikach mogą powodowac problemy
@robroypl: Czyli anty-debilny kalkulator w Cpp.
@Szab @arhu @Mave @ZapomnialWieprzJakProsiakiemByl : Wołam was chłopaki, ostatnio pomogliście C:
2. Wezmę pod uwagę ;)
3. Mam pomysł, jak to zrobić, dzięki :D
4. Nie rozumiem,
Jeśli chodzi o hardcode nazwy plików - załóżmy, że chcesz miec dwa plany, albo historię planów i wczytywać ten, który aktualnie obowiązywał w jakimś czasie - ile zmian w kodzie musisz zrobić? Niby jedną, ale wiąże się to z przekompilowaniem projektu. Niezbyt skalowalne rozwiązanie ;)
Poza