Можно доработать условие добавления элемента, например если строка не инициализирована null
или пуста, то ячейку считать пустой. Здесь неоднозначность какую ячейку инвентрая считать пустой: null
или ""
? Обе?
Тогда условие проверки должно всегда проверять оба эти условия. Например условие для пустой ячейки можно реализовать так: if (string.IsNullOrEmpty(inventorySlots[i])) {...}
, а для не пустой вот так if (inventorySlots[i]?.Length > 0) {...}
.
Однозначности можно конечно достичь тем путем, который вы выбрали, избежать null
с помощью инициализации массива пустыми строками, в этом случае инициализацию можно упростить вот так:
public string[] inventorySlots = Enumerable.Repeat(string.Empty, 9).ToArray();
Посмотрите другие статические методы класса Enumerable
, там много всего полезного.
Затем можно оптимизировать сравнение if (inventorySlots[i].Length == 0)
.
Но я бы на вашем месте в инвентаре хранил бы не строки, а числа int[]
, и отдельно словарь с элементами инвентаря Dictionary<int, string>
, где по айдишнику элемента можно получить название элемента и возможно другие его характеристики. Так же, если вы захотите переименовать элемент, или использовать локализации, например интерфейс игры на русском и английском, тогда это не затронет сам инвентарь.
В случае с хранением айдишников int
- все будет вполне однозначно, 0
- пусто, или -1
- пусто, просто выберите, что вам больше по душе. К тому же в C# числовые операции сравнения работают значительно быстрее, чем строковые. А еще строка занимает больше памяти. Представьте 1000 инвентарей, хранящих строки, содержащие одни и те же элементы, зачем столько 9000 копий повторяющихся строк хранить в памяти, если можно хранить только по одному экземпляру каждой строки, и добывать ее с помощью числового индекса, хранящегося в инвентаре.
Подумайте над архитектурой того, что вы делаете. Чем раньше вы предусмотрите все возможные проблемы, тем легче будет потом разрабатывать приложение.
string.Empty
, то в массиве будут значения по умолчанию:null
.List<string>
.Sorter
. Вроде он ничего не сортирует, а добавляет.