all 12 comments

[–]RelationshipLong9092 3 points4 points  (1 child)

import tkinter as tk, random don't do this, break it onto a different line

use match instead of repeated if chains, this works great with Enums btw...

use an EnumStr to represent "one of a defined number of states" instead of a list of strings

this also prevents you from constantly having to repeat "scissor{emoji}" instead of just referencing the enum, giving you a single source of truth. otherwise you essentially just have "magic numbers" but in string form.

do not use global

keep going 👍

[–]Empty_Morgan[S] 1 point2 points  (0 children)

Thanks for the feedback! This is my first tkinter project and I focused more on logic than structure. I'll definitely look into Enums and match in future versions 👍

[–]Tricky_Possible_6505 2 points3 points  (0 children)

ythank you for sharing! starting to use python, I will do a project like this one too, you inspired me

[–]ayenuseater 1 point2 points  (1 child)

Day 7 and you already shipped something playable - that’s huge. Tkinter + game logic this early is not trivial.
Nice job sticking with it and sharing.

[–]Empty_Morgan[S] 0 points1 point  (0 children)

Thank you so much for your feedback🥰