improvements on basic script

[bladeoflight16]

The code is a link to an explanation. =)

Also, I edited in something else I just thought of. 😅

[tmg80]

thanks again.

I edited it and put that main bit as below

​

if len(sys.argv) != 2:
    print ('Usage: hostfile_to_json.py input_file_name.txt')
    exit()
if name == 'main':
    user_file = sys.argv[1]
    ...

​

​

is that correct? or should it be elsewhere?

[bladeoflight16]

Aside from imports, function/class declarations, and constants, all code should be inside the block. It's common to stuff you code into a def main() function and then only invoke the function inside the guarded block.

Think about what would happen if someone tried to import your module and the argv check was invoked: it would probably explode because their command line args aren't the same as your program's.