Java

nomad____1 · Конфигурация компьютера

Цитата User001:

Написано public. »

Это сеттеры они должны быть публичными. Иначе они не будут доступными с других классов.

Цитата nomad____1:

Код:

if (aSide + bSide > cSide && bSide + cSide > aSide && aSide + cSide > bSide) {
 planarity = true;

»

Это проверка на планарность и установка флага планарности (флаг нужен если например мы просто хотим получить инфу о планарности), а вызывает другой метод для того что бы обеспечить то что зовется "самостоятельность". Притом я же сказал

Цитата nomad____1:

Проходит проверка на планарность, а потом вызывается метод по котором проверяется фигура на прямоугольность. Можно написать просто методы. Но все равно их вызвать надо будет в методе square() что бы метод сам подтянул что ему нужно для вычисления. Хм... так красивее было бы. Так и сделаю... »

Будет правильнее сделать как я писал. Наверно так будет целесообразно. Есть методы которые что то проверяют, считают независимо. Но что бы оно обрело смысл то поместью вызов методов в метод square. Так просто при вызове данного метода будет выводится площадь сразу. Вся логика метода данного закрыта и недоступна для изменения. Мы сможем только вызвать данный метод на выполнение через публичный геттер или какое то промежуточное значение (так же через геттер (планарность, прямоугольность и т.д.)) но при этом не можем изменить значение для правильной логики и вычислений.

Цитата lxa85:

Зачем мы вводили флаг прямоугольности, если я могу его проигнорировать?
Код:

Код:

Triangle triangle = new Triangle();
 triangle.inputSides();
System.out.println(triangle.getaSide());
 System.out.println(triangle.getbSide());
 System.out.println(triangle.getcSide());
System.out.println(triangle.getSquare());
 System.out.println(triangle.getRightTriangleSquare());
 System.out.println(triangle.getGeronSquare());

»

Вот например. Проигнорировать больше не получится. Будет выполнятся принудительно и последовательно. Я же кажется писал что перепишу. Сейчас времени нет. В течении 2-3 часов выложу. И еще кое что.

Цитата nomad____1:

Простите. Я не хотел обидеть. Без злости и задней мысли все пишу. Я просто так раскрываю свою точку зрения. Еще раз прошу прощения если показался дерзким или где то хамил. »

nomad____1 · Конфигурация компьютера

Тогда как такой код???

мейн

Код:

package mathprogram;

import java.util.Scanner;

public class MathProgram {

    public static void main(String[] arg) {
        Triangle triangle = new Triangle();
//        triangle.inputSides();

        Scanner input = new Scanner(System.in);
        System.out.println("Введите первую сторону");
        float sideA = input.nextFloat();
        System.out.println("Введите вторую сторону");
        float sideB = input.nextFloat();
        System.out.println("Введите третью сторону");
        float sideC = input.nextFloat();
        triangle.setSides(sideA, sideB, sideC);
        triangle.getSquare();
        if (triangle.getSquare() <= 0) {
            System.out.println("Введены некорректные данные");
        }else{
            System.out.println("Площадь треугольника равна "+triangle.getSquare()+" кв.ед.");
        }
    }
}

методарий

Код:

package mathprogram;

public class Triangle {

    private float aSide;
    private float bSide;
    private float cSide;
    private boolean setSides;
    private boolean planarity;
    private boolean rightTriangle;
    private float square;

    public boolean setSides(float sideA, float sideB, float sideC) {
        setSides = false;
        if (sideA > 0 && sideB > 0 && sideC > 0) {
            aSide = sideA;
            bSide = sideB;
            cSide = sideC;
            setSides = true;
        }
        return setSides;
    }

    private boolean planarity() {
        planarity = false;
        if (aSide + bSide > cSide && bSide + cSide > aSide && aSide + cSide > bSide) {
            planarity = true;
        }
        return planarity;
    }

    private boolean rightTriangle() {
        rightTriangle = false;
        if (aSide * aSide + bSide * bSide == cSide * cSide || aSide * aSide + cSide * cSide == bSide * bSide || bSide * bSide + cSide * cSide == aSide * aSide) {
            rightTriangle = true;
        }

        return rightTriangle;
    }

    private float geronSquare() {
        if (planarity == true) {
            float perimeter = (aSide + bSide + cSide) / 2;
            square = (float) Math.sqrt(perimeter * ((perimeter - aSide) * (perimeter - bSide) * (perimeter - cSide)));
        }
        return square;
    }

    private float rightTriangleSquare() {
        if (planarity == true && rightTriangle == true) {
            square = aSide * bSide / 2;
        }
        return square;
    }

    private float square() {
        if (planarity == true) {
            if (rightTriangle == true) {
                rightTriangleSquare();
            } else {
                geronSquare();
            }
        }
        return 0;
    }

    public float getGeronSquare() {
        planarity();
        geronSquare();
        return square;
    }

    public float getRightTriangleSquare() {
        planarity();
        rightTriangleSquare();
        return square;
    }

    public boolean isPlanarity() {
        return planarity;
    }

    public boolean isRightTriangle() {
        return rightTriangle;
    }

    public float getSquare() {
        planarity();
        rightTriangle();
        square();
        return square;
    }

    public float getaSide() {
        return aSide;
    }

    public float getbSide() {
        return bSide;
    }

    public float getcSide() {
        return cSide;
    }
}

Все защищено. Лишнего ничего нет. Тявкает то что ему сказано. Может и не тявкать если надо. Это уже мы выбираем. Стороны вводятся через сеттеры. Внутренние методы работают только то что надо, а именно считают и не вызываю других методов. Можно получить любые интересующие данные благодаря геттерам. Принудительно что то в логике изменить нельзя. Также класс сам считает более подходящим способом.

lxa85 · Отправлено: **00:28, 09-05-2015** | #33

nomad____1, хочешь иголку воткну?
Ну так, чтобы не скучно было?

В код не подглядывай, а утром поищи ошибку самостоятельно.
Если не найдешь - смотри тест. Но лучше сам.

Тест

Не, не, не! Не надо!

Да я ж под пару тройку спойлеров то точно спрячу ;)

