You need more subs, less global variables, subs that take arguments and return values, subs with meaningful names, with no numbers in the name

When you write programs like this they're easy to read and "loop issues" solve themselves 

while( $row  = $sth->fetchrow_hashref) {
  my($con, $job, $pic) = GetContactJobPic( $dbh, $row->{jobtype} );
  MailNotifyConJobPic( $row->{email}, $con, $job, $pic, ... );
}
...
sub GetContactJobPic {
  my( $dbh, $jobtype ) = @_;
  my $hashref = $dbh->selectrow_hashref(
    q{Select * from database where jobtype=? AND active='yes' ORDER BY
+ RAND() LIMIT 1}m
    {},
    $jobtype
  );
  return @{$hashref}{qw' contact jobnum picture'};
}
sub MailNotifyConJobPic {
 my( $mail, $con, $job, $pic, ... ) = @_;
 ...  ## Mail::Sendmail or ...
}

[download]

Also placeholders are always a good idea bobby-tables.com: A guide to preventing SQL injection in Perl

Also because that sendmail stuff is still part of the code you posted, others who read this node should also heed caution

In reply to Re: More loop issues (more well named subs that take args return values, less global vars, placeholders) by Anonymous Monk
in thread More loop issues by htmanning

Title:
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, details, 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, summary, 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.