Well I do not see any reason for the use of eval nor do I understand why there had to be 2 separate loops made to call the same function?
Also, what is the purpose of creating the 'main' function when all you then do is simply call it anyway, ie. why not just have the data in the main function left as the actual
code of the script as the function itself never needs to be called again so would seem redundant.
Your description also states:
Quote:
Generate 5 random numbers between 1 and 50 and two random
|
Yet you later state:
Quote:
# * 1 number between 1 to 50
# * 2 numbers between 1 and 25
# * 2 numbers between 26 to 50
|
Which would imply that they are not random between 1 and 50 as your solution will never arrive at the numbers 1,2,3,4,5 which is just as random as any other numbers within the associated boundary.
Lastly, you supply code that even according to you has a bug in it:
Quote:
@bug There is something wrong with this function
|
Which at least to me meant straight off that I would not run this code to try it. Admittedly better than hiding the fact, but I would suggest fixing said bug before raising this type of ticket.