I wrote this out to get the factorial of "x". I feel like it is not practical though, is it ok?

```
def factorial (x):
number = x
while x > 0:
x -= 1
if x >= 1:
number *= x
else:
return number
```

??