Random number #178
Random number #178
Conversation
| @@ -0,0 +1,4 @@ | |||
| from random import randint | |||
|
|
|||
| def random_number(min: int, max: int) -> int: | |||
There was a problem hiding this comment.
Il nome delle variabili in questa maniera e' estremamente esplicito, pero' c'e' il problema che vanno a sovrascrivere (e quindi nascondere) le funzioni built-in di python, max e min, che quindi non potresti andare ad utilizzare all'interno della funzione.
Potresti quindi optare per nomi alternativi come min_value o minimum o puffo. Magari non l'ultimo. A meno che tu non sia Barbanera.
| t = random_number(0, 100) | ||
| assert t >= 0 and t <= 100 |
There was a problem hiding this comment.
Volendo potresti mettere questo test in un ciclo giusto per fargli fare un po' di passate e vedere piu' output, ma essendo random in principio non cambia tantissimo
| from src.random_number import random_number | ||
|
|
||
| def test_random_number() -> None: | ||
| t = random_number(0, 100) |
There was a problem hiding this comment.
Dai un nome più "parlante" alla variabile t.
In questo contesto è ovviamente semplice da capire cosa contiene ma non è sempre così.
There was a problem hiding this comment.
Definisci delle variabili costanti che contengano 0 e 100.
Aggiunta funzione per generare numeri interi randomici e test da 0 a 100