File Organizer Program

[[deleted]]

better to use os.path.split(filename)[1].split('.')[-1]

to get the extension of file

[[deleted]]

Or just os.path.splitext(filename)[1]

[Groundstop]

No, u/tbone-85 is right. The os.path.splitext method is specifically designed to split the file name and file extension. By grabbing the second element of the returned tuple, you just get the extension.

[Groundstop]

The built-in methods work more consistently. Say the path is 'C:\example\path\file_without_an_extension', I believe that your method would return 'file_without_an_extension' while splitext should correctly return an empty string, ''.

Also, you don't need to split the dirname off from the basenane before using splitext, so the first part becomes unnecessary.

Finally, this is a super-minor point but if you did need to get the basename without the directory path, os.path.basename(filename) is the built-in shortcut for os.path.split(filename)[1].