配列@arrayからファイル名を読み取り、その中身を順次解析し、すべて解析し終わると@arrayの先頭に戻り同じ動作を行うコードなのですが、やたらとネストが増えすぎてしまい困っています。一応、サブルーチンを複数作っても見たのですがどうもスッキリしませんでした。
もう少し良いコードに改良、ご指導して頂けないでしょうか?
$| = 1;
my @array = ("1.txt","2.txt","3.txt");
for(;;){
for my $line (@array){
open(F,"$line");
my @f = <F>;
close F;
for my $line2 (@f){
if($line2 =~ m/test/) {
print $line2;
}else{
print "err\n";
}
}
sleep 1;
}
}
因みに、上記が最終ではなく、最終的には下記のようにしたい(@array部分を外部list.txtにする)のですがネストが増えてしましまい自分でも何を書いているのかよく分からなくなってしまいます。
$| = 1;
for(;;){
my $file = "list.txt";
open(F,"$file");
my @array = <F>;
close F;
chomp @array;
for my $line (@array){
open(F,"$line");
my @f = <F>;
close F;
for my $line2 (@f){
if($line2 =~ m/test/) {
print $line2;
}else{
print "err\n";
}
}
sleep 1;
}
}
__END__
----list.txt-----
1.txt
2.txt
3.txt
a.txt
b.txt
c.txt
-----------------
---1.txt---
test
text
taxt
test1
ttet
-----------
A 回答 (3件)
- 最新から表示
- 回答順に表示
No.3
- 回答日時:
私も、自分用に捨てプログラム作るときは、コメントは少ないし、変数も$a,$b,$c...みたいな感じになります。
後から使い回そうとしたとき、perltidyにはよくお世話になります。
perltidyは結構高機能です。
例えば、 -csc オプションを付けると、ループ終わりの } に、その対応する { がわかるようなコメントが追加されます。
今回の場合は「ネストしてループ構造がよくわからない」ということですが、このコメントで大分わかりやすくなります。
他にもいろんなオプションがあるので、試してはどうでしょうか?
例) perltidy -csc -ce -bbb -lbl=1
$| = 1;
for ( ; ; ) {
my $file = "list.txt";
open( F, "$file" );
my @array = <F>;
close F;
chomp @array;
for my $line (@array) {
open( F, "$line" );
my @f = <F>;
close F;
for my $line2 (@f) {
if ( $line2 =~ m/test/ ) {
print $line2;
} else {
print "err\n";
}
} ## end for my $line2 (@f)
sleep 1;
} ## end for my $line (@array)
} ## end for ( ; ; )
No.2
- 回答日時:
現状でもそんなに複雑ではないので、あとは、名前の付け方とか改行とかコメントとかじゃないですかね。
$| = 1;
for(;;){
#listファイルから対象ファイル一覧を取得する
my $listfilename = "list.txt";
open(FP0,'<', $listfilename );
my @filelist = <FP0>;
close FP0;
chomp @filelist;
#全ファイルを処理する
for my $filename(@filelist){
#指定したファイルを読み込む
open(FP,'<',$filename);
my @lines = <FP>;
close FP;
#読み込んだファイルの中を確認する
for my $line (@lines){
if($line =~ m/test/) {
print $line;
}else{
print "err\n";
}
}
sleep 1;
} ## for my $filename(@filelist)
} ## for(;;)
ありがとうございます。自分用に作った場合、コメントアウト無しで作ることが多いです。それでも見直す事もあり、コメントアウトがあると非常に分かりやすくいいですね。
No.1
- 回答日時:
インデントがきちんとしていれば、今のままでも、十分わかりやすいとは思います。
このgooのサイトはインデントが投稿時、崩れるので、元のがきちんとしている前提ですが・・・
とりあえず、サブルーチンを呼び出す形式に変えてみました。
現行のソースより行数は、多少増えています。
-----------------------------------
$| = 1;
for(;;){
sub1("list.txt")
}
sub sub1{
open(F1,$_[0]) or die "open error [$_[0]] $!";
while(<F1>){
chomp $_;
sub2($_);
sleep 1;
}
close F1;
}
sub sub2{
open(F2,$_[0]) or die "open error [$_[0]] $!";
while(<F2>){
if ($_ =~ m/test/){
print $_;
}else{
print "err\n";
}
}
close F2;
}
-----------------------------------------------------
ありがとうございます。
見やすくなりましたね。また、openハンドルをそのまま読ませる方がスッキリして良いですね。
インデントについては普段perltidyで整形しています。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- Ruby パイソンでテキストファイルが読み込めない 1 2022/11/14 16:42
- Excel(エクセル) Excelにて、フォルダ内のTextファイルをマクロで統合すると文字化けしてしまう時の解消コード 4 2023/01/01 07:32
- Excel(エクセル) マクロでテキストファイルを読み込んだ際の最終セルにデータと改行が含まれる問題の改善方法 2 2022/03/25 16:50
- Perl perl このテキストファイルを簡単に配列に入れるには? 2 2022/04/27 20:24
- その他(プログラミング・Web制作) atcoder python コードへの助言 2 2022/08/12 15:31
- CGI perlで書いたcgiでsqliteの使い方を教えてください 2 2023/05/08 21:29
- Visual Basic(VBA) 型が一致しませんとエラー 6 2023/07/06 20:14
- その他(プログラミング・Web制作) listへのappendが出来ない件 1 2022/12/06 21:44
- Visual Basic(VBA) エクセルのマクロについて教えてください。 2 2023/07/15 15:12
- その他(プログラミング・Web制作) python 3.10で 同じlistに同じ構文で同じデータ代入した結果が異なる現象発生 7 2022/06/18 11:08
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
Perl エラーログを指定の場所...
-
「パスが見つかりません」とい...
-
ファイルの一部を別のファイル...
-
VBAでCSVファイルを途中行まで...
-
openした後、closeしないでプロ...
-
ファイル出力をUTF8Nではなくて...
-
C言語でのファイルのデータ更...
-
Perlで空白行を削除
-
batファイルでrenameができませ...
-
バッチファイルの作り方(CSV→...
-
perlにて2つのファイル比較
-
ファイルからある文字列の個数...
-
無料配布の郵便番号自動入力cgi...
-
fgets で値が取得できない
-
Perlで フォルダ内の全て...
-
エクセルVBA コードが同じでも...
-
perl上からfindを使い検索した...
-
awkスクリプトでダブルクォーテ...
-
Perl 特定のフォルダ以外削除
-
文字列をカウントする方法
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
ファイル名を複数個配列で確保...
-
Fortranで1行飛ばして読み込む方法
-
perlで指定範囲を複数ファイル...
-
Pythonでegrep機能をつかいたい
-
「パスが見つかりません」とい...
-
ifstream/ofstream について
-
ifstream を利用した1行分のテ...
-
C++でのテキストファイル読み込...
-
テキストファイルの各行を配列...
-
Perlのワンライナーをスクリプ...
-
perlプログラム 外部複数ファ...
-
Perl 重複カウント 上位3名
-
system関数と引数について
-
shellのコマンド deffの差分の...
-
Pythonで非日本語のUnicode文字...
-
perlで先頭の数値をみて昇順に...
-
while文がうまく動かない
-
Visual Basicを使って三平方の...
-
Perlによるディレクトリ内の連...
-
Perl Grepについて
おすすめ情報