[lxa85]

Код как код. Не обременен комментариями, не выполняет каких-либо сверх задач, содержит ошибку вычисления при реализации прямоугольного треугольника.
На мой взгляд процедура public void squareOut() { ... } лишняя.
Я не люблю, когда за меня думает процедура. Т.е. я сам решу, какой текст следует писать, и нужен ли он мне вообще.
Из более грубых ошибок - отсутствие проверки треугольника. Т.е. нельзя в планиметрии получить треугольник со сторонами 1, 1, 6.
А в этой программе можно. Тоже касается и отрицательных величин.

[Iska]

Т.е. нельзя в планиметрии получить треугольник со сторонами 1, 1, 6. А в этой программе можно. Тоже касается и отрицательных величин. »

Неевклидовость. Антивселенная. .

[nomad____1]

Код как код. Не обременен комментариями, не выполняет каких-либо сверх задач, содержит ошибку вычисления при реализации прямоугольного треугольника. На мой взгляд процедура public void squareOut() { ... } лишняя. Я не люблю, когда за меня думает процедура. Т.е. я сам решу, какой текст следует писать, и нужен ли он мне вообще. Из более грубых ошибок - отсутствие проверки треугольника. Т.е. нельзя в планиметрии получить треугольник со сторонами 1, 1, 6. А в этой программе можно. Тоже касается и отрицательных величин. »

Благодарю за ответ. Комментарии писать буду, а то я забываю что кроме меня еще кто то будет его читать. Насчет правил планиметрии так сказать "проверку на дурака" я не писал... просто хотел по быстрому накидать прожку. Сегодня допишу с учетом всех ошибок и скину сюда. Интересовало мнение насчет построения класса, методов, конструктора. Благодарю за критику. Если вы не против то по мере изучения я буду сливать сюда коды. Хотелось бы видеть от вас критику по их поводу. Мне кажется просто что так я научусь писать более менее нормальный код.

[nomad____1]

Еще можно ли использовать геттеры и сеттеры? А то много негативного о них в интернете написано... Или их использование будет дурным тоном?

[lxa85]

В "интернетах" много что пишут. Если специфика работы объекта подсказывает использования сеттера или геттера - то вполне.
Если сеттер и геттер "поганит" данные и вызывает неявное/непредсказуемое действие - тогда это проблема.
nomad____1, setter и getter - это инструмент. Пользоваться им или нет - решать не мне.
В данном случае в процедуру установки параметров можно прописать процедуру установки флага "валидности" треугольника.
Тогда в процедуре вычисления площади будет проверяться условие планарности треугольника по флагу.
Моветон?
----
nomad____1, как вам сказать. Точность - вежливость королей. По таким мелочам судят о специалисте.
На счет построения конструктора класса и методов - сказать сложно. Ваш пример с одной стороны достаточно каноничен и в нем нет сложности, с другой оторван от реальности и практики.
Те же прямоугольные треугольники здесь не учитываются. Ответ будет дан с определенной погрешностью (кстати вопрос "на засыпку" как специалисту: какая погрешность метода при вычислении площади прямоугольного треугольника?).
Отсутствие условий планиметрии приведет к ошибке - вычислению корня из отрицательного числа (NaN).
----
Iska, а вот интересно. Допустим у нас некая выпуклая фигура, на плоскости которой я могу нарисовать треугольник со сторонами 1:1:6. И ... наверно собственно все. По данным сторонам я не смогу сказать какая это фигура? Допустим шар, сфера.
Наверно нет.
Получится у меня должны (пусть катеты) две стороны лежать на прямых плоскостях, а гипотенуза в 6 ед. должна быть свернута в гармошку.
Так что ли?
----
P.S. Не стоит без особой надобности выделять текст жирным шрифтом.

[nomad____1]

валидность треугольника УЧТЕНО И ДОРОБОТАНО

]В данном случае в процедуру установки параметров можно прописать процедуру установки флага "валидности" треугольника. Тогда в процедуре вычисления площади будет проверяться условие планарности треугольника по флагу.[/i] »

Сделал. Не знаю так ли как Вы написали.

Код:

//      Проверка планарности треугольника       
            if ((aSide + bSide) > cSide && (aSide + cSide) > bSide && (bSide + cSide) > aSide) {
//                Вычисление площади

точность УЧТЕНО

Точность - вежливость королей. По таким мелочам судят о специалисте. »

Программу буду улучшать по мере изучения

учтено. программу буду улучшать по мере изучения

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

Те же прямоугольные треугольники здесь не учитываются. Ответ будет дан с определенной погрешностью (кстати вопрос "на засыпку" как специалисту: какая погрешность метода при вычислении площади прямоугольного треугольника?). Отсутствие условий планиметрии приведет к ошибке - вычислению корня из отрицательного числа (NaN). »

Вы сами дали ответ на свой вопрос) точность! в прямоугольном треугольнике площадь равна (а*b)/2 ... очень высокая точность!) А вот вычисление квадратного корня с числа например 2 или 3....)

