Не получается взаимодействовать с params. Что делать?

Question

[CallMeYourDaddy]

Я нахожу площадь фигур и делаю следующее:

public FigureArea(params double[] numbers)
        {
            if (numbers.Length == 1)
            {
                var area = Math.PI * Math.Pow(numbers, 2);
            }
        }

Ну вы поняли, что здесь происходит. Проблема в том, что numbers - это массив и не совсем понятно, как элемент этого массива умножить на 2

я могу, конечно, и просто конструкторов понаделать:

public class FigureArea : IFigureArea
    {     
        public FigureArea(double radius)
        {
            CirleByRadius(radius);
        }

        public FigureArea(double firstSide, double secondSide, double thirdSide)
        {
            TriangleBySide(firstSide, secondSide, thirdSide);
        }

Но мне такой вариант не нравится, я хочу более лаконичный (через парамс)

Answer 1

[freeExec]

var area = Math.PI * Math.Pow(numbers[0], 2);

Comments

[CallMeYourDaddy]

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

[Hemul GM]

CallMeYourDaddy, на джуна надеюсь?

[CallMeYourDaddy]

Hemul GM, на стажера

[Василий Банников]

CallMeYourDaddy, если честно, оба варианта выглядят не очень.
Сделайте лучше для каждой фигуры отдельный класс.

Answer 2

[Владимир Коротенко]

Сделайте интерфейс ifigure у него метод getsquare и просто дергайте его. А каждый класс пусть сам считает

Comments

[CallMeYourDaddy]

а какие тогда аргументы в getsquare должны быть?
вообще в тз вот это: Вычисление площади фигуры без знания типа фигуры

[Владимир Коротенко]

Interface ifigure {
Double getsquare();}
Class Square : ifigure {
Public double getsquare => w*h;
} }

Дальше для всех фигур делаете то же, а при вычислении просто дергаете этот метод.

[freeExec]

CallMeYourDaddy, так это тогда всё превращается в разбиение многоугольника на треугольники и подсчёт суммы их площадей.

[Владимир Коротенко]

freeExec, не вводи в заблуждение, та и просто фигурки квадрат круг прямоугольник треугольник что то еще

[freeExec]

Владимир Коротенко, а в чём заблуждение, даже круг можно апроксимировать в кучу треугольников, но проще конечно воспользоваться формулой.
А вот ваш вариант тут выходит с большим оверхедом, чтобы определять, что нам пришло на вход: квадрат, прямоугольник, ромб, трапеция или ни чего классического?

[Владимир Коротенко]

freeExec, на вход пришёл интерфейс с одним методом. Вызывающий вообще не знает что за фигура там просто просит площадь и все. Что там внутри знает только он сам, в лучших традициях ООП

[freeExec]

Владимир Коротенко, какой ещё интерфейс, на вход длины сторон. Для того, чтобы был интерфейс, надо сначала решить какой класс взять, в этом вся и загвоздка.

[Владимир Коротенко]

freeExec, так и нужно его передавать, а не гадать по преданным параметрам. А не то получиться жёсткое связывание с возможными ошибками. Допустим есть Элипс и прямоугольник, как их различать? Есть ширина и высота

[freeExec]

Владимир Коротенко, тогда это было бы слишком легко, для тестового задания.

[Владимир Коротенко]

freeExec, обрати внимание на фразу вычисление площади без знания типа фигуры. Тут явно неоднозначность. В моем случае ее нет

[freeExec]

Владимир Коротенко, но в твоём случае мы знаем фигуру, что противоречит первоначальным условиям.
Ну и наверняка там было описано что да как, чего автор нам не привёл, что-то типа: 1 параметр - радиус круга, 2 - прямоугольник, остальное длины сторон многоугольника.

[Владимир Коротенко]

Сам автор написал, то есть судя по этому коду он знает какая фигура, во всяком случае в момент вызова
freeExec,

public FigureArea(double radius)
        {
            CirleByRadius(radius);
        }

        public FigureArea(double firstSide, double secondSide, double thirdSide)
        {
            TriangleBySide(firstSide, secondSide, thirdSide);
        }

[freeExec]

Владимир Коротенко, судя по первоначальному params double[] numbers + if (numbers.Length == 1) он сам интерпретирует фигуру по количеству параметров.

[Владимир Коротенко]

freeExec, хм ну это явно нарушение тз и инкапсуляции.