この会社辞めようと思ったソースコード

別にローカル変数に短い名前を付けるのは全然かまわんと思うぞ。
つうか、普通そうするだろ。「シンプルさ」はわかりやすさにつながるんだ、って
だれか言ってたよ。グローバル変数は当然フルスペルにするがな。
関数ならせいぜい数十行だから、変数の定義みて忘れることはほぼないだろ。
それより、変数の宣言にコメントつける方が大事。

for (boxNumber = 0; boxNumber < maxNumberOfBox; boxNumber++) {
...
}

より、

for (i = 0; i < max; i++) {
...
}

の方がはるかにわかりやすい。