人の管理システムの変更のやり方について

Question

名前(name)、性別(sex)、年齢(age)の値一覧を持っているリストhumanlistを作成します。

Register:R
Exit:E

と表示してRかEを入力します。
Rが入力されたら名前(name)、性別(sex)、年齢(age)を入力してhumanlistのname、sex、ageにそれぞれ代入します。登録を何回か繰り返した後

Register:R
Change:C
Exit:E

との表示にCを入力したとき、

name?:
age:

と表示するので名前と年齢を入力します。このとき入力された名前がhumanlistのある名前に一致したところの年齢を変えるというプログラムを作成したいのですが、リスト構造の仕組みがよく理解できておらず、strがCのところの処理がわからずに変なコードになってしまいうまく動かすことができませんでした。どのようにすればよいでしょうか？

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

class hu{
    public static void main(String[] args){
    Scanner scan = new Scanner(System.in);
        
    ArrayList humanlist = new ArrayList<>();

    System.out.println("Register:R");
    System.out.println("Exit:E");

    String str = scan.next();


    while(!(str.equals("E"))){
    if(str.equals("R")){
    System.out.println("name?");
    String name = scan.next();

    System.out.println("sex?");
    String sex= scan.next();

    System.out.println("age?");
    int age = scan.nextInt();

    humanlist.add(
    new human(name,sex,age)
    );
    System.out.println("Register:R");
    System.out.println("Chabge:C");
    System.out.println("Exit:E");

    str = scan.next();
    }

    if(str.equals("C")){
        System.out.print("name?:");
        String name = scan.next();
        System.out.print("age:");
        int age = scan.nextInt();

        for(int i=0;i<humanlist.size();i++){
            if(humanlist.get(i).name.equals(name)){
                humanlist.get(i).set(age,age);
            }
    }
    }
}
}
}

class human{
    final String name;
    final String sex;
    int age=0;

    human(String name,String sex,int age){
    this.name=name;
    this.sex=sex;
    this.age=age;
    
}
}

Comments

[退会済みユーザー]

コンパイルエラーとなるコードをまず直しましょう

[wafflek]

すいません。コンパイルエラーとなるのはわかるのですがどのように直したらよいかわからずに手が止まってしまったもので直そうとしたのですができませんでした。

Answer 1

ArrayListについては以下のように宣言します。

arraylistの書き方

ArrayList<型> リスト名 = new ArrayList<型>();

(引用先: https://techacademy.jp/magazine/18607)

ただ、こちらの書き方の方が良いようで、詳しくは引用先でご確認ください。

ArrayList<String> values = new ArrayList<String>();

これは書いては行けないコードで，正しくは
List<String> values = new ArrayList<String>();
と書くべき

(引用先: https://qiita.com/Mura-Mi/items/e52c28ab7cb5db140d53)

以上のことから
今回Listで管理しようとしているクラスはHumanクラスかと思います(Javaではクラス名の先頭文字を大文字にするように言われるので、humanからHumanにしました)ので、型はHumanで、

List<Human> humanList = new ArrayList<Human>();

と書けるかと思います。
また、java7以降であれば右のジェネリクス内の型は省略可能なので、

List<Human> humanList = new ArrayList<>();

と書けます。

質問者さんのコードを尊重したもの

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

public class Main {
    
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        List<Human> humanList = new ArrayList<>();

        System.out.println("Register:R");
        System.out.println("Exit:E");

        String str = sc.next();

        while(!(str.equals("E"))) {
            if(str.equals("R")){
                System.out.println("name?");
                String name = sc.next();
                System.out.println("sex?");
                String sex = sc.next();
                System.out.println("age?");
                int age = sc.nextInt();
                humanList.add(new Human(name, sex, age));
                System.out.println("Register:R");
                System.out.println("Change:C");
                System.out.println("Exit:E");
            } else if(str.equals("C")) {
                System.out.print("name?:");
                String name = sc.next();
                System.out.print("age:");
                int age = sc.nextInt();
                for(int i = 0; i < humanList.size(); i++) {
                    if(humanList.get(i).name.equals(name)) {
                        humanList.get(i).age = age;
                    }
                }
            }
            str = sc.next();
        }

    }

}

class Human {

    final String name;
    final String sex;
    int age = 0;

    Human(String name, String sex, int age) {
        this.name = name;
        this.sex = sex;
        this.age = age;
    }

}

自分なりに作ったもの（これが良いというわけでもないですが）

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

public class Main {
    
    public static void main(String[] args) {
        Scanner sc = new Scanner(System.in);
        List<Human> humanList = new ArrayList<>();

        System.out.println("Register:R");
        System.out.println("Exit:E");

        String str = sc.next();
        String name, sex;
        int age;

        while(!(str.equals("E"))) {
            if(str.equals("R")){
                System.out.println("name?");
                name = sc.next();
                System.out.println("sex?");
                sex = sc.next();
                System.out.println("age?");
                age = sc.nextInt();
                humanList.add(new Human(name, sex, age));
                System.out.println("Register:R");
                System.out.println("Change:C");
                System.out.println("Exit:E");
            } else if(str.equals("C")) {
                System.out.print("name?:");
                name = sc.next();
                System.out.print("age:");
                age = sc.nextInt();
                for(int i = 0; i < humanList.size(); i++) {
                    if(humanList.get(i).isEqualsName(name)) {
                        humanList.get(i).setAge(age);
                    }
                }
            }
            str = sc.next();
        }

    }

}

class Human {

    private final String name;
    private final String sex;
    private int age = 0;

    Human(String name, String sex, int age) {
        this.name = name;
        this.sex = sex;
        this.age = age;
    }

    public void setAge(int oldAge) {
        age = oldAge;
    }

    public int getAge() {
        return age;
    }

    public boolean isEqualsName(String comparisonName) {
        return name.equals(comparisonName);
    }

}

あと、<code>タグで囲むのは良いのですが、上のようにインデントすることも大切です。

Comments

[wafflek]

ご回答ありがとうございます。
細かい書き方まで教えてくださりとても勉強になりました。
また問題も解決できました。
本当にありがとうございます。