Yes, that would work. Of course,
(state.attackTransferMoves).add(new AttackTransferMove(myName, fromRegion, toRegion, armies));
will only add one attack/transfer order to the list.
Also, you could declare attackTransferMoves private, with public getters and setters.
The BotStarter's getAttackTransferMoves method, then, only contains one line:
"return state.getAttackTransferMoves();"
Edited 2014-03-26 13:14:54