Zadanie 3 - CRC - wersja innym inputem #31
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "s426271/DALGLI0:Zadanie-3"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Poprawiona wersja inputa, teraz wystarczy przejść do folderu Zadanie-03\out\artifacts\Zadanie_03_jar
a następnie wpisać w konsoli java -jar Zadanie-03.jar a b
a - tryb (1 lub 2)
b - wiadomość
wygląda na to, że fcs jest pusty?? poza tym pisanie znaków do konsoli, które nie mieszczą się w standardzie ascii, albo nawet jeśli się mieszczą, są np znakiem DEL (ascii 127) nie jest najlepszym pomysłem.
fcs powinien być typu
bytesarray
(lub cokolwiek innego co java ma do zaproponowania co jest tablicą zawierającą liczby z przedziału 0-255)nie pisałem nigdy w javie, więc nie umiem zaproponować technicznego rozwiązania, ale liczę, że Pan je poda ;-)
Poprawione, teraz po zakodowaniu zwraca wiadomość w postaci wiadomośćX , gdzie X jest FCS np. Hello_world 0xc50e
gdybym miał się czepiać, to:
ale niech będzie że ok;
uwagi do kodu:
Przyznaję, że nie jest on doskonały (a ja nie piszę w javie), ale lepszy niż to co jest tutaj (miks operacji na stringach i liczbach)
To co tutaj widać, to programowanie proceduralne, dlaczego
CrcTask
ma atrybuty (i ile ich??)message
? przecież algorytmCrc
nie zależy od wiadomości?! Trochę lepiej napisane jestdecode
; co ma oznaczaćCrcTask.encode()
?! enkodujemy pusty string?co ma wspólnego
private short[] removeUnecessaryZeros(short[] remainder)
z klasąCrcTask
? a jeśli nic, to dlaczego jest jej metodą? czy (jeśli już) nie pasowałaby ta metoda do klasyPolynomial
??jest tego wiele...
Co do błędu "2 "Hello world" "0xbd22" to przez pomyłkę dodałem spację w opisie, ciąg argumentów powinien być następujący: 2 "Hello world0xbd22" .
Podsumowując to zdaję sobie sprawę, że jeśli chodzi o kwestię programistyczną to nie jest to najlepiej napisany program, lecz uważam, że spełnia wymogi zadania.
Gdyby się dokładnie w treść zadania wczytać, to oczywiście nie spełnia (
Hello world0xbd22
jest o 6 znaków dłuższy a nie o 2 itd.) ale w tego ping-ponga moglibyśmy grać w nieskończoność ;-), ponieważ Panu zależy na tym, żeby "spełniał wymogi zadania", a nie żeby się czegoś przy okazji nauczyć. A szkoda, bo to trochę strata i Pana i mojego czasu.No i to, że "spełnia wymogi zadania" to nie oznacza, że dostanie maksymalną liczbę punktów.
Ma Pan rację, dlatego zaimplementowałem klasę Polynomial, która wykonuje działania na wielomianach.
ok, mam nadzieję, że po napisaniu klasy
Polynomial
sam algorytm pisało się już Panu łatwo.Mi się na pewno znacznie znacznie łatwiej czytało ;-)
Pull request closed