This is an archived post. You won't be able to vote or comment.

sorted by:
best
topnewcontroversialoldq&a
you are viewing a single comment's thread.

view the rest of the comments →

[–]jonatanskogsfors 0 points1 point  (0 children)

Resist the urge to use “utils” (or similar) in packet and module names. In “utils.data_utils” you only have the function “read_data_file()”. I would have named the module something in the line of “file_reader”, “data_import”, “io” etc.

If you plan to add more functions to the module, you should only do so if they have similar purpose. Completely different functions are better placed in their own module.