Пишем игровую логику на C#. Часть 2/2

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

Можете, пожалуйста, поуточнять? Что имеете ввиду под цепочками? У меня команда может вызывать другую команду — чем не цепочка. Для вас система недостаточно сложная? Просто для данной игры более сложная система не нужна. Вот когда делал Генералы — там было сложнее и команды были иерархическими, у команд были дети, сложная система событий, которая позволяла подписаться до команды, в середине команды, после команды, более сложная валидация. Каждому приложению ровно такая сложность, которая необходима, но не больше.

О да, я забыл, ведь создание файлов — это столь сложная операция. Гораздо удобнее каждый раз рыться в файле на тысячи строк. Поиском? Главное преимущество файлов — значительно легче искать то, что необходимо. Когда у тебя весь необходимый код вмещается на одном экране и другой экран просто не нужен) Один файл на 50-100 строк, файлы структурированы. Можно открыть несколько вкладок. Код, который сейчас не нужен — не путается под ногами. Вот мне необходима функциональность А и Г, а между ними — Б и В. Так я просто не открыл лишнее, а только два файла в разный вкладках — а так метаться постоянно видя лишний код.

И какие причины НЕ создавать файлы? Сплошные плюсы ведь)

Изначально у меня было что-то вроде такого:

public enum Status {
	Success,
	
	RoomNotEmpty,
	NotEnoughItems,
	CrewMemberIsFreezed
}


public abstract class Command
{
	public Core Core { get; private set; }
	public bool IsValid { get; private set; }
	public Status Status { get; private set; }

	public Command Execute (Core core)
	{
		Core = core;
		Status = Run();
		IsValid = (Status == Status.Success);
		return this;
	}

	protected abstract Status Run ();
}

И тогда каждая команда при ошибке возвращает не false, а код ошибки. Как часть сервера работает прекрасно — можно и узнать все возможные ошибки и сериализуется легко.

protected override Status Run ()
{
	if (Room.Building.Type != BuildingType.Empty) {
		return Status.RoomNotEmpty;
	}
	if (Building.Type == BuildingType.Empty) {
		return Status.CantBuildEmpty;
	}

	if (!new Pay(Building.Config.ConstructionCost).Execute(Core).IsValid) {
		return Status.NotEnoughResources;
	}

	Room.Building = Building;
	return Status.Success;
}

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