40 Most frequent - #32
Conversation
dslavicek
left a comment
dslavicek
left a comment
There was a problem hiding this comment.
V pull requestu máš commity s řešením předchozích úloh. Odstraň je prosím způsobem, který jsem ti napsal do PR k úloze 38.
V kódu máš připomínky k názvům proměnných. I když se můžou zdát nepodstatné na takových malých jednoduchých příkladech, tak v programování je důležité se naučit proměnné pojmenovávat výsitžně.
Jinak je řešení správné :-)
| """ | ||
| Vrátí nejčastěji se vyskytující prvek v seznamu. | ||
| """ | ||
| new_list = [] |
There was a problem hiding this comment.
new_list není vhodný název proměnné. Navrhuju třeba unique_values
| Vrátí nejčastěji se vyskytující prvek v seznamu. | ||
| """ | ||
| new_list = [] | ||
| counter = [] |
There was a problem hiding this comment.
Název counter mi nepřijde idální. Evokuje jedno počítadlo. Tady máš list počítadel. Navrhuji přejmenovat na množné číslo counters.
| """ | ||
| Vrátí nejčastěji se vyskytující prvek v seznamu. | ||
| """ | ||
| new_list = [] |
There was a problem hiding this comment.
Data která ukládáš do dvou seznamů new_list a counter by bylo elegantnější ukládat do jednoho slovníku, jehož klíče by byly z new_list a hodnoty z counter.
Toto není potřeba přepracovávat, je to jen tip.
|
|
||
| def maximum(counter): | ||
| max = 0 | ||
| for i in counter: |
There was a problem hiding this comment.
Navrhuji přejmenovat řídící proměnnou for cyklu z i na count
|
|
||
|
|
||
| def maximum(counter): | ||
| max = 0 |
There was a problem hiding this comment.
max je nevhodný název proměnné. Koliduje s vestavěnou funkcí max(). Proměnnou přejmenuj například na max_count.
Vyzkoušej si u sebe spustit následující program:
x = max(2,3)
print(x)
max = 7
y = max(3,4)
Uvididíš, že když něco uložíš do proměnné s názvem max, nemůžeš pak funkci max volat.
1631ec7 to
78af479
Compare
3 participants
No description provided.