Beefy Boxes and Bandwidth Generously Provided by pair Networks
good chemistry is complicated,
and a little bit messy -LW

Comment on

( #3333=superdoc: print w/replies, xml ) Need Help??
I have taken your computer feedback to heart and cleaned up my code a little bit - I think this a lot more perl like and pretty technical.

Indeed this does look more like Perl, but code must not be "perl like", it must have valid Perl syntax... well, if you want it to be valid Perl, and the program to work, that is. Also, it doesn't seem "pretty technical". It seems pretty basic, errors apart.

#!/usr/bin/perl -w use strict;

It's very very good that you're under strictures and warnings, but with recent enough perls it's better (i.e. more flexible) to

use warnings; # rather than -w
system "netbackup/bin/admincmd/bpdbjobs -report -all_columns\n";

You don't need that \n. Nay, I would have thought it to issue an error, but I checked and that's not the case. Whatever, it's still confusing at best. Also, it's better to use the multiple args form of system.

mail "

There's no mail() Perl builtin:

campari:~ [18:37:59]$ perldoc -f mail No documentation for perl function `mail' found

See? And even if there were, you have an incomplete statement. The above should read:

mail "xxxxxx\";

Notice that I quoted the @ sign, for otherwise perl would interpret that as if you wanted to interpolate the @xxxxx array into your string, because yes: double quoted strings interpolate (scalars and arrays). Else, you would use a single quoted string, like thus:

mail '';
but still I get:
Bareword "com" not allowed while "strict subs" in use at ./ line +7.<br> Unquoted string "com" may clash with future reserved word at ./ l +ine 7.<br> Execution of ./ aborted due to compilation errors. <br>

I don't trust you! That cannot be your actual program. When I run perl -c on it I get

String found where operator expected at line 7, at end of line (Missing semicolon on previous line?) Can't find string terminator '"' anywhere before EOF at line 7 +.

instead, which makes much more sense as an error message.

BTW: what's with those <br>'s anyway?

In reply to Re: Okay I have cleaned it up a little bit by blazar
in thread Okay I have cleaned it up a little bit by Kevin_Raymer

Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post; it's "PerlMonks-approved HTML":

  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.
  • Log In?

    What's my password?
    Create A New User
    and all is quiet...

    How do I use this? | Other CB clients
    Other Users?
    Others chanting in the Monastery: (4)
    As of 2017-11-24 04:16 GMT
    Find Nodes?
      Voting Booth?
      In order to be able to say "I know Perl", you must have:

      Results (344 votes). Check out past polls.