Как решить данную задачу корректно?

Question

[MacDag]

Решил задачу. Всё работает. Но преподаватель указал некоторые ошибки. Вот условие задачи:

Задание 1. Написать класс Round, задающий круг с указанными координатами центра, радиусом, а также свойствами, позволяющими узнать длину описанной окружности и площадь круга. Обеспечить нахождение объекта в заведомо корректном состоянии. Написать программу, демонстрирующую использование данного круга.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApplication2
{
    class Program
    {
        static void Main(string[] args)
        {
            var circle = new Circle();
            circle.Radius = 10;
            Console.WriteLine("Координаты x = {0} , y = {1}  " , circle.X , circle.Y);
            Console.WriteLine("Площадь круга = {0}" , circle.GetArea);
            Console.WriteLine("Длина  = {0}" , circle.Length);
        }

        public class Circle 
        {
            private double x, y , r;
            public double Radius
            {
                get 
                {
                    return r;
                }

                set 
                {
                    x = y = 0;
                    if (value <= 0) { throw new ArgumentException (" Wrong! ") ;}
                    r = value;
                }
            }

            public double GetArea
            {
                get
                {
                    return Math.PI * r * r;
                }
            }

            public double Length
            {
                get
                {
                    return 2 * Math.PI * r;
                }
            }

            public double X
            {
                get
                {
                    return x;
                }
            }

            public double Y
            {
                get
                {
                    return y;
                }
            }
        }
    }
}

Answer 1

[Иван Ершов]

C++ отличается от С# поэтому поменяйте тэг

class Round
{
private:
	double _x;
	double _y;
	double _radius;

public:
	Round(double x, double y, double radius)
	{
		_x = x;
		_y = y;
		_radius = radius;
	}

	Round(){}

	double GetAbscissa()
	{
		return _x;
	}

	double GetOrdinate()
	{
		return _y;
	}

	double GetRadius()
	{
		return _radius;
	}

	double GetSquare()
	{
		return M_PI * _radius * _radius;
	}


};

Answer 2

[brutal_lobster]

Ошибки такие же как и в других задачах - почитайте книжку по ООП - особенно в части инкапсуляции.

У окружности только два параметра - координаты центра и радиус. Их вы должны передавать при создании объекта. После создания их нельзя изменять напрямую.
Если есть необходимость, то можно создать два доп. метода - move() и enlarge()/shrink()/...

Ну и форматирование, сериализация..

Comments

[prometian]

Пожалуйста, укажите на ошибки.
pastebin.com/LTzvCNCf

[brutal_lobster]

Прочитай книжку по ООП, пожалуйста :) И по языку, на котором пишите.

1. Не является хорошей практикой так использовать внешние объекты (в вашем случае std::in,std::out) в конструкторах. Во-первых, они не нужны для инициализации, во-вторых - появляется жесткая зависимость от конкретных внешних in/out. Передавайте параметры в конструктор как аргументы
2. Не используйте интерактивный ввод параметров - указывайте их в аргументах приложения или считывайте из файла
3. Добавьте проверки входных данных
4. Добавьте возможность сериализации объекта или хотя бы его приятного текстового отображения
5. Зачем вам сеттеры для радиуса и координат? Подумайте над семантикой этих методов
6. pi задана совсем уж не точно - есть же соответствующие константы и функции

[MacDag]

Можешь решение мне отправить как нибудь ?

[brutal_lobster]

эх ты

[prometian]

@brutal_lobster благодарю за развернутый ответ. 1.2. учту, тесты всегда так писал, новичку проще, но видимо привыкать не стоит... 4. 5. ок. 6. торопился =) нужно было math.h подключать и там можно было взять пи, но наизусть не знаю.

[MacDag]

@brutal_lobster отправь плиз решение.. я эту задачу уже неделю как делаю. но не могу понять в чем ошибки. и не знаю как их исправить...

[MacDag]

@brutal_lobster и что означает форматирование и сериализация?

[prometian]

@brutal_lobster 5. чтобы изменить объект и пересчитать длину и площадь =) я думаю что там стоит проверку добавить. А возможность изменить координаты, это по сути движение объекта, т.е. если мы имеем два таких объекта, то можно посчитать пересекаются они или нет (к примеру).

Answer 3

[scottparker]

комментарии к коду, это лично мое мнение

public class Circle 
        {
            private double x, y , r;  
            public double Radius   //Имя инициализатора должно соответствовать имени поля 
//которое он инициализирует, логичнее поле назвать radius
            {
                get 
                {
                    return r;
                }

                set 
                {
                    x = y = 0;   //Почему в инициализаторе радиуса, инициализируются 
//координаты центра круга? и почему они именно равны нулю? 
                    if (value <= 0) { throw new ArgumentException (" Wrong! ") ;}
                    r = value;
                }
            }
//Возможно следовало бы написать конструктор с параметрами 
            public double GetArea
            {
                get
                {
                    return Math.PI * r * r; //Доступ к полям проводить через инициализаторы 
                }
            }

            public double Length
            {
                get
                {
                    return 2 * Math.PI * r;
                }
            }

            public double X
            {
                //set?
                get
                {
                    return x;
                }
            }

            public double Y
            {
                //set?
                get
                {
                    return y;
                }
            }
        }
    }
}