むしろこのコードを罵る人の方がアレだ

「世界一IQの低いソースコードはこれ」。これはこれでありだと思う。少なくともコードの意図は明確だし。どちらかと言うとリフレクションを使う方が筋が悪い。フィールド名と文字列の対応に同じルールが適用できること、他の用途に使われるフィールドが追加されないことをコードを書く人が保証できないからね。Javaのswitch/caseにも問題がないわけではないし、このコードよりも明らかに良いコードを書くって案外難しい気がする。Mapを使えば多少はコード量を抑えることができるかなぁという程度。現時点であれば、このコードが書かれたよりもあとに提供されたKeyEvent.keyCodeToStringを使えばすむ話らしいけどね。もっともこのAPIの実装も似たようなコードになってるんじゃないかなぁ。