Perl: the Markov chain saw | |
PerlMonks |
comment on |
( [id://3333]=superdoc: print w/replies, xml ) | Need Help?? |
please do give the feedback......You should post code that someone can download and compile without errors. Since you are new to the Monastery, you should re-read Writeup Formatting Tips, then try to download the code yourself, and try to run it. It is full of "br" tags, and the code itself has syntax errors: Do not capitalize 'strict': use strict; $pkt was never assigned a value; you probably meant to use $packet. $data=~s/[^-A-Za-z0-9]/./g needs a semicolon at the end. Also, you could take advantage of regular expression CHARACTER CLASSES to make the code less 'noisy'. Your code would be easier on the eyes if you made better use of whitespace. Try running your code through perltidy. any suggestions (or abuse) would be gladly accepted at the same email idThe convention here at the Monastery is to provide constructive feedback as a reply to your node here, not via email. Update: use Strict; #i usually don't use itYou should always use strict and warnings. In reply to Re: my first packet sniffer in perl
by toolic
|
|