(zakończony)Warsztat 15: etap 06: KONTROLA JAKOŚCI KODÓW źródłowych

Borixon911 napisał:W projekcie MadagaskarUserInterface można by dodać metodę np. getColor - ustawiającą jedynie kolor czcionki a w pozostałych metodach wykasować wszystkie parametry związane z ustawianiem koloru.
Zmniejszyła by nam się wtedy znacznie liczba parametrów.

No proszę Borixon, a jednak wdrażając inne niż Twoje poprawki Foresa okazało się , że Ty masz jednak rację i metodę do zmiany koloru trzeba stworzyć .

Jeszcze takie malusienkie uwagi:

Klasa lpgCalculator:

Dla tych funkcji:

• FuelCostSavingPer1Km()
  
• TotalMileageKm()
  

dalbym przedrostek Get.

Klasa Console:
Zmiana nazwy metody z ChangeColor na SetColor.

Klasa CalculateForm
Zmiana nazwy metody z SetResultColor na GetResultColor.


I w samym wideo bardzo dokladnie bedzie trzeba zaznaczyc, ze z uwagi na poziom wiedzy zdajacego nie zastosowano wiele elementow jezyka, jakie powinny sie znalezc w tej aplikacji (wlasciwosci, konstruktory i jeszcze pewnie cos, by sie wiecej znalazlo)

Fores , ostatnie Twoje uwagi również dobre, wpisałem w arkusz, wdrożę.

Fores napisał:I w samym wideo bardzo dokladnie bedzie trzeba zaznaczyc, ze z uwagi na poziom wiedzy zdajacego nie zastosowano wiele elementow jezyka, jakie powinny sie znalezc w tej aplikacji (wlasciwosci, konstruktory i jeszcze pewnie cos, by sie wiecej znalazlo)

Mam właśnie taki zamiar, w takim razie zachęcam abyś się zapisał również (potwierdź mailem) do 11-go etapu produkcji laborki (kontrola jakości opisówki tekstowej pod filmem) ... czyli kontrola m.in tego, gdzie prelegent wciska kit uczestnikom ale też będziesz strażnikiem tego aby w opisówce wyraźnie było zaznaczone, że pewne techniki programowania nie są w w15 zastosowane ze wzgl. na ograniczenia bieżącego etapu szkolenia.

Ja mam uwagę co do aplikacji Morse Translator - a dokładniej to pewnie do klasy z której okienka korzystają. Byłoby według mnie czytelniej gdyby translator wstawiał znak "/" zamiast spacji w tłumaczonym zdaniu - wtedy widać poszczególne słowa.
Przy wstawieniu entera tłumacząc na język morsa wstawiany jest znak zapytania - w drugą stronę spacja - nie wiem czy to zamierzone czy traktować jak błąd, ale zwracam na to uwagę.
Napis "Latin <-> Morse translator" nie wygląda według mnie zbyt dobrze, szczególnie przez to <->. Może lepiej by wyglądało "Morse Code Translator"?
Nie mam więcej uwag po za tymi kosmetycznymi.

Wg mnie ta spacja jest zamierzona. Translator nie tłumaczy całego tekstu na morsa lecz pojedyncze słowa które są oddzielane spacjami. Alfabet morsa ma również znaki takie jak oddzielenie słowa od kolejnego słowa, początek nadawania, koniec nadawania itp. Można by zaznaczyć w wideo że translator morsa jest uproszczony.

Ja bym jednak nie widział slasha zamiast spacji, bo kod morsa charakteryzuje się serią kresek i kropek oddzielonych przerwami. Slash to będzie (wizualnie) 3 cia kategoria znaku, a więc przetłumaczony tekst nie będzie robił wrażenia kodu Morsa.

Hej

Zamykam ten etap.
Uważam, że bardzo fajnie to wyszło, bo aż 9-ciu kontrolerów zgłosiło ponad 30 poprawek do kodu (wdrożone wszystkie te, które przez autorów aplikacji zostały uznane za sensowne):
https://docs.google.com/spreadsheet/ccc?key=0AiEDbuEU88x4dHdsY293WVlOR05TRkhDMkktUnFyOFE&usp=sharing

Ostateczna wersja kodów znajduje się w znanej Wam już lokalizacji (jako ver. 3):
https://drive.google.com/folderview?id=0ByEDbuEU88x4ZGkxU200Y01PRkU&usp=sharing

Bardzo wszystkim dziękuję za wsparcie ! Blokuję ten wątek (dalsze ewentualne uwagi piszcie proszę w mailu) oraz rozpoczynam tym samym kolejny etap 07 (weryfikacja anglojęzycznych treści w aplikacjach warsztatu, wykonywana przez "native speakera" j. angielskiego).