受けたアドバイス

ファイルを読み込んで、その内容をDBに更新するシステムを作っている。

処理の流れとしては、
1. パラメータを取得する
2. ファイルを開く
3. ファイルを読み込む
4. ファイル内容をチェックする
5. 更新する

実際においらが書いたコード。細かいところは違うけど。

# ファイルを開く
open (TEXT_FILE, "<", "test.txt")
	or die($!);

# ファイルを読み込む
while (<TEXT_FILE>) {
	push (@lines, $_);
}

# チェックをする
while (@lines) {
}

# ファイルを閉じる
close (TEXT_FILE);

# DBを更新する

先輩のアドバイス
・while は2回もいらない
　-> 1行読み込んで、その時にチェックもすれば良い。あんまり大きなデータファイルではないから大丈夫だとは思うが、これだとあまりよろしくない。

ということでこうした。

# ファイルを開く
open (TEXT_FILE, "<", "test.txt")
	or die($!);

# ファイルを読み込む
while (<TEXT_FILE>) {
	push (@lines, $_);
        # チェックをする
}

# ファイルを閉じる
close (TEXT_FILE);

# DBを更新する