caly program w mainie.

caly program w mainie.
CP
  • Rejestracja:ponad 8 lat
  • Ostatnio:ponad 7 lat
  • Postów:4
0

witam serdecznie odwiedzajacych,

Od niedawna uczę się c#. Ostatnio stanalem przed zadaniem ulepszenia prgramu w c#, ktory caly zostal napisany w mainie (okolo 400 linijek kodu).
Program jest aplikacja konsolowa, ktora odczytuje pliki xml z okreslonej lokalizacji i o ile sa one zgodne, to procesuje je dalej i zapisuje je na serwerze. Istnieje rowniez zapis do logow wykonywanych akcji.

Moim zadaniem jest stworzenie 2 (albo i wiecej metod):
1 metode ktora sprawdza czy plik jest ok i dalej go przetwarza
2 metode ktora zapisuje poprawny plik na serwerze
3 zastapienie goto (albo loggerem albo petla?)

moje problemy:
1 na obecny moment nie wiem gdzie moge dodac te dodatkowe metody w kodzie. Jak rozdzielac kod szukac zaleznosci itp.
2 wszystko od wszystkiego wydaje sie zalezne...obejrzalem kilka tutoriali o c#, rowniez czytam kilka ksiazek... O ile rozumiem na czym to wszystko polega to o tyle nie mam pojecia jak zqstasowac to (przerobic) w juz istniejacym kodzie.
3 prosze opodpowiedz jak mam sie poruszac po kodzie tak aby znalezc zaleznosci miedzy poszczegolnymi skaldnikami
4 niestety napisanie calej apki od poczatku nie wchodzi w rachube
5 czy ewentualnie jest ktos z Warszawy lub okolic kto pomoglby mi to zrozumiec? (oczywiscie odwdziecze sie za pomoc)

Wiem, ze pewnie moje pytania moga wydawac sie trywialne, ale na ten moment jest to dla mnie nie lada problem...a czas plynie i efektow nie ma :(
Bede wdzieczny za pomoc i porady

ŁF
Moderator
  • Rejestracja:ponad 22 lata
  • Ostatnio:około 13 godzin
0

Podziel na klasy funkcjonalnościami: odczyt pliku xml, porównywanie xml, procesowanie xml, zapis na serwerze, zapis do logów. Cztery lub pięć klas (bo zapis i odczyt można zrobić w jednej klasie).

[edit]
Doczytałem "zastąpienie goto". WTF? W co Ty wdepnąłeś? Użycie goto świadczy o skrajnym braku umiejętności programistycznych. Przepisywanie kodu napisanego przez takiego przedszkolaka programistycznego będzie kosztowało więcej czasu, niż napisanie go na nowo.


edytowany 1x, ostatnio: ŁF
CP
  • Rejestracja:ponad 8 lat
  • Ostatnio:ponad 7 lat
  • Postów:4
0

Niestety, ze tak powiem "ciezkie" poczatki...ogolnie kod porgramu jest po zdekodowaniu, wiec sadze, ze poczatkowo nie bylo tego typu kwiatkow.

Patryk27
kodowanie oznacza czynność, którą można odwrócić w stosunku 1:1 (czyli kodowanie oraz dekodowanie następuje bez utraty jakichkolwiek informacji). To, o czym piszesz, to dekompilacja.
somekind
@Patryk27, dekompilacja w .NET jest właściwie 1:1, więc to niemalże dekodowanie. :)
Progress
Może kod jest obfuskowany i stąd to goto
CP
  • Rejestracja:ponad 8 lat
  • Ostatnio:ponad 7 lat
  • Postów:4
0

nie wiem, ale czy ktos bylby w stanie mnie naprowadzic jak to moge zmienic. Wiem ze na petle ale co i jak dalej, to nie mam pojecia

Aventus
  • Rejestracja:około 9 lat
  • Ostatnio:ponad 2 lata
  • Lokalizacja:UK
  • Postów:2235
0

Ehh, uwielbiam to forum. Ilekroć jakiś <ort>świerzak</ort> prosi o poradę to ludzie wyskakują z bardziej zaawansowanymi tematami. Kolega walnął wszystko w mainie co oznacza że nawet nie wie jak prawidłowo rozbić program na funkcje/metody a ktoś tu z klasami wyjeżdża, litości...

@chce_programowac poczytaj sobie o metodach i funkcjach a o klasach <ort>na razie</ort> zapomnij. Pomyśl jak możesz rozbić swój kod na bloki odpowiedzialne za konkretne zadanie. W razie pytań pisz. Jeśli piszesz i uczysz się w języku angielskim (do czego zachęcam) to wyszkuj pod hasłami method (metoda) i function (funkcja). W necie jest masa tutoriali, wystarczy wyguglować.


Na każdy złożony problem istnieje rozwiązanie które jest proste, szybkie i błędne.
edytowany 2x, ostatnio: ŁF
4p
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 6 lat
  • Postów:28
0

Zacznij od testów. Dlaczego? Zapewnisz sobie jakaś siec by wykryć czy udał się kolejny etap refactoringu. Do takich zadań są fajne biblioteki typu http://approvaltests.com/ którymi można szybko sobie ustawić testy wejście/wyjście. Nie martw się czy są to unitowe czy integracyjne, ważne zęby mieć jakieś testy zanim zaczniesz.

Jak nie czujesz się pewnie by to żywcem kroić, to prze-debuguj program parę razy, i zrozum co robi. Dodawaj sobie po kolei komentarze do kodu, tak abyś rozumiał co się dzieje. Jak już się mniej więcej połapiesz co jest grane, przystępujesz do następnego etapu.

A tu mianowicie to co sugerowali koledzy, tzn rozbij na zbiór metod które mniej więcej wykonują jedną czynność.

ŁF
Testy do kodu napisanego w jednej metodzie? Dość nietrywialne zadanie... A dla początkującego to Mount Everest.
vpiotr
@ŁF: ale przecież to już jest: https://github.com/sstephenson/bats
vpiotr
a nie zauważyłem że to Redmond, no to może https://github.com/pester/Pester
4p
@ŁF zawsze można przetestować wejście/wyjście. Czy to będą efektowne testy? Nie. Ale nie oto chodzi. Chodzi oto by zwiększyć prawdopodobieństwo że się cokolwiek wyłapie.
MarekR22
Moderator C/C++
  • Rejestracja:ponad 17 lat
  • Ostatnio:3 minuty
2

IMO 400 linii to nie dużo.
Zamiast przerabiać, dużo łatwiej będzie pisać od zera. Z opisu wynika, że masz dużo większego skilla od autora oryginału.
No chyba, że nie masz opisanej fikcjonalności i na podstawie kodu musisz ustalić co masz zrobić.


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.