Где лучше создавать экземпляр класса C#?

Question

[estry]

Привет. В методе нужно создавать экземпляр класса и возвращать его из метода. Подскажите где лучше его создать, в методе или поле класса.
Так?

class Cars
{	
	public Engine CreateEngine()
	{
		Engine engine = new Engine();
		
		return engine;
	}
}

Или вот так?

class Cars
{	
	Engine engine = new Engine();
	
	public Engine CreateEngine()
	{
		
		
		return engine;
	}
}

Answer 1

[Vladimir S]

в ваших примерах нет разницы где создавать - т.к. там разное поведение.
в первом случае на запрос двигателя будет создаваться новый каждый раз, а во втором - возвращаться один и тот же.

Comments

[estry]

Перед тем как вернуть класс будут переназначаться поля в обоих случаях.

[Vladimir S]

если нужен каждый раз новый двигатель - то создавать его лучше в методе

Answer 2

[yuopi]

Первый жизнеспособный, если только к двигателю больше никакой метод в классе Car не будет обращаться.

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

class Cars 
{
      private Engine _engine;

      public Engine Engine => _engine;  //если нужен доступ к Engine из другого класса

      public void CreateEngine() 
      {
            _engine = new Engine();
      }
}

Answer 3

[Дмитрий Макаров]

Машина не создаёт двигатели, двигатели создаёт завод.

Но если абстрагироваться от смысла и названий, то оба варианта могут существовать:

В первом варианте мы создаём каждый раз новый объект. Например, это может быть метод для парсинга строк и создания на основе этих строк разных объектов. Что-то типа: Node ParseNode(string textNode);

Во втором случае объект выдаваемый методом создаётся при создании класса. В этом случае мы имеем что-то подобное паттерну Singleton (Одиночка). У нас есть один объект, эксплуатация которого затратна по ресурсам (например, долго создаётся, или требует много памяти, или требует отдельное сетевое соединение), но в целом для работы остального кода достаточно и одного этого объекта. Мы каким-то способом этот объект создаём и затем всем кому надо отдаём один и тот же объект.

Comments

[Hemul GM]

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

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

Псевдокодом говоря...
Авто = Завод.СоздатьАвто()

И на этом всё.

В твоём же случае получается...

Авто = Завод.СоздатьАвто()
Авто.Двигатель = Завод.СоздатьДвигатель()

Как-то не очень...

[Дмитрий Макаров]

Hemul GM, мы немного не в ту степь ушли, но тем не менее можно продолжить...

Вообще не факт, что если объект агрегирует другой объект, то он его и создаёт. Например, коллекции. Элементы списка входят в список, но обычно не списки сами создают свои элементы.
И есть очень объёмная тема Dependency Injection (внедрение зависимостей) - там очень много нюансов: кто, когда и где инстанцирует объекты. Можно, конечно, поговорить об этом, но наверное лучше не здесь. И в случае с DI действительно получается что-то типа:

Двигатель = Завод.СоздатьДвигатель()
Авто = Завод.СоздатьАвто(Двигатель)

В случае DI тут "Завод" - это DI-контейнер (как пример и очень грубо).

То какие методы (публичные) есть у класса определяют API класса. То есть то как его будет воспринимать пользователь этого класса.
Чисто семантически (по смыслу, в обыденном представлении) странно от автомобиля ожидать, что он будет создавать двигатель. Пользователь автомобиля (в том числе программного объекта) может ожидать от API автомобиля, например, таких действий как поворота, изменения скорости, включения/выключения фар и т.п.

Answer 4

[DanielMcRon]

Если нужно создавать и возвращать, то первый

Comments

[estry]

В методе поля постоянно переназначаются перед тем как вернуть экземпляр.
Лучше создать один раз, а потом обращаться к методу? Или постоянно новый создавать?

[DanielMcRon]

estry, если ваша задача новый экземпляр делать, то первый вариант. Если везде один и тот же - второй

[freeExec]

estry, зависит же от задачи. Может вам вообще не нужен метод CreateEngine и двигатель должен всегда создаваться в конструкторе Cars

[estry]

freeExec, Конструктор не подойдет. При обращении к методу всегда нужны новые (другие, рандомные) параметры

[estry]

DanielMcRon, Экземпляр может быть один, а при обращении поля переназначаться. Так можно?

[freeExec]

estry, Можно всё что компилируется, но зачем о хранит двигатель отданный кому-то?

[estry]

freeExec, вот теперь понятно. Если в методе создавать, а потом отдавать, то экземпляр уничтожается?