配列@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 エラーログを指定の場所...
-
「パスが見つかりません」とい...
-
どなたか教えて下さい。
-
window.open でのファイル指定方法
-
VBAでCSVファイルの特定行を書...
-
ReadLineでの読み出し行を指定する
-
awkスクリプトでダブルクォーテ...
-
VBAで巨大なファイルの途中から...
-
ExcelをCSV書き出す場合のシー...
-
VBAでCSVファイルを途中行まで...
-
htaccessで特定のディレクトリ...
-
fopenでディレクトリ内の全ファ...
-
ファイル出力の改行コードをLFに
-
エクセルVBAで素数だけを出力す...
-
fgets で値が取得できない
-
エクセルVBA コードが同じでも...
-
配列の中に重複文字列があるか...
-
DOSコマンドで、標準出力を出力...
-
MATLABのm-fileについて
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
ファイル名を複数個配列で確保...
-
Pythonでegrep機能をつかいたい
-
Fortranで1行飛ばして読み込む方法
-
テキストファイルの各行を配列...
-
C++でのテキストファイル読み込...
-
C言語初心者です。プログラミン...
-
ファイルの内容をスカラー変数...
-
shellのコマンド deffの差分の...
-
<>で区切られたdatファイルをエ...
-
ifstream/ofstream について
-
perlの引数で複数のファイル処...
-
Pythonでテキストを行数指定し...
-
Perl エラーログを指定の場所...
-
perlでCSVをソートする方法につ...
-
「パスが見つかりません」とい...
-
ファイル名に日付を入れてアッ...
-
文字列操作に関する質問です。
-
連番のファイルを何個も開きたい
-
perlでIEのクッキーを削除したい
-
MATLABでカーブフィットしたデ...
おすすめ情報