Что не так с кодом Java?

Question

[Saika2020]

У меня задание:
1. Создайте форму класса
2. Создайте массив точек
3. Создайте метод, который добавляет точку в массив точек (addPoint)
4. Создайте метод, который вычисляет периметр фигуры
5. Протестируйте программу

Я написал вот так:

import java.util.ArrayList;

public class Main
{
	public static void main(String[] args) {
		arrayList.addPoint(5.0,5.0,5.0);
		console.log(arrayList.perShape());
	}
}

class Shape {
    ArrayList<Double> arrayList = new ArrayList<>();
    
    public void addPoint(double x, double y, double z) {
        arrayList.add(x);
        arrayList.add(y);
        arrayList.add(z);
    }
    
    public double perShape() {
        double per = arrayList.get(0) + arrayList.get(1) + arrayList.get(2);
        return per;
    }
}

У меня возникает ошибка и я не знаю где. Кто знает где у меня ошибка?

Comments

[Saika2020]

Кстати нужно решить создать массив с помощью ArrayList

[Сергей Горностаев]

Отформатируйте код и добавьте в вопрос полный текст ошибки.

[Сергей Водаков]

Почему не знаете где возникает ошибка? Это может быть только в одном случае: вы пишите код на бумажке, а компилирует его преподаватель у себя в голове.

Answer 1

[Денис Загаевский]

Учись читать сообщения компилятора. А так у тебя всё не так, в коде практически нет правильных строк.
Точка должна содержать свои координаты.
Массив должен содержать точки, а не даблы.
Массив ее не должен быть внутри точки.
Периметр одной точки это оксюморон, и конечно, периметр вычисляется не сложением координат.

Comments

[Сергей Водаков]

Массив у него не внутри точки а внутри фигуры, с этим всё ок.
А то что он точки хранит в массиве как даблы, это конечно говнокод, но если массив строго упорядочен и выбирать из массива по три значения для обработки, то можно работать и с этим. Хотя повторюсь, это жуткий говнокод, с букетом неприятных последствий.

[Денис Загаевский]

Сергей Водаков, ааа, ну тогда-то всё нормально))

[Сергей Водаков]

Денис Загаевский, Нет, там всё не нормально =) но надо же человека направить в правильном направлении, а не просто поругать.

Answer 2

[Сергей Водаков]

В классе Main в методе main идет обращение к необьявленной переменной arrayList.
Далее, кроме грубой синтаксической ошибки, здесь есть логическая. При вычислении периметра, вы просто складываете все координаты первой точки. Это точно не периметр!
Для определения расстояния между двумя точками, нужно извлечь квадратный корень из суммы квадратов разностей всех координат.

Answer 3

[Орхан Гасанлы]

console.log(arrayList.perShape());
Т.е. никого не напрягает, что эта строчка кода вообще не из Java, а скорее из JavaScript?