in reply to Re: multiple db connects
in thread multiple db connects

My first thought is that unless you are connecting to more than one DB, you don't need a routine. You leave the connection open until you are done. So any routine should handle some sort of argument for which DB to connect to.

I have a question of style/usage regarding the die clause in the subs... Would we still want to call such a function thusly: 
my $dbh->db_handle($db_name) || die "SNAFU getting to $db_name: $!";
or
my $dbh->db_handle($db_name) || &explain_to_user_that_db_unavailable( 
+$db_name );

[download]
Or something like that? Especially if this is a CGI wouldn't we want to use a function to tell the user to quit hitting refresh until the server is fixed? The die clause still returns a false value for the sub call, right? But this could cause errors if not handled in the main clause, right?

Replies are listed 'Best First'.
Re: Re: Re: multiple db connects
by btrott (Parson) on Dec 21, 2000 at 21:18 UTC
    Sure, you could do whatever you wanted, really. Personally I like to die when something seems like an obviously fatal error. That doesn't mean that your *program* will die, because you can catch the die in an eval block. Which is what I do.

    You also said: 

        > The die clause still returns
    > a false value for the sub call, right?

    [download]
    Well, no; the die just *dies*. Your sub won't return at all. Your program will die, unless you have an eval block, in which case the die will shoot you out to the closest eval block, and will populate $@ with the error message. And then you said: 
        > But this could cause
    > errors if not handled in the main clause, right?

    [download]
    Certainly. That's why the errors should be handled. :)

    Instead of dying, then, you could have your sub return a false value. That would work. Then you could do your checking out in the caller. Some people find that preferable, I'm sure, and that's fine; my preference is to throw exceptions as soon as something bad happens. Then I catch them.

[reply]
[d/l]
[select]
      So that's the behavior I'm trying to understand about die. If it is going to cause the program to bomb, then it seems like (especially for CGI) a much better idea to me (both in terms of user-friendly solutions and in terms of code re-usability) to build the function in such a way that it returns a false value if it fails. That way the function doesn't kill the whole process (i.e. HTTP result code 500) and it can be used in statements like 
      $foo = biff($bap) || 'Bar'; 

or

if ($dbh = handle($db)) {
    do_something($dbh);
} else {
    dont();
}

      [download]
      (for instance).
[reply]
[d/l]
[reply]

In Section Seekers of Perl Wisdom