Forループのjavaで何が間違っていますか？

Question

私はこの質問に対するいくつかの答えを探していましたが、私はそれを理解できないようです。問題は自分のコードの 'スコープ'にあるようだが、私は何が間違っているのか分からず、実際に助けてもらうことができる。私はJavaにとって非常に新しいです。オブジェクトはすでにそれを追加する前に、アレイ内に存在する場合

私の目標

1. をテストします。

2. オブジェクトがすでに配列に存在する場合はnullを返します。

3. アレイに存在しない場合は作成してから戻します。

コード

public Business addBusiness(String person, String business, String location) { 
    int id = 0; 

    for (Business business : businesses) { 
     if (business.getPerson().equals(person)) { 
      if (business.getBusiness().equals(business)) { 
       if (business.getLocation().equals(location)) { 
        return null; 
       } 
      } 
     } 
    } 
    Business newBusiness = new Business(person, business, location, id); 
    return newBusiness; 
} 

私はそれだけで幸せに同じ内容を持つオブジェクトの後にオブジェクトを作成するコードを実行すると、私は

それを実行するとどうなりますか。これは私の論理に問題があると私に信じさせる。私はそれがヌルを返したときに、それがそれの終わりになることを期待していました。

この問題を解決するためのアドバイスをいただければ幸いです。

EDIT：あなたはあなたがループ内で反復処理しているオブジェクトと関数のシグネチャからのパラメータbusinessをマスキングした

public Business addBusiness(String person, String business, String location) 
{ 
    int id = 0; 

    for (Business b : businesses) { 
     if (b.getPerson().equals(person)) { 
      if (b.getBusiness().equals(business)) { 
       if (b.getLocation().equals(location)) { 
        return null; 
       } 
      } 
     } 
    } 
    Business newBusiness = new Business(person, business, location, id); 
    return newBusiness; 
} 

「Businessのクラス

public class Business {  
// Attributes // 
private String person, business, location; 
private int id; 

// Constructor // 

public Business(String person, String business, String location, int id) { 
    this.person = person; 
    this.business = business; 
    this.location = location; 
    this.id = id; 
} 

// Getters // 

public String getPerson() { 
    return person; 
} 

public String getBusiness() { 
    return business; 
} 

public String getLocation() { 
    return location; 
} 

public int getId() { 
    return id; 
} 
} 

Answer 1

は試してみてください。これをbに変更すると、この問題は回避されます。

Answer 2

私はそれを理解したと思います。この件についてお手伝いいただき、ありがとうございます。

public Business addBusiness(String person, String business, String location) 
{ 
int id = 0; 

for (Business business : businesses) { 
    if (business.getPerson().equals(person)) { 
     if (business.getBusiness().equals(business)) { 
      if (business.getLocation().equals(location)) { 
       return null; 
      } 
     } 
    } 
} 
Business newBusiness = new Business(person, business, location, id); 
return newBusiness; 
} 

この問題は、実際にnewBusinessオブジェクトが作成された後に配列に実際に追加されたことが原因で発生しています。したがって、実行されるたびに、配列にエントリがないため、既存のエントリと一致することはありません。私はまだ学ぶことがたくさんある、明らかに

public Business addBusiness(String person, String business, String location) 
{ 
int id = 0; 

for (Business b : businesses) { 
    if (b.getPerson().equals(person)) { 
     if (b.getBusiness().equals(business)) { 
      if (b.getLocation().equals(location)) { 
       return null; 
      } 
     } 
    } 
} 
Business newBusiness = new Business(person, business, location, id); 
b.add(newBusiness); // This was the missing line, now it seems to work fine. 
return newBusiness; 
} 

：

正しいコードのようなものでなければなりません。