>>126
・変数を全て関数の先頭で宣言している。
・sprintfを使用している。(しかも<cstdio>のインクルードがない)
・fileはスコープを抜ければ自動的にcloseするのだから不要。
・str[200]なる決め打ちマジックナンバーが存在している。
以上について、個人的にはあまり好ましくないと感じる。あくまで、個人的に。