Next

Program Repair Shop

21

Comments

     115        @FileList1 = readdir (FILELISTDIR1);

     118        @FileList2 = readdir (FILELISTDIR2);

• Every implicit relationship is one more thing the maintainer must remember

• You could put in a comment to explain

• A comment should be the explanation of last resort

• It is better when things explain themselves:

             push @FileList, map "$dir/$_", readdir FILELISTDIR;

Next

Copyright © 2002 M. J. Dominus