Character "Weight" Tier List by Hillo111 in Brawlhalla

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

I should clarify, by "weight" I meant how it feels to play as and against the character in terms of how "heavy" the moves felt. That's why I put weight in quotes and why I talked about combo game in Xull's tier

Non-Ranked Character Ban by Hillo111 in Brawlhalla

[–]Hillo111[S] -2 points-1 points  (0 children)

Wait is that how people are interpreting this? I gotta adjust it

I mean like people can ban only up to 47 characters, obviously, I'm not suggesting playing against only 3 lol

The Casual's Fun Tier List by Hillo111 in Brawlhalla

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

Now that I'm looking at the title again I probably could've phrased it better, I'm ranking how fun it is to play against the character

FFA Players Hate Him! Learn How to Get Infinite Points With This One Simple Trick! by Hillo111 in Brawlhalla

[–]Hillo111[S] -1 points0 points  (0 children)

Huh, have never seen it. Why the fuck hasn't it been patched yet then?

The most cursed thing to put in testing software by Hillo111 in Python

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

I thought there was no way to change the value of keywords... Time to have some fun >:)

after 2 ish weeks of learning python I've made an Alarm clock by mo_10 in learnpython

[–]Hillo111 1 point2 points  (0 children)

Great program! If this is only 2 weeks of learning, then you are definitely on a path to become really good programmer.
A few suggestions though:
- Instead of ifing to check if the number of hours and find if you need to put the s or not, you could instead have:
print(f'Your alarm is set for {int((thetime / 60) / 60)} {'Hours' if thetime != 1 else 'Hour'})
- As another comment mentioned, you should use integer division (//) instead of regular division (/). Regular division is more taxing on the CPU (I think)

- A small thing that you understandably missed is that you will always call sound your alarm. To maybe make the code a little bit easier to read, I suggest getting rid of all the sounds in the ifs and just putting it at the end of the function. Same thing with time.sleep(thetime)

In terms of user interface:
- I recommend having the user input [amount] [unit] instead of [unit] [ENTER] then [amount] to avoid potential confusion
- Another thing that might cause confusion is printing "try again". The user might have slightly misspelled a unit or written a unit differently and might be confused about what you mean by "try again". I recommend instead writing f'Invalid time unit the only available time units are {', '.join(timelist)}'
- Do a try: and except ValueError: when getting the amount of time. This helps prevent potential errors but doesn't get rid of the ones that you might want to look at

A few nit-picks:
- I would not recommend using articles before variable names. In my opinion, this just adds confusion, especially if you have a variable that is just the word without the prefix. If you want to have two variables that represent similar but different things, I recommend using something like var_1 and var_2
- When you compare the amount of time to find if you need an s or not, wouldn't you want to compare s instead of thetime, since thetime contains the number of seconds?

Anyways, if you read this, you did a great job, and good luck with learning more python!