java - ファンクションのクラス

Question

私は大学の部に関数を書く必要があります：

追加機能は講師を追加します。 アクションは、追加の講師を追加する場所がない場合はfalseを返し、講師が追加された場合は同じことを返します。私がこれまでに書いた何

：

は

public boolean newLecturer(Lecturer[] AllLecturer) { 
    int MaxLecturer = 0; 
    MaxLecturer = this.maxLecturer; 
    int sum = 0; 
    sum += 1; 

    if (sum < MaxLecturer) { 
     System.out.println("true"); 
     return true; 
    } 

    else { 
     System.out.println("false"); 
     return false; 
    } 

} 

機能が正しく動作しませんが、それは常にtrueを返します（理由は合計よりも常に大きくなっている最大講師）。

メイン：

パブリッククラス

public static void main(String[]args){ 

Lecturer[] L1 = new Lecturer[]{new Lecturer("David",3,"Banana",1001)}; 
Lecturer[] L2 = new Lecturer[]{new Lecturer("Yossi",5,"apple",1002)}; 
Lecturer[] L3 = new Lecturer[]{new Lecturer("Y",2,"t",1003)}; 

    College myCollege = new College("College1",20,L1,3); 

    //System.out.println(myCollege); 
    //myCollege.allLecturer=L2; 
    //System.out.println(myCollege); 

    myCollege.newLecturer(L1); 
    myCollege.newLecturer(L2); 
    myCollege.newLecturer(L3); 
} 
} 

クラス・カレッジ（ここでは機能）{メイン：

public class College { 

public String name; 
public int numOfLecturer; 
public Lecturer[] allLecturer; 
public int maxLecturer; 

// constructor 
public College(String Name, int NumOfLecturer, Lecturer[] AllLecturer, 
     int MaxLecturer) { 
    this.name = Name; 
    this.numOfLecturer = NumOfLecturer; 
    this.allLecturer = AllLecturer; 
    this.maxLecturer = MaxLecturer; 
} 

public College(String Name) { 
    this.name = Name; 
} 

public College(Lecturer[] AllLecturer) { 
    this.allLecturer = AllLecturer; 
} 

public boolean newLecturer(Lecturer[] AllLecturer) { 
    int MaxLecturer = 0; 
    MaxLecturer = this.maxLecturer; 
    int sum = 0; 
    sum += 1; 

    if (sum < MaxLecturer) { 
     System.out.println("true"); 
     return true; 
    } 

    else { 
     System.out.println("false"); 
     return false; 
    } 

} 

@Override 
public String toString() { 
    String lecturers = ""; 

    for (Lecturer lecturer : allLecturer) { 
     lecturers += lecturer; 

    } 

    return "[Name College: " + name + "] " + " [num Of Lecturer: " 
      + numOfLecturer + "]" + " [all Lecturer: " + lecturers + "]" 
      + " [max Lecturer " + maxLecturer + "]"; 
} 
} 

クラス講師：

public class Lecturer { 
public String name; 
public int numOfTimesPenFalls; 
public String favoriteIceCream; 
public int autoNumber; 

// constructor 
public Lecturer(String Name, int NumOfTimesPenFalls, 
     String FavoriteIceCream, int AutoNumber) { 
    this.name = Name; 
    this.numOfTimesPenFalls = NumOfTimesPenFalls; 
    this.favoriteIceCream = FavoriteIceCream; 
    this.autoNumber = AutoNumber; 
} 

public Lecturer(String Name) { 
    this.name = Name; 

} 

@Override 
public String toString() { 
    return "[name: " + name + "] " + " [num Of Times Pen Falls: " 
      + numOfTimesPenFalls + "] " + " [favorite Ice Cream: " 
      + favoriteIceCream + "] " + " [auto Number: " + autoNumber 
      + "]"; 
} 
} 

そして最後にどのようにすることができますそれを印刷する？ このようにコンパイラエラーが発生します。

myCollege.newLecturer("David",2,"Apple",1004); 

ありがとうございます。

Answer 1

新しいものです。あなたは多くの助けが必要です。

Java coding standardsから学習してください。変数名は小文字で始まる必要があります。クラスはupperで始まります。それからの偏見はあなたのコードを読みにくいものにします。

あなたの方法は間違っています。あなたは、そのクラス内でこのような何かが必要です。

private static final int MAX_LECTURERS = 3; 

private int numLecturers = 0; 
private Lecturer [] lecturers = new Lecturer[MAX_LECTURERS]; 

public boolean addLecturer(Lecturer lecturer) { 
    boolean addedLecturer = false; 
    if (this.numLecturers < MAX_LECTURERS) { 
     this.lecturers[numLecturers++] = lecturer; 
     addedLecturer = true; 
    } 
    return addedLecturer; 
} 

をここでは、この方法を使用する方法は次のとおりです。

Lecturer newLecturer = new Lecturer("foo", 1, "bar", 3); 
college.addLecturer(newLecturer); 

全てその配列のナンセンスで停止してください。配列はCollegeクラス内にあります。

Answer 2

コード内のsum変数はローカル変数です。そのスコープは関数レベルにのみあります。つまり、関数newLecturer（）が呼び出されるたびに、合計は常に0に初期化され、1に増加します。そのため、合計は常にMAX_LECTURER（1 < 3）より小さくなります。

上記のduffymo答えのようなクラス変数numLecturersを使用する必要があります。