du1_jachym_cernik's Introduction
du1_jachym_cernik's People
du1_jachym_cernik's Issues
Funkce používají proměnné, které jim nebyly předány jako argumenty
Viz to, co bylo na cvičení - veškeré proměnné, které jsou použity ve funkci by v ní měly být deklarovány nebo jí předány jako argumenty.
Kolečko neškáluje při změně velikosti pole.
Informace o tom, co dělá která funkce, se píšou do dokumentačního komentáře
Dokumentační komentář je v """
první řádek pod definicí funkce.
Kontrola vstupů je zbytečně složitá
Ve výsledku funguje, ale stačil by jediný while
cyklus, a to while row > pocet_radku or row < 1 or column > pocet_sloupcu or column < 1: # zajisti ze player nemuze dat input mimo rozmezi hriste
, všechny ostatní jsou tam nadbytečné.
Dokumentační komentář patří hned pod `def`
Abych věděl, co funkce dělá dřív, než začnu číst její kód.
Funkce používají proměnné, které v nich ani nevznikly ani je nedostaly jako argumenty
V tomto úkolu to nemá na bodování vliv, ale je to připomenutí pro příště.
První má hrát křížek
Proměnné by měly mít samovysvětlující názvy
Komentář by měl být pouze doplňkem popisujícím případné nejednoznačnosti.
Lépe využít zbytek po dělení dvěma pro zjištění, který hráč je na tahu.
Porovnávání, jestli celočíselná a neceločíselná polovina jsou stejné bude pravděpodobně také fungovat, ale je to méně čitelné forma.
Program neošetřuje chybné vstupy
Například zápornou velikost plochy, ale je tam toho mnohem víc.
Program je poněkud zamotaný
Velké množství ne úplně dobře popsaných funkcí činí program nepřehledným, už by se při takovéto složitosti hodilo mít dokumentaci, která by stručně popisovala, co která část programu dělá.
Dokumentační komentáře nepopisují, co funkce dělá
Například odkazují na proměnné, které se v dané funkci vůbec nevyskytují,
Program odmítá validní vstupy
vyber pocet radku3
vyber pocet sloupcu1
hraje kolečko
vyber řádek (1 až 3) - 3
vyber sloupec (1 až 1) - 1
souradnice jsou mimo hraci plochu, hraj znova
Oddělovat funkční celky prázdnými řádky
Funkční celky jsou například definice jednotlivých funkcí (je zvykem mezi dvěma definicemi vynechat řádek), vykreslení mřížky, úvodní definice, tahy každého hráče, ...
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.