in reply to Efficiency on Code

The efficiency will depend on the SQL - specifically, how much you are SELECTing. Since you only use the statement to produce an array of account numbers, you should only select that column.

What is $TOALLCLAIM? It looks like a different name for $TOALL, which should be lexical, or else omitted entirely.

Assuming 'ACCOUNT#' is a legal column name in your db, and that your DBD supports the method, the whole thing might be written,

my @array_claim = map { cleaner($_) } @{$dbh->selectcol_arrayref( 'select ACCOUNT# from datable')};
You'll need something a little more elaborate to quote data to be entered in a where clause.

After Compline,
Zaxo

Replies are listed 'Best First'.
Re^2: Efficiency on Code
by Anonymous Monk on Jan 06, 2005 at 19:22 UTC
    Sorry there isn't $TOALLCLAIM, it's $TOALL