for で i を使うのを玄人っぽくないからって理由で否定してる人から滲み出るにわかっぽさ感じる

実際問題、一文字変数名にしても俺々略字にしても、それ自体は問題じゃない
なにが格納されててどう使われてるのかが、すぐに読み解ければローカル変数名なんて何でも構わん

重要なのは、後から前情報なしでコードを見て脳内で仕様に戻す作業がどれくらいやりやすいかってとこだろ

昨今の開発環境じゃ、メモリ確保とか開放なんかはきっちり意識する必要少ないし、
メソッド分けて呼びまくったりもパフォーマンスに大きな差なんて出ない

省略された変数名が把握しづらくてコードの見通しが悪いようなら、
まずは変数名を変更するよりスコープの狭いメソッドに処理を分割するほうが、リファクタリングとしては正解