You're assigning a new list to the hash each time and overwriting whatever is there before.

my %_attack;
foreach (...) {
  $attack{$id} = ...;
}
[download]