Я предупреждал!

Код:

        float sideA = 5; float sideB = 4; float sideC = 3;
        triangle.setSides(sideA, sideB, sideC);
        triangle.getSquare();
        System.out.println("Площадь треугольника равна "+triangle.getSquare()+" кв.ед.");
         
        sideA = 3; sideB = 4; sideC = 5;
        triangle.setSides(sideA, sideB, sideC);
        triangle.getSquare();
        System.out.println("Площадь треугольника равна "+triangle.getSquare()+" кв.ед.");

nomad____1 · Конфигурация компьютера

ок... на честность... поищу... утром... пффф))) я ложусь в 4 часа ночи))) по пару раз на день захожу сюда... так что до утра попробую найти, а нет то посмотрю текст

может этот кусок переписать

Код:

    public float getRightTriangleSquare() {
        planarity();
        rightTriangleSquare();
        return square;

на такой

Код:

    public float getRightTriangleSquare() {
        planarity();
        rightTriangle();
        if (rightTriangle == true) {
            rightTriangleSquare();
        } else {
            square = 0;
        }
        return square;
    }

а то как то совсем не фонтан.
и еще парочку...

Скрытый текст

было

Код:

    private float square() {
        if (planarity == true) {
            if (rightTriangle == true) {
                rightTriangleSquare();
            } else {
                geronSquare();
            }
        }
        return 0;
    }

стало

Код:

    private float square() {
        if (planarity == true) {
            if (rightTriangle == true) {
                rightTriangleSquare();
            } else {
                geronSquare();
            }
        }else{
            square=0;
        }
        return square;
    }

Скрытый текст

было

Код:

    public float getRightTriangleSquare() {
        planarity();
        rightTriangleSquare();
        return square;
    }

стало

Код:

    public float getRightTriangleSquare() {
        planarity();
        rightTriangle();
        if (rightTriangle == true&&planarity==true) {
            rightTriangleSquare();
        } else {
            square = 0;
        }
        return square;
    }

Скрытый текст

было

Код:

    public float getRightTriangleSquare() {
        planarity();
        rightTriangleSquare();
        return square;
    }

стало

Код:

    public boolean isRightTriangle() {
        planarity();
        if (planarity==true){
            rightTriangle();
        }else{
            rightTriangle=false;
        }
        return rightTriangle;
    }

nomad____1 · Конфигурация компьютера

Цитата lxa85:

nomad____1, хочешь иголку воткну?
Ну так, чтобы не скучно было?
В код не подглядывай, а утром поищи ошибку самостоятельно.
Если не найдешь - смотри тест. Но лучше сам.
Тест
Не, не, не! Не надо!
Да я ж под пару тройку спойлеров то точно спрячу

Я предупреждал!
Код:

Код:

float sideA = 5; float sideB = 4; float sideC = 3;
 triangle.setSides(sideA, sideB, sideC);
 triangle.getSquare();
 System.out.println("Площадь треугольника равна "+triangle.getSquare()+" кв.ед.");
sideA = 3; sideB = 4; sideC = 5;
 triangle.setSides(sideA, sideB, sideC);
 triangle.getSquare();
 System.out.println("Площадь треугольника равна "+triangle.getSquare()+" кв.ед."); »

Ахах)))) даже и не подумал) сортировку массива и пере присвоение убрал, а тут такой фейл оставил)))
Четко)

Я хотел написать метод нахождение минимума и максимума и с его помощью считать. Но как то вылетело. И пожалуйста скиньте рисунки которые обещали... Или будет массив и будет сортировка но только! в даном методе исключительно ради точности вычислений.

Зато сам еще потестил и нашел пару багов)))

P.S.
Утром перепишу)

nomad____1 · Конфигурация компьютера

С днем Великой победы товарищи!!!!

Для исправление данной "иголки" использовал массив и сортировку. Но только в данном методе. На остальные данные не влияет и на логику программы в целом.

Код:

    private float rightTriangleSquare() {
        float [] arr={aSide, bSide, cSide};
        Arrays.sort(arr);
        if (planarity == true && rightTriangle == true) {
            square = arr[0] * arr[1] / 2;
        }
        return square;
    }

