何故過去の遺物のforを使う？
foreach使えば良いのに。