zad 3 #29

s426285 · 2018-06-25T13:48:05+02:00

s426285 commented

2018-06-25 13:48:05 +02:00

First-time contributor

No description provided.

s426285 commented

2018-06-25 13:49:24 +02:00

First-time contributor

Program przyjmuje argumenty z konsoli (argv), na pierwszym miejscu string, a na drugim flagę -e (1. punkt zadania) lub -d (2. punkt zadania). Bazuje na wcześniejszej implementacji działań na wielomianach, nie ma optymalizacji. Przykładowe uruchomienia na LTS:

python3 hw3.py abc -e

python3 hw3.py abcQJ -d

Zadanie w terminie wysłałem mailem, ponieważ gdy wtedy próbowałem zrobić pull request Git zwracał 404 error.

Program przyjmuje argumenty z konsoli (argv), na pierwszym miejscu string, a na drugim flagę -e (1. punkt zadania) lub -d (2. punkt zadania). Bazuje na wcześniejszej implementacji działań na wielomianach, nie ma optymalizacji. Przykładowe uruchomienia na LTS: ``` python3 hw3.py abc -e ``` ``` python3 hw3.py abcQJ -d ``` Zadanie w terminie wysłałem mailem, ponieważ gdy wtedy próbowałem zrobić pull request Git zwracał 404 error.

kalmar commented

2018-06-25 16:20:28 +02:00

Dziękuję;

Program wyrzuca na konsolę tekst w UTF-8, co powoduje niemożność odczytania FCS; encode powinno zwracać bytes object, wtedy nie będzie problemu, np. bytes(some_string, 'unicode_escape')
np. dla Hello world! fcs powinien być 0xbd22, tuaj jest ½", co prawie się zgadza biorąc pod uwagę, że ½ to 0xc2bd (znaki unicode mają podwójną szerokość).

Swoją drogą nie wiem skąd python bierze te pierwsze c2...

Dziękuję; Program wyrzuca na konsolę tekst w UTF-8, co powoduje niemożność odczytania FCS; encode powinno zwracać bytes object, wtedy nie będzie problemu, np. `bytes(some_string, 'unicode_escape')` np. dla `Hello world!` fcs powinien być `0xbd22`, tuaj jest `½"`, co _prawie się zgadza_ biorąc pod uwagę, że `½` to `0xc2bd` (znaki unicode mają podwójną szerokość). Swoją drogą nie wiem skąd python bierze te pierwsze `c2`...

s426285 commented

2018-06-26 17:28:19 +02:00

First-time contributor

Program zmieniony by wyświetlanie znaków nie było problemem

python3 hw3.py -e "Hello world"
['H', 'e', 'l', 'l', 'o', ' ', 'w', 'o', 'r', 'l', 'd', 'E', '\x91']

python3 hw3.py -d "Hello world" "['E', '\x91']"
True

Program zmieniony by wyświetlanie znaków nie było problemem ``` python3 hw3.py -e "Hello world"``` ```['H', 'e', 'l', 'l', 'o', ' ', 'w', 'o', 'r', 'l', 'd', 'E', '\x91']``` ``` python3 hw3.py -d "Hello world" "['E', '\x91']" ``` ```True ```

kalmar commented

2018-06-26 21:22:43 +02:00

ok, uwagi do kodu:

przeciążanie __truediv__ jest ok, chociaż w przypadku tego co robimy (chyba?) lepsze byłoby przeciążenie __mod__
sporo używania operatora % można by uniknąć gdyby elementy były od razu typu IntMod; No i można było użyć klasy Polynomial z pliku obok (dzielonego pomiędzy hw2.py, hw3.py a za moment hw4.py ;-)
ogólnie encode powinno zwracać ba = bytearray(....), bo jest to najbardziej przenośne (nie interesuje nas kodowanie, itp), wtedy to co trafia do decode możemy podzielić fcs = ba[-2:0]; msg = ba[0:-2].decode('ascii').

ogólnie nieźle!

ok, uwagi do kodu: 1. przeciążanie `__truediv__` jest ok, chociaż w przypadku tego co robimy (chyba?) lepsze byłoby przeciążenie `__mod__` 2. sporo używania operatora `%` można by uniknąć gdyby elementy były od razu typu `IntMod`; No i można było użyć klasy `Polynomial` z pliku obok (dzielonego pomiędzy `hw2.py`, `hw3.py` a za moment `hw4.py` ;-) 3. ogólnie encode powinno zwracać `ba = bytearray(....)`, bo jest to najbardziej przenośne (nie interesuje nas kodowanie, itp), wtedy to co trafia do decode możemy podzielić `fcs = ba[-2:0]; msg = ba[0:-2].decode('ascii')`. ogólnie nieźle!

kalmar closed this pull request

2018-06-29 01:48:02 +02:00

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

2 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: kalmar/DALGLI0#29