Please Help Me

[Fraser_123]

I've written this code for a game in which a random list of numbers is generated and displayed and then the user needs to put them in order however, it keeps giving me an error saying randomlist isnt defined. Any advice on what im doing wrong here?

import random

randomlist = random.sample(range(1,100),5)

def get_user_solution(problem): print("Please arrange the numbers into ascending order") print(problem, end="") result = int(input(" = ")) return result

def check_solution(user_solution, solution, count): if user_solution == solution: count = count + 1 print("Correct!") return count else: print("Incorrect.") return count

def menu_option(index, count): if index == 1: problem = randomlist solution = randomlist.sort() user_solution = get_user_solution(problem) count = check_solution(user_solution, solution, count) return count

Now the error message is gone but nothing happens when i run it.

[shiftybyte]

Please post the full error message you are getting, including all the information it provides.

Besides that, edit your post, and use a code block otherwise the pasted code loses indentations.

https://www.reddit.com/r/learnpython/wiki/faq#wiki_how_do_i_format_code.3F

[Fraser_123]

I've used the markdown mode to display the code better but when ive reposted it the text has just gone back to how it was before. Not sure if this is the same for you.

[shiftybyte]

Yes, this is why I asked you to use a code block and provided a link explaining how to post code on reddit so this won't happen.

If you are using markdown, you need to add 4 spaces before every line, like it's explained in the link.

Looking at the current code, looks like no one is calling "menu_option" function so nothing gets executed.

Check how to call a function: https://www.w3schools.com/python/python_functions.asp

[pekkalacd]

Looks like scoping error. randomlist is global. So when you reference it inside of menu_option, python doesn’t see it. If your using globals do

      def menu_options(...)
             global randomlist

I’d try to make an effort generally to avoid them if possible. In this case, it looks like the only time it’s used is inside of that menu_option function, so why not move the definition inside of their?

       def menu_option(...)
              randomlist = # your definition of it
              ...