Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Card's effects are applied on robots according to card's priorities #237

Merged
merged 10 commits into from May 20, 2019

Conversation

Projects
3 participants
@befeleme
Copy link
Contributor

commented May 12, 2019

Fixes #165

Dodatečně jsem pozměnila funkci apply_all_effects (tu naši hru) - už jsem narazila v testech, že kdy chci mít roboty na konci tahu mrtvé, nemůžu "stopnout" hru v tuto chvíli. Přišla mi tedy tato úprava vhodná pro kód, který mám v úmyslu napsat a rozšířit tak framework.

@befeleme befeleme added this to Rozdělané in Projekt May 13, 2019

Show resolved Hide resolved backend.py Outdated
Show resolved Hide resolved backend.py Outdated
Show resolved Hide resolved backend.py Outdated
@befeleme

This comment has been minimized.

Copy link
Contributor Author

commented May 16, 2019

Ještě jak na to myslím, mohla bych napsat porovnávání samotných karet, based on priority, jako metodu třídy Card, a následně nevytvářet sortující příkaz z lambdou a druhým prvkem dvojice, ale spíš použít to naimplementované porovnání. Tak si s tím ještě chvilku pohraju ve volné chvíli.

@befeleme

This comment has been minimized.

Copy link
Contributor Author

commented May 18, 2019

Tak jsem snad u konce, těším se na merge :)

@JureckovaK
Copy link
Contributor

left a comment

Za mě jen pár drobností.

Show resolved Hide resolved backend.py Outdated
apply_register(state, register)

except NoCardError:
print("No card on hand, continue to tile effects.")

This comment has been minimized.

Copy link
@JureckovaK

JureckovaK May 20, 2019

Contributor

Tady mi to vrtá hlavou. Může nastat situace, kdy roboti budou mít na ruce různý počet karet? Třeba v testech? Pokud to správně chápu, tak teď se provede registr pouze v situaci, kdy všichni roboti mají v daném registru kartu.

This comment has been minimized.

Copy link
@befeleme

befeleme May 20, 2019

Author Contributor

Za mne nemuze a s takovou situaci jsem zde nepocitala. Neplanuju taky psat testy tak, aby nekdo dostal 3 karty, a nekdo jiny 2.

This comment has been minimized.

Copy link
@JureckovaK

JureckovaK May 20, 2019

Contributor

OK, tak v tom případě za mě vše ok a můžeme začlenit.

This comment has been minimized.

Copy link
@befeleme

befeleme May 20, 2019

Author Contributor

Super. Po zacleneni tohoto PR by mel prestat padat PR s testovacim frameworkem, to hned opravim. :)

Show resolved Hide resolved test_backend.py
Update docstring
Co-Authored-By: Katerina Jureckova <38920177+JureckovaK@users.noreply.github.com>

@JureckovaK JureckovaK merged commit f0d3b2b into PyLadiesCZ:master May 20, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

Projekt automation moved this from Rozdělané to Hotovo May 20, 2019

@befeleme befeleme deleted the befeleme:priorities branch May 20, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.