Тут уже вопрос как заставить ее работать... У меня выводит 0. Так как не может вычислить целый квадратный корень.

P.S. Не стоит без особой надобности выделять текст жирным шрифтом. »

Это чисто для себя. Так сказать "вопрос-ответ". Выделение своих сообщений (вопросов). Для более простейшей навигации.

Вот код после доработки:

главный класс

package square;

Код:

public class Square {

    public static void main(String[] args) {
        Triangle triangle = new Triangle(2, 5, 6);
        triangle.squareOut();
    }
}

Класс с методами и конструктором

package square;

Код:

public class Triangle {

    private int aSide;
    private int bSide;
    private int cSide;
    private double square;
    private double perimeter;
    private int vuvod;

    public Triangle(int aSide, int bSide, int cSide) {
        this.aSide = aSide;
        this.bSide = bSide;
        this.cSide = cSide;
    }

    private void print(String str) {
        System.out.println(str);
    }

//    
//      Проверка "на дурака" введенных значение и вычисление площади
//        
    public void square() {
//       Проверка нет ли стороны с отрицательным значением или равным нулю 
        if (aSide <= 0 || bSide <= 0 || cSide <= 0) {
            vuvod = 1;
        } else {
//      Проверка планарности треугольника       
            if ((aSide + bSide) > cSide && (aSide + cSide) > bSide && (bSide + cSide) > aSide) {
//                Вычисление площади
                perimeter = (aSide + bSide + cSide) / 2;
                square = Math.sqrt(perimeter * (perimeter - aSide) * (perimeter - bSide) * (perimeter - cSide));
                vuvod = 2;
            } else {
//      Когда данные числа не могут соответствовать длине сторон треугольника     
                vuvod = 3;
            }
        }
        if (square == 0) {
            vuvod = 0;
        }
    }

// 
// Вывод значения на экран   
//    
    public void squareOut() {
        this.square();
        switch (vuvod) {
            case 1:
                print("Длинна не может быть отрицательным числом");
                break;
            case 2:
                print("Площадь треугольника равна " + square);
                break;
            case 3:
                print("У треугольника не может быть сторон с такими значениями");
                break;
            default:
                print("\tхм... что то не так с программой... Советую заглянуть в метод  square... \n\t Скорее всего программа не может добыть квадратный корень. ");
        }
    }
}

P.S.
Сегодня ночью думаю скину код где будет проверка треугольника на прямоугольность и вычисление площади по отдельной формуле. Думаю использовать сравнение трех сторон. Две самые "мелкие" должны быть катетами. Дальше с помощю теоремы Пифагора проверить будет ли получена гипотенуза равна "нашей" введеной. пока мой извращенный мозг более простого не придумал)))

P.P.S.
Надеюсь етот смайл пробет стенку головой)

[User001]

Допустим у нас некая выпуклая фигура, на плоскости которой я могу нарисовать треугольник со сторонами 1:1:6. И ... наверно собственно все. По данным сторонам я не смогу сказать какая это фигура? Допустим шар, сфера. Наверно нет. »

Жуть.
А по теме мои 5 копеек:

private int vuvod; »

Поменяйте название переменной.

case 1: »

case 2: »

Если так и пойдет дело, потом тяжело будет угадывать что это за магические константы.

public void square() { »

public void squareOut() »

Лучше это переделать. Как вариант, square возвращает площадь или что-то в этом роде. В вашем случае непонятно зачем square() public.

vuvod = 0; »

Его в switch нет.

Длинна не может быть отрицательным числом»

Бросается в глаза.

[nomad____1]

vuvod = 0; » Его в switch нет. Цитата nomad____1: Длинна не может быть отрицательным числом» »

если я правильно понял то 0 это дефолтное значение.

Цитата nomad____1: case 1: » Цитата nomad____1: case 2: » Если так и пойдет дело, потом тяжело будет угадывать что это за магические константы. »

Это да косяк. Забываю. С СИ начинал учить программирование.

Насчет неправильного вывода то разобрался что нужно поменять типы данных переменных в которые записываются длинны сторон на флоаты или даблы.

[lxa85]

nomad____1, касательно public'ов, static'ов и пр. я не скажу. Но переменная vuvod у меня вызывает подозрения.
Почему используется конструкция public void SquareOut.
Почему не int или не double ?
Я от программы хочу получить ответ. Желательно наиболее простой и пригодный для "дальнейшей эксплуатации".
Мне совершенно не хочется заниматься парсингом строки (притом не возвращаемой в силу void).
Чтобы сигнализировать об ошибке следует возвращать отрицательные значения. -1, -2 и т.д.
Поведение разумеется должно быть описано в сопровождающей документации и в комментариях к процедуре.
----

Сделал. Не знаю так ли как Вы написали. »

Нет. Я сказал написать проверку в процедуре установки параметров, а не в процедуре вычисления площади.
Напомню, речь шла про сеттеры, гетторы и ввод "флага валидности" треугольника.
----

Две самые "мелкие" должны быть катетами »

А вдруг нет? Вдруг "мелкие" не катят?!!