質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.11%

Java 入力された数字をリストから削除されるようにしたい

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 569

snmn16

score 12

現在、キーボードで入力した数字を、予め用意されたリストから削除されるコードを記述しているのですが、記述しいざ実行しても入力されたものと違う数字が消えてしまいます。
どうやら要素番号?を削除しているのではないかと言うことはぼんやりわかったのですが、ここからどうするべきかが分かっておりません。

発生している問題・エラーメッセージ

【黒ひげ危機一髪ゲーム!】
下記から好きな数字を入力してください。
[1,2,3,4,5,6,7,8,9,10]
2
セーフ!

下記から好きな数字を入力してください。
[1, 2, 4, 5, 6, 7, 8, 9, 10]
3
セーフ!

下記から好きな数字を入力してください。
[1, 2, 4, 6, 7, 8, 9, 10]
4
【ドッカーン!!】

該当のソースコード

package kurohige;

import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Kurohige {

public static void main(String[] args) {

System.out.println("【黒ひげ危機一髪ゲーム!】");
System.out.println("下記から好きな数字を入力してください。");
System.out.println("[1,2,3,4,5,6,7,8,9,10]");
Scanner scan = new Scanner(System.in);
int inputValue = scan.nextInt();

List<Integer> list = new ArrayList<Integer>();

list.add(1);
list.add(2);
list.add(3);
list.add(4);
list.add(5);
list.add(6);
list.add(7);
list.add(8);
list.add(9);
list.add(10);

while(inputValue != 4) {
// ↑か

System.out.println("セーフ!");
System.out.println("");
System.out.println("下記から好きな数字を入力してください。");
list.remove(inputValue);
// ↑これ 多分これ

System.out.println(list);
inputValue = scan.nextInt();

}
scan.close();
System.out.println("【ドッカーン!!】");
}
}

試したこと

削除される場所がずれるなら、list.remove(inputValue-1)などと記述すればいいのではないかと思い試しましたが、当然と言えば当然ですがしっかり消えるのは1以外を入力した場合のみでその後はやはりずれて行きました。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • nskydiving

    2019/04/01 17:52

    コードがコードタグに囲まれていないようですので、修正してください。

    ```java
    ここにコードを書きます。
    ```

    キャンセル

回答 2

+1

メソッドが想定通りの働きをしてくれないときは、何より先にリファレンスを見ましょう。

E remove​(int index)
このリスト内の指定された位置にある要素を削除します(オプションの操作)。 

引用元: List (Java SE 10 & JDK 10 )

どうやら要素番号?を削除しているのではないか

この推測でビンゴなのですが、憶測ではなく確証を持ってコードを組んでいきたいものです。


解決策は主に二つあり、後者の方が直接的でお勧めです。
なお、削除したい要素が含まれていないときの挙動は双方で異なります。

  1. 削除したい要素のindexを取得して、それを利用する方法
    List#remove(int)を活かす考え方です。
    リファレンスをつらつらと見ると、List#indexOf​(Object)が使えそうなことが分かります。
  2. 削除したい要素を直接指定する方法
    List#remove(Object)を利用します。

どちらの方法を選ぶにしても、Integer型のオブジェクトを利用する必要があります。
intから同じ数値を表すIntegerを得たいときはInteger.valueOf(int)が有用です。

おまけ

class MyClass {
    void func(int arg) {
        System.out.println("MyClass#func(int): " + arg);
    }
    void func(Object arg) {
        System.out.println("MyClass#func(Object): " + arg);
    }
}

class Main {
    public static void main(String[] args) {
        int     num1 =  6;
        Integer num2 = 42;

        MyClass myClass = new MyClass();

        myClass.func(num1);                  // MyClass#func(int): 6
        myClass.func(num2);                  // MyClass#func(Object): 42
        myClass.func(Integer.valueOf(num1)); // MyClass#func(Object): 6
    }
}

Wandbox

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

0

list.remove(inputValue);
// ↑これ 多分これ

ここですね。

list の中身は [1,2,3,4,5,6,7,8,9,10] なので、最初の要素は「1」ですが、list の最初のインデックスは「0」なので、一つずれてしまいます。

[1,2,3,4,5,6,7,8,9,10] の最初(0 番目)の要素「1」を削除したい場合は、list.remove(0)とする必要があります。

最初は List のインデックスが 0 から始まることに違和感を感じるかもしれませんが、そういうものだと思ってください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.11%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる