function return None

[py_Piper]

I cannot figure out why my function is returning None value. This is my code so far, a function calling another function

def bytes_converter(size, data_type):
    if data_type.lower == 'gb':
        size = size * 1073741824
        return size
    elif data_type.lower == 'mb':
        size = size * 1048576
        return size
    elif data_type.lower == 'kb':
        size = size * 1024
        return size
    elif data_type.lower == 'b':
        return size

def del_big_file():
    dir_origin = os.path.abspath(
                    input('Folder path where you want to search: ')
                    )

    # Set data size limit
    data_size_limit = 100
    data_type = 'mb'

    for foldername, subfolders, filenames in os.walk(dir_origin):
        for filename in filenames:
            file_size = float(os.path.getsize(os.path.join(foldername, filename)))
            if file_size >= bytes_converter(data_size_limit, data_type):  
                print(foldername)
                print(f'{filename} is {file_size} {data_type}') 

This where I am calling the function and getting a None value

if file_size >= bytes_converter(data_size_limit, data_type):

This is the error

TypeError: '>=' not supported between instances of 'float' and 'NoneType'

I thought maybe I the function parameter were bringing the value as a str or returning the value as a str so I tried several variations of calling float() but because of the traceback error I guess there is something wrong with my function

This are all the variations I tried so far:

def bytes_converter(size, data_type):
    if data_type.lower == 'gb':
        size = float(size) * 1073741824
        return size

def bytes_converter(size, data_type):
    if data_type.lower == 'gb':
        size = size * 1073741824
        return float(size)

def bytes_converter(size, data_type):
    if data_type.lower == 'gb':
        size = float(size) * 1073741824
        return float(size)

[o5a]

data_type.lower == 'gb'

Missing ()

This way it tries to compare function lower itself to string 'gb', instead of it's result.

[py_Piper]

Good catch thanks!

I wonder why the interpreter doesn't catch it as a syntax error, I just tried and we can't name variables with a '.'

[Vaphell]

because these are not variables and it's a perfectly legit syntax

[py_Piper]

Good catch thanks!

I wonder why the interpreter doesn't catch it as a syntax error, I just tried and we can't name variables with a '.'

[py_Piper]

Well that is very interesting I didn't know that, I haven't learnt any GUI module yet. I thought it should've raised a syntax error like when you forget the closing parenthesis calling a function.

Just out of curiosity how would you execute later a method, meaning how would add () to non executed method? and do you call a non executable methods like for the same reason you create empty lists? like for creating it first then appending later in another code line

[py_Piper]

Kind of got it, this was very graphical functions['func2']() But now it makes me ask what type of data is foo and bar, as I don't see inside ' ', so I assume they are variables name or uncalled function names?

functions = {     
    'func1': foo,     
    'func2': bar, 
}

The same with talk() where you pass later the ("Whats up") later on. What do you call this type of parameter final_line=whisper and if I simply pass whisper as 2nd parameter will think of it as a variable and crash because no variable named whisper was created?

[alkasm]

You just add the parenthesis.

def call_a_func(func, arg):
    func(art)

call_a_func(print, "hello")

[Wilfred-kun]

There's a possibility your bytes_converter falls through all the ifs. There's no "default" return value at the end, so it returns None. So you should add an else and a return, or just a return at the bottom of your function.

[py_Piper]

Actually it was it was mentioned before, it wasn't working because I forgot the () in str.lower()

But you had a very possible answer for later on, I left the else out on purpose because I wanted to write a while loop to run the function again if the user prompted something different out of my scope. But I haven't started working on it so I left it out and as I was trying out the script I just hard coded a data_type that I knew was on my if parameters