# tennis loading screen by snowyuwuwu :)
import os
import time
def clear():
os.system('cls' if os.name == 'nt' else 'clear')
clear()
animation = [
"o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
" o",
"o"
]
while(True):
for x in animation:
clear()
print(x)
time.sleep(.2)
#🔒 Code reviews needed :>
woven hazel
high talonBOT
```py
# tennis loading screen by snowyuwuwu :)
import os
import time
def clear(...
@woven hazel
Python help channel opened
woven hazel
any code suggestions or reviews? 0/10
arctic crypt
There isn't a ton here to review, so there's not much to say.
You can generate animation to make it cleaner and adjustable:
!e
animation = [(' ' * n) + 'O' for n in range(10)]
animation += reversed(animation)
print(*animation, sep='\n')
Oops
untold citrus
id recommend to use ansi sequences instead of guessing the clear command based on the os
and you could use a loop and multiply space by a counter instead of hardcoding the animation list i guess, ^
high talonBOT
!e
```python
animation = [(' ' * n) + 'O' for n in range(10)]
animation += rever...
:white_check_mark: Your 3.12 eval job has completed with return code 0.
001 | O
002 | O
003 | O
004 | O
005 | O
006 | O
007 | O
008 | O
009 | O
010 | O
011 | O
... (truncated - too many lines)
Full output: https://paste.pythondiscord.com/PRAUSMAUYEZBS44ZRFWVKHA3MA
woven hazel
One moment
untold citrus
id recommend to use ansi sequences instead of guessing the clear command based o...
printing "\x1B[2J" should do what cls/clear does but "cross-platform", the \x1B is like, a special byte thats used as the start for a bunch of "commands" like that
arctic crypt
You can also use itertools.cycle to make an infinitely long, repeating animation.
woven hazel
!e
tennis loading screen by snowyuwuwu 🙂
import os
import time
def clear():
os.system('cls' if os.name == 'nt' else 'clear')
clear()
animation = [(' ' * n) + 'O' for n in range(10)]
animation += reversed(animation)
while(True):
for x in animation:
clear()
print(*animation, sep='\n')
time.sleep(.2)
oops
high talonBOT
!e
# tennis loading screen by snowyuwuwu 🙂
import os
import time
def clear():
...
:x: Your 3.12 eval job timed out or ran out of memory.
001 | O
002 | O
003 | O
004 | O
005 | O
006 | O
007 | O
008 | O
009 | O
010 | O
011 | O
... (truncated - too many lines)
Full output: https://paste.pythondiscord.com/O66LOUPFWP2ANYRZ4F2RNGYN5E
woven hazel
I didn't expect that output
untold citrus
(ran out of memory because the buffer it prints (writes) to is well, memory)
arctic crypt
print(*animation, sep='\n') was just to display the example here.
woven hazel
# tennis loading screen by snowyuwuwu :)
import os
import time
def clear():
os.system('cls' if os.name == 'nt' else 'clear')
clear()
animation = [(' ' * n) + 'O' for n in range(10)]
animation += reversed(animation)
while(True):
for x in animation:
clear()
print(*animation, sep='\n')
time.sleep(.2)```It is not doing the animation
arctic crypt
You'd use the animation exactly l;ike you did before.
woven hazel
it is just cleaning the console and printing this
001 | O
002 | O
003 | O
004 | O
005 | O
006 | O
007 | O
008 | O
009 | O
010 | O
011 | O
print(*animation) "unpacks" the list in the animation and puts each item as like, a positional argument
e.g. print(*[1, 2]) == print(1, 2)
arctic crypt
Don't change the loop. Keep what you had before.
arctic crypt
Also, it would need to be adjusted slightly to avoid the double max line maybe:
max_n = 10
animation = [(' ' * n) + 'O' for n in range(max_n)]
animation = animation + [(' ' * max_n) + 'O'] + animation[::-1]
The main benefoit to this is you can adjust the width to match something like the screen size, and you don't need to hard-code every possiblity in the code.
opal mica
i'd say u dont necessarily need to make a generator for the animation
just writing it as you did at the first place is fine
well it all depends i guess on how long the code gets
your code is pretty short so u can make it very abstract i suppose
woven hazel
!close
high talonBOT
Python help channel closed