Project Euler 14

[Wictorian]

This says o isn't an int therefore it cant be ordered. Do you know why?

i = 2
o = 0
chain = 0
number = 0
def collatz(x):
    if x % 2 == 0:
        return (x/2)
    else:
        return (x*3+1)
def sequences(x):
    a = 1
    x = collatz(x)
    a += 1
    if x == 1:
        return(a)
while(i<1000000):
    o = sequences(i)
    if o > chain:
        chain = o
        number = i
    i+=1

i = 2
o = 0
chain = 0
number = 0
def collatz(x):
    if x % 2 == 0:
        return (x/2)
    else:
        return (x*3+1)
def sequences(x):
    a = 1
    x = collatz(x)
    a += 1
    if x == 1:
        return(a)
while(i<1000000):
    o = sequences(i)
    if o > chain:
        chain = o
        number = i
    i+=1

 

 

[Sauron]

def sequences(x):
    a = 1
    x = collatz(x)
    a += 1
    if x == 1:
        return(a)

this doesn't return anything if x is not 1 so when you run

    o = sequences(i)

you only get an integer if collatz(i) is 1. Otherwise it's None which can't be compared with an integer..

[Wictorian]

19 hours ago, Sauron said:

def sequences(x):
    a = 1
    x = collatz(x)
    a += 1
    if x == 1:
        return(a)

this doesn't return anything if x is not 1 so when you run

    o = sequences(i)

you only get an integer if collatz(i) is 1. Otherwise it's None which can't be compared with an integer..

Thanks, I should have put a loop there.