セッター/ゲッターとのgetName（）とアプリケーションエラー

Question

マイcustomer.datファイル次のように次のようになります。最初の文字列が名前である一方、

4 
Jon 1000 
Jane 2000 
Jake 20000 
Drake 100000 

最初の数字は、そのファイルの顧客の合計数を表し後者は彼らが持っているバランスです。

次のコードは、私が書いたアプリケーションコードです。私はcustomer.datから読み込み、配列を作成します。アプリケーションの問題は、顧客IDを作成し、ユーザーが入力した名前（配列に存在する場合は顧客IDを設定する場合）を確認することですが、アプリケーションを実行するときは、私はドレイクを入力した場合でも - とポートフォリオを表示するには、「1」を入力し、それは常にジョンである最初の名前を示す終わる

Scanner infile2 = new Scanner(new File("customer.dat")); 

    customer[]mycustomers; // declaring an array to stock 

    int numofcustomers = infile2.nextInt(); 
    mycustomers = new customer [numofcustomers]; 

    for (int i = 0; i < numofcustomers; i++) 
    { 
     mycustomers[i] = new customer(infile2.next(),infile2.nextDouble(), numofcustomers); 
     System.out.println(mycustomers[i].toString()); 
    } 
    Scanner input = new Scanner (System.in); 
    System.out.println("Please type in your name"); 
    String Name = input.nextLine(); 
    int cusID = 0; 
    for (int i = 0; i < numofcustomers; i++) 
    { 
     if (Name == mycustomers[i].getName()) 
     { 
      cusID = i; 

     } 
    } 
    System.out.println("Please type in 1 for a customer to view their portfolio, 2 to trade stocks or 3 to exit the application"); 
    int choice = Integer.parseInt(input.nextLine()); 

    if (choice == 1) 
    { 
      System.out.println(mycustomers[cusID].toString()); 

    } 

次は私のcustomer.javaに位置しているものの抜粋です。

public customer(String n, double b, int Size) 
    { 
     Name = n; 
     balance = b; 
     stock[] A = new stock[Size]; 
     Portfolio = A; 
    } 

    public String getName() { 
     return Name; 
    } 

    public void setName(String name) { 
     Name = name; 
    } 

私は私が正しく問題を記述することができたかどうかわからないんだけど、いない場合は、任意のより多くの情報が必要な場合は私が知っている：セッターとゲッターが含まれています。

Answer 1

最初に、Javaの命名規則がCamelCaseのクラス名（つまり、顧客の代わりに顧客に名前を変更）。

私はあなたの問題はここにあると考えています。Javaで

Name == mycustomers[i].getName() 

、文字列はオブジェクトであり、あなたはそうのように、比較を行うためのequals（）メソッドを使用する必要があります。

Name.equals(mycustomers[i].getName()) 

標準==を使用すると、比較はオブジェクトのメモリ位置で行われるため、NameとgetName（）が文字どおり完全に同じStringオブジェクト（そうでない場合）でなければfalseになります。つまり、if文がtrueに評価されることはありません.cusIDは初期化しても0のままであるため、リストの最初の名前Jonを出力します。

上記の変更に加えて、cusIDを-1に初期化して、リストにない名前とリストの最初の要素の選択を区別できるようにすることをお勧めします。また、同じ名前の2人がリストに含まれている場合はどうなりますか？ （ヒント：現在の実装では最初のものは決して選択されません）