I need to generate 100 random values between 0 and 500 and store the even values in a list and the odd in an other one. I am stuck in the beginning.
import random
def randomval():
size=random.randint(0, 500)
return
if val%2==0:
listeven.append()
else:
listodd.append()
both = [ random.randint(0, 500) for i in range(100) ]
odd = [ x for x in both if x % 2 == 1 ]
even = [ x for x in both if x % 2 == 0 ]
Solution depends on whether random number can be repeated or no.
random_values = [ random.randint(0, 500) for i in range(100) ]
odd = [ x for x in random_values if x % 2 ]
even = [ x for x in random_values if not x % 2 ]
And If You would like to get unique random number I would propose You to use shuffle
random_values = random.sample(range(0, 500), 100)
odd = [ x for x in random_values if x % 2 ]
even = [ x for x in random_values if not x % 2 ]
You can reduce memory usage using generators for example
random_values = (random.randint(0, 500) for i in xrange(100))
odd = []
even = []
for val in random_values:
if val % 2:
odd.append(val)
else:
even.append(val)
Do it all in One Pass - much more efficient.
from random import randint
even, odd = [], []
for _ in range(100):
num = randint(0, 500)
if num % 2:
odd.append(num)
else:
even.append(num)
As a function:
def get_random_even_and_odd(amount=100, _min=0, _max=500):
even, odd = [], []
for _ in range(amount):
num = randint(_min, _max)
if num % 2:
odd.append(num)
else:
even.append(num)
return even, odd
if statement. Just use if num % 2: odd.append(num), else: even.append(num). If you have to use a one-liner, odd if num % 2 else even).append(num) is possibly uglier but at least you use the conditional expression as an actual expression for its own sake.
if statement would mean to double the code massively. In this case it is just the .append(num) which gets doubled so it doesn't matter that much, but if that part is larger, then I'd prefer the conditional expression you proposed.
randomval()function doesn't return what you expected to get?return size. then callrandomval()function 100 times. then you are done.returnis now in the way and you seem to have forgotten to include a loop, but it's more of an attempt at coding it yourself now.