Beefy Boxes and Bandwidth Generously Provided by pair Networks
Perl Monk, Perl Meditation
 
PerlMonks  

Re^2: perl script to compare two directories

by haukex (Chancellor)
on Jul 13, 2019 at 09:38 UTC ( #11102783=note: print w/replies, xml ) Need Help??


in reply to Re: perl script to compare two directories
in thread perl script to compare two directories

I don't have enough time to go through all of the code right now, but I do see quite a few reinvented wheels, as has been discussed before, for example, I've pointed you to File::Spec before. Reinventing wheels gives you more code to maintain, more code to test, and more chances to introduce bugs. For example, toNicePath is incorrect: a backslash \ is a valid character in *NIX filenames. Getting portability right is tricky, which is why modules like File::Spec exist in the first place.

I strongly recommend you take the time to study what modules are available to you, especially what core modules, as those are (almost) always installed.

Replies are listed 'Best First'.
Re^3: perl script to compare two directories
by afoken (Canon) on Jul 14, 2019 at 00:46 UTC
    I've pointed you to File::Spec before. Reinventing wheels gives you more code to maintain, more code to test, and more chances to introduce bugs.

    I fully agree to use modules instead of re-inventing the wheel, poorly.

    BUT: File::Spec is one of the modules that have a conceptual error, probably since the first version. See:

    Even with that error, it is better than re-inventing the wheel.

    Alexander

    --
    Today I will gladly share my knowledge and experience, for there are no sweeter words than "I told you so". ;-)

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://11102783]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others rifling through the Monastery: (5)
As of 2019-09-17 06:16 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    The room is dark, and your next move is ...












    Results (200 votes). Check out past polls.

    Notices?