0

for ループで配列からフィールドにアクセスする際に問題があります。私は構文を台無しにしているだけだと確信しています：

public class Person {

String  name;
Person  mother;
Person  father;
ArrayList<Person> children;

public boolean isMotherOf(Person pers1) {
    //First check if Persons mother equals pers1
    if (mother.name == pers1.name) {            
        //Second checks if pers1s children contains Person
        for (int i = 0; i < pers1.children.size(); i++) {
            if (pers1.children.name.get(i) == name) {
// ERROR ON THE LINE ABOVE: "name cannot be resolved or it not a field"
                return true;
            } 
        } // END second check
    } else { return false; }


    } // END method isMotherOf
}

編集：私のコードには論理エラーが含まれていました（間違った人を比較して）。この安全性チェックは機能しますか? または、pers1 の母親に名前があるかどうかをチェックするときに、pers1 が存在しない場合にエラーが発生しますか?

public class Person {

String  name;
Person  mother;
Person  father;
ArrayList<Person> children;

// Checks if THIS person is the mother of pers1
public boolean isMotherOf(Person pers1) {
    // Safety check: Both persons exists and Person has children and pers1 has mother 
    if (name == null || children == null || pers1 == null
        || pers1.mother.name == null) {
        return false;
    } // END safety check
    // First check if Persons name equals pers1's mother
    else if (name.equals(pers1.mother.name)) {          
        // Second check if Persons children contains pers1
        for (int i = 0; i < children.size(); i++) {
            if (children.get(i).name.equals(pers1.name)) {
                return true;
            } 
        } // END second check
    } // END first check 
    return false;
} // END method isMotherOf
} // END class Person
4

3 に答える 3

4

あなたは 2 つのものを交換したpers1.children.name.get(i)はずですpers1.children.get(i).name。

ここでは、文字列を と比較しています。==これは、オブジェクトの参照を比較し、equals(..)代わりに使用します。ここでチェックしてください。

于 2013-02-01T14:52:08.753 に答える
0

==関数の代わりにequals()文字列を比較しているようです。

この行を次のように変更してみてください。

if (pers1.children.name.get(i).equals(name))
于 2013-02-01T14:53:35.983 に答える
0

文字列の比較に「==」を使用しないでください。それはあなたが思うことをしていません。次のように記述します。

if (mother.name.equals(pers1.name)) { ...

もちろん、NullPointerExceptionどちらmotherかmother.nameが null の場合に s が返されるため、これは少し危険なアプローチです。そのため、少し防御的なプログラミングを行う必要があります。

if (mother == null) return false;
if (pers1 == null) return false;
if (mother.name != null && mother.name.equals(pers1.name)) ....
于 2013-02-01T14:56:20.223 に答える