kalmar/DALGLI0
1
0
Fork 15
Code Issues Pull Requests 2 Releases Wiki Activity

Zadanie 4 #34

Closed
s426284 wants to merge 17 commits from s426284/DALGLI0:Zadanie-4 into master
pull from: s426284/DALGLI0:Zadanie-4
merge into: kalmar:master
Conversation 1 Commits 17 Files Changed 4 +171 -40
s426284 commented 2018-06-28 11:00:16 +02:00
First-time contributor
Copy Link

Interpreter:

python3 quotient_ring.py 3 "[1,1,2,2]"
Interpreter: ``` python3 quotient_ring.py 3 "[1,1,2,2]" ```
kalmar commented 2018-07-01 01:42:09 +02:00
Owner
Copy Link

ok, nie jest źle, ale mogłoby być lepiej:

  1. Proszę nie dodawać statycznych metod add skoro jest metoda __add__ (i __radd__), które można przeciążać (podobnie: __mul__, __pow__, którą Pan przeciąża??, __mod__, __div__, etc.

  2. degree nie powinno być atrybutem, tylko metodą wielomianu, która np. znajduje współczynnik wiodący i zwraca jego indeks. Co się stanie jeśli wielomian będzie podany jako [1,0,0,1,0]?

  3. Proszę porównać Pana metodę create_remainder_set z https://git.wmi.amu.edu.pl/kalmar/DALGLI0/pulls/32/files#diff-8c83edd6f00219f13ca7a4bdb2d5762a98cb21fR16.
    Proszę wysnuć wnioski.

Dopóki Panowie nie uzgodnią kto od kogo, moje stanowisko jest takie, że widzę się z Panami na poprawce. A szkoda, bo było nieźle...

ok, nie jest źle, ale mogłoby być lepiej: 1. Proszę nie dodawać statycznych metod `add` skoro jest metoda `__add__` (i `__radd__`), które można przeciążać (podobnie: `__mul__`, `__pow__`, którą Pan przeciąża??, `__mod__`, `__div__`, etc. 2. `degree` nie powinno być atrybutem, tylko metodą wielomianu, która np. znajduje współczynnik wiodący i zwraca jego indeks. Co się stanie jeśli wielomian będzie podany jako `[1,0,0,1,0]`? 3. Proszę porównać Pana metodę `create_remainder_set` z https://git.wmi.amu.edu.pl/kalmar/DALGLI0/pulls/32/files#diff-8c83edd6f00219f13ca7a4bdb2d5762a98cb21fR16. Proszę wysnuć wnioski. Dopóki Panowie nie uzgodnią kto od kogo, moje stanowisko jest takie, że widzę się z Panami na poprawce. A szkoda, bo było nieźle...
kalmar closed this pull request 2018-07-01 23:25:40 +02:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#34
No description provided.
Delete Branch "s426284/DALGLI0:Zadanie-4"

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?