А вот переписанное!

мейн

Код:

package mathprogram;

import java.util.Scanner;

public class MathProgram {

    public static void main(String[] arg) {
        Triangle triangle = new Triangle();
        Scanner input = new Scanner(System.in);
        System.out.println("Введите первую сторону");
        float sideA = input.nextFloat();
        System.out.println("Введите вторую сторону");
        float sideB = input.nextFloat();
        System.out.println("Введите третью сторону");
        float sideC = input.nextFloat();
        triangle.setSides(sideA, sideB, sideC);
        if (triangle.getSquare() <= 0) {
            System.out.println("Введены некорректные данные");
        } else {
            System.out.println("Площадь треугольника равна " + triangle.getSquare() + " кв.ед.");
        }
    }
}

методарий

Код:

package mathprogram;

import java.util.Arrays;

public class Triangle {

    private float aSide;
    private float bSide;
    private float cSide;
    private boolean setSides;
    private boolean planarity;
    private boolean rightTriangle;
    private float square;

    public boolean setSides(float sideA, float sideB, float sideC) {
        setSides = false;
        if (sideA > 0 && sideB > 0 && sideC > 0) {
            aSide = sideA;
            bSide = sideB;
            cSide = sideC;
            setSides = true;
        }
        return setSides;
    }

    private boolean planarity() {
        planarity = false;
        if (aSide + bSide > cSide && bSide + cSide > aSide && aSide + cSide > bSide) {
            planarity = true;
        }
        return planarity;
    }

    private boolean rightTriangle() {
        rightTriangle = false;
        if (aSide * aSide + bSide * bSide == cSide * cSide || aSide * aSide + cSide * cSide == bSide * bSide || bSide * bSide + cSide * cSide == aSide * aSide) {
            rightTriangle = true;
        }
        return rightTriangle;
    }

    private float geronSquare() {
        if (planarity == true) {
            float perimeter = (aSide + bSide + cSide) / 2;
            square = (float) Math.sqrt(perimeter * ((perimeter - aSide) * (perimeter - bSide) * (perimeter - cSide)));
        }
        return square;
    }

    private float rightTriangleSquare() {
        float [] arr={aSide, bSide, cSide};
        Arrays.sort(arr);
        if (planarity == true && rightTriangle == true) {
            square = arr[0] * arr[1] / 2;
        }
        return square;
    }

    private float square() {
        if (planarity == true) {
            if (rightTriangle == true) {
                rightTriangleSquare();
            } else {
                geronSquare();
            }
        } else {
            square = 0;
        }
        return square;
    }

    public float getGeronSquare() {
        planarity();
        geronSquare();
        return square;
    }

    public float getRightTriangleSquare() {
        planarity();
        rightTriangle();
        if (rightTriangle == true && planarity == true) {
            rightTriangleSquare();
        } else {
            square = 0;
        }
        return square;
    }

    public boolean isPlanarity() {
        return planarity;
    }

    public boolean isRightTriangle() {
        planarity();
        if (planarity == true) {
            rightTriangle();
        } else {
            rightTriangle = false;
        }
        return rightTriangle;
    }

    public float getSquare() {
        planarity();
        rightTriangle();
        square();
        return square;
    }

    public float getaSide() {
        return aSide;
    }

    public float getbSide() {
        return bSide;
    }

    public float getcSide() {
        return cSide;
    }
}

lxa85 · Отправлено: **22:41, 09-05-2015** | #37

nomad____1, хорошо, только маленькая поправка.

Код:

    private float rightTriangleSquare() {
        if (planarity == true && rightTriangle == true) {
                    float [] arr={aSide, bSide, cSide};
                    Arrays.sort(arr);
            square = arr[0] * arr[1] / 2;
        }
        return square;
    }

Если треугольник непланарный и неправильный, то лишние действия нам не нужны.
P.S. На счет картинок. Почти все ошибки исправлены, поэтому они тебе здесь уже не нужны. Там уже ничего примечательного.

nomad____1 · Конфигурация компьютера

Цитата lxa85:

Если треугольник не планарный и не правильный, то лишние действия нам не нужны. »

Ок благодарю) Вашу поправку внесу в класс. Уже упаковал с помощью javaFX в оболочку. Теперь еще поднаростить функционал)))

Цитата lxa85:

Почти все ошибки исправлены »

А что не так еще осталось. Это уже ради интереса.

P.S.
Завтра допишу функционал еще... Как говорил

Цитата nomad____1:

девушка подкинула идею. Усложнить можно если треугольник задается в 2Д плоскости координатами вершин и потом в 3д плоскости. Соответственно написать конструкторы и написать пару методов для нахождения длинны отрезков заданных координатами их концов а далее по написанному моему трафарету. »

nomad____1 · Конфигурация компьютера

что можете посоветовать учить после завершения ООП. JavaFX немного понял принципы. Можно ли приступать к изучению JDBC, мавен, ант т в какой последовательности.

lxa85 · Отправлено: **15:03, 18-05-2015** | #40

nomad____1, мысль вслух.
Лучший учитель - это натура.
Учиться согласно поставленных целей.
Задачками мир наполнен более чем.