This uses ikegami's approach; this will print the previous $chro as soon as a new '@' is found. You seem to want to print $sequence at the same time. This moves the print statements inside the loop. Also, you need to reset sequence after the new @ is found. If you would give us something more detailed than 'nnnn' we could see where things should go. Try using __DATA__ next time to hold some sample data for everyone to see.
use warnings; use strict; print "Using __DATA__ sample cns file:\n"; #chomp($t = <STDIN>); #open(THE, $t) or die "Can not open file: $!\n"; #@data = <THE>; my @data = <DATA>; #close THE; my $sequence=""; my $chro; foreach my $line (@data) { if ($line =~ /^\@/) { print $chro if length $chro; $chro = $line; print $sequence; $sequence = ""; } elsif ($line =~ /^[a-zA-Z]/) { $sequence .= $line; } else { print "no match: \$line=$line\n" } } #prints the last one, I'm sure theres a better way print $chro if length $chro; print $sequence,"\n"; __DATA__ @chr1 ACAAGATGCCATTGTCCCCCGGCCTCCTGCTGCTGCTGCTCTCCGGGGCCACGGCCACCGCTGCCCTGCC CCTGGAGGGTGGCCCCACCGGCCGAGACAGCGAGCATATGCAGGAAGCGGCAGGAATAAGGAAAAGCAGC @chr2 CTCCTGACTTTCCTCGCTTGGTGGTTTGAGTGGACCTCCCAGGCCAGTGCCGGGCCCCTCATAGGAGAGG AAGCTCGGGAGGTGGCCAGGCGGCAGGAAGGCGCACCCCCCCAGCAATCCGCGCGCCGGGACAGAATGCC @chr55 CTGCAGGAACTTCTTCTGGAAGACCTTCTCCTCCTGCAAATAAAACCTCACCCATGAATGCTCACGCAAG
Update: would be better to replace the foreach with while (my $line=<DATA>) and get rid of @data. Especially since the files will probably be large.
In reply to Re^3: foreach loop
by Gulliver
in thread foreach loop
by morio56
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |