質問: new MovieManager(match, match.getSeats(),match.getSetted()); mng.getEmptySeatShow(); // => 席番号1番は空いています // => 席番号2番は空いています // => 席番号3番は空いています //[ユーザ] 空き座席を選択 mng.setSeat(2); //[システム] 空き座席を表示 mng.getEmptySeatShow(); // => 席番号1番は空いています // => 席番号2番は埋まっています // => 席番号3番は空いています } }
回答: ・lengthはmovieLenghの方がわかりやすい (lengthという言葉自体Javaでは頻出するので、後々「どれの長さ(length)の変数だっけ?」となりやすい) ・席番号と埋まってる席番号の変数の型が違う (ほぼ同じ意味であれば、同じ型の方が混乱しなくて済む) 席番号と埋まってる席番号はMapあたりを使った方が楽な気がします。 (キーが席番号、値は埋まってるなら1、埋まってなければ0、みたいな感じ)