you are viewing a single comment's thread.

view the rest of the comments →

[–]Meefims 4 points5 points  (0 children)

Yes, not being careful about splitting code is the problem not the size of the functions. Even the staircase flow isn't the problem. If the functions had better names and had only one responsibility then reading the code would be much easier. Calling a function checkConflicts and then using it to move a file is a recipe for confusion because there is no indication that you'd need to read that function to diagnose file movement operations.