Как построить кривую массива и массива?

Question

Я создаю кривую потребителя и производителя, каждый из которых является собственным классом. У потребителя есть массив точек, а у производителя есть список массивов точек. Мне нужно создать метод, который создает «график» (consumerCurve и производительCurve). В этих методах необходимо проверить недопустимый параметр (т. Е. Отрицательные числа), создать экземпляр массива/списка массивов, а затем создать экземпляры/сохранить их в правильных слотах. Вот что у меня есть для consumerCurve:

public class ConsumerCurve { 

private Point[] myConsumerCurve; 

public void Curve() 
{ 
    myConsumerCurve = new Point[10]; 

    for(int x=0; x<10; x++) 
    { 
     myConsumerCurve[x] = new Point(x,x); 
    } 
} 

public ConsumerCurve(int np, double m, double b, int dx) 
{ 
    //*1)check for invalid parameters 
    Point[] myConsumerCurve = new Point [np]; 

    if (np < 0) { 
      throw new IllegalArgumentException("'np' must not be negative"); 
     } 

    //2) Instantiate array using size n 
    for(int i=0; i<np; i++) 
    { 
     int x = i*dx; 
     double y = m*x+b; 
     myConsumerCurve[i] = new Point (x,y); 
    } 

    //3*)instantiate points store in correct slots 

} 

У меня есть вторая часть, но я не знаю, как проверить на invaid PARAMATERS и получать очки в правом слоте. Кроме того, будет ли он одинаковым для списка массивов? Возможно, потому, что я слишком долго смотрел на этот код и просто пропустил его.

Answer 1

Несколько замечаний по коду:

Point[] myCurve = new Point [np]; // if np is negative here, you will get an exception 
if(np< 0) { 
    np= -1; 
} 

Здесь вы должны проверить npперед вы используете его в new Point[np], иначе вы получите исключение в случае отрицательного np. Установка np на -1 не решила проблему. Вы должны либо установить его в 0, или, предпочтительно, сгенерирует исключение, указывающее, что аргумент не является приемлемым:

if (np < 0) { 
    throw new IllegalArgumentException("'np' must not be negative"); 
} 

Аналогично для других аргументов, если это необходимо.

Если Point является java.awt.Point, то его конструктор Point(int, int) - вам придется забрасывать double Ань int: new Point(x, (int) y). Если Point - это ваш собственный класс, который принимает в конструкторе double, то все в порядке.

И, наконец, вместо хранения точек локальной переменной myCurve, почему вы не храните их непосредственно в myConsumerCurve, как вы это делаете в методе curve()?

myConsumerCurve = new Point [np]; 

for(int i=0; i<np; i++) { 
    int x = i*dx; 
    double y = m*x+b; 
    myConsumerCurve[i] = new Point (x, y); // or 'new Point (x, (int) y)' if it's java.awt.Point 
} 

На стороне записки, это соглашение, что методы начинаются с нижним регистром (т.е. curve(), не Curve()).