in reply to Re^3: powerpoint problem
in thread powerpoint problem

I find that with a long statement like that, the modifier gets lost. A casual peruser might not notice there's a loop there at all. It's a little better if the modifier gets its own line like so:

$slide = $ppt->Slides->Add($_, ppLayoutBlank) for 1..4;

I like temporary variables too. Usually when $_ appears, it's a comprehension vortex. In this case, the variable ($i) is not much better, but sometimes a good name on a variable can add a lot of documentation. For example,

foreach my $good_guy ( @ARGV ) { push @my_attr, get_attributes( $good_guy ); }