in reply to Re^18: How to store the output from foreach loop into variable/array without printing?
in thread How to store the output from foreach loop into variable/array without printing?
Can I ask if I could also take a look at how you get the average of the ratio values as well?
Sure, here it is both makeRatioAvg and makeFinal
my $avgRatio = makeRatioAvg ( $ratio_ref ); my $final = makeFinal( $avgRatio ); dd( { avgRatio => $avgRatio }, { final => $final } ); sub makeRatioAvg { print "makeRatioAvg \n"; my( $ratio ) = @_; my $rows = $#$ratio; my $columns = $#{ $ratio->[0] }; my $half_rows = $rows / 2; my $halff = int( 1 + $half_rows ); ## eew my @avg; for my $tre ( 0 .. $columns ) { for my $tri ( 0 .. $columns ) { my $avg = 0; for my $six ( 0 .. $half_rows ) { my $val = $ratio->[$six][$tri][$tre]; $avg += $val; print " + $val "; } $avg = $avg / $halff; print " == $avg \n"; push @avg, $avg; } } for my $tre ( 0 .. $columns ) { for my $tri ( 0 .. $columns ) { my $avg = 0; for my $six ( $halff .. $rows ) { my $val = $ratio->[$six][$tri][$tre]; $avg += $val; print " + $val "; } #~ $avg = $avg / 3; $avg = $avg / $halff; print " == $avg \n"; push @avg, $avg; } } return \@avg; } ## end sub makeRatioAvg sub makeFinal { my( $avgRatio ) = @_; my @first = @{$avgRatio}[ 0 .. ($#$avgRatio/2) ]; my @second = @{$avgRatio}[ (1+($#$avgRatio/2) ) .. $#$avgRatio ]; my @final = map { [ $first[$_] ] , [ $second[$_] ] } 0 .. $#first; return \@final; }
As you might guess, makeRatioAvg and makeFinal is still me following closely the pencil and paper moves -- didn't want to get lost :) Now I have reference program with reference test data, so if I get ambitious and try to combine makeRatioAvg+makeFinal I will notice easily when I make mistake :)
Another question, when splitting the data (by tab since its tab delimited file), it will be split according to columns right (downwards). Is it possible to split by rows (horizontal)? Will that be easier for this kind of ratio thing or it doesn't make a difference?
I'm not quite sure what you mean, but I doubt it makes much of a difference, here is why
rows are lines ... each lines consists of tab seperated values (columns of a row) , so you're already doing what is possible in the most straight forward way
the only issue I saw with your code is you had "while while" loop, where the outer while was used just for the header ... I wrote that as
sub spankTheMonkey { my( $infh ) = @_; my @full_data; my $header = readline $infh; if( $header =~ /\d+/ ) { ## in case there is no header push @full_data, [ split ' ', $header ]; } push @full_data, [split] while( <$infh> ); return \@full_data; } ## end sub spankTheMonkey
It was called from DoWorkOriginal
sub DoWorkOriginal { my( $firstinput, $secondinput, $inFile, $outFile ) = @_; use autodie qw/ open close /; open my $curInfile, '<:raw', $inFile; ## or die by autodie open my $OUT1, '>:raw', $outFile; ## or die by autodie my $full_data = spankTheMonkey( $curInfile ); monkeyBusiness( $firstinput, $secondinput, $OUT1, $full_data ); close $curInfile; close $OUT1; } ## end sub DoWorkOriginal
Where sub monkeyBusiness is the foreach loops from your code which I didn't understand
The general strategy when I don't understand why/how something works, before I can begin to debug/improve, is to isolate/sequester everything into subroutines, and make sure it produces same output as original
|
|---|
| Replies are listed 'Best First'. | |
|---|---|
|
Re^20: How to store the output from foreach loop into variable/array without printing?
by hellohello1 (Sexton) on Mar 21, 2014 at 08:38 UTC | |
by Anonymous Monk on Mar 21, 2014 at 08:54 UTC | |
by hellohello1 (Sexton) on Mar 24, 2014 at 01:41 UTC | |
by hellohello1 (Sexton) on Mar 25, 2014 at 09:22 UTC | |
by Anonymous Monk on Mar 25, 2014 at 15:28 UTC |