Java KeyListener を使ったキー入力の処理について

// interfaceを一つ作る。
interface Proc {
    public void doProc();
}

// キーごとに処理をmapに登録していく
Map<Integer, Proc> procMap = new HashMap<>();
procMap.put(KeyEvent.VK_UP, new Proc(){
    public void doProc() {
        // やりたい処理を書く
    }
});
procMap.put(KeyEvent.VK_DOWN, new Proc(){
    public void doProc() {
        // やりたい処理を書く
    }
});
// 以下たくさん
// ちなみにJava8ならラムダ式を使えばこれで済む
procMap.put(KeyEvent.VK_LEFT, () -> {
    // やりたい処理を書く
});

// _update内
for(Integer key : key_pressed) {
    procMap.get(key).doProc();
}

このように処理をinterfaceの無名クラス実装で書くことにより、スマートにfor文でまとめることができます。

注意

このやり方だと処理される順がキーを押された順になるので、質問の中でやっているspeedの調整等はすこし工夫が必要になると思います。

同時押しを処理する場合、質問者さんのコードにあるように現在押されている状態のキーを覚えておくわけですが、その覚え方の工夫は可能かも知れません。

キーを一つ一つリストの要素とすると同時押しのキー数がそれほど多くなることはないので効率が問題になるわけではないですが、一つ一つのキーが押されたかどうかの判定になります。それをなんとかしたいならListをやめてキーに対応したビットマスクのようなもので考える方法もあるかと思います。

興味のあるキーの集合(K1, K2, ...)に対してそれぞれビット位置を決めておき、KEY_PRESSEDでint型のmaskの対応ビットをセットしてKEY_RELEASEDで対応ビットをクリアすればよいわけです。

しかし・・・例に挙げておられるキー処理だとせっかくビットマスクにしてもキー判定（下記のdoSomethingByKeyStatus)はスッキリしますが別の情報の初期化なんかのコードは増えます。どちらかというと「スッキリさせる」というより若干効率がよくなる気がする自己満足的なコードと言えるかもしれません。下記のコードはイメージを掴む目的でいろいろ省略してますが意図は伝わると思います。

class なんかのクラス implements KeyListener {
  int keyMask;
  int[] horizontalMove = { .... };
  int[] verticalMove = { .... };

  @Override public void keyPressed(KeyEvent ev) {
    Integer bit = keyCodeToBitPositionMap.get(ev.getCode());
    if (bit != null) {
      keyMask |= 1 << bit;
    }
  }

  @Override public void keyReleased(KeyEvent ev) { 省略 }

  void doSomethingByKeyStatus() {
    speed = (keyMask & SHIFT) == 0 ? 1 : 2;
    viewport_x += speed * horizontalMove[keyMask & (LEFT|RIGHT)];
    viewport_y += speed * verticalMove[keyMask & (UP|DOWN)];
    ... etc.
  }
}