Есть такой фрагмент кода:



где X, Y и Location - свойства.
Как бы его можно было оптимизировать?
Думал сначала использовать шаблонную функцую swap, но .NET сабака не дает properties передавать по ссылке. А передавать их по значению смысла мало. Как быть?

versus, оптимизировать - понятие относительное. Если X, Y и Location везде по коду связаны, то вынеси их в отдельный класс/структуру. И работай с ними как с цельным объектом. Если не связаны - то метод swap у тебя странно написан. Зачем создавать Cell tmp, если можно обменять свойства без нее? Да и вообще, сам смысл существования этого метода вызывает сомнения - совершенно непонятно что же он делает.

Это сообщение отредактировал(а) PashaPash - 19.9.2008, 01:36

Прошу прощения, за туманность формулировки.
Попробую описать ситуацию более подробно.

Я пишу простую программу-игру типа "пятнашек" (ну или "осьмушек" http://www.rsdn.ru/article/multimedia/octals.xml).  
Сначала пользователю показывается поле (матрица) из кнопок в определенной последовательности (например, дни недели). 
Затем, пользователь тыкает в какую-нибудь кнопку и кнопки перемешиваются в случайном порядке, причем пользователь видет процесс перемешивания, то есть кнопки меняются местами с некоторой задержкой.
После перемешивания пользователь должен восстановить исходную последовательность нажимая на соответсвующие кнопки (понедельник, вторник, среда, ...).

В процессе перешивания у каждой кнопки есть 5 параметров, которые мне интересны:
ширина и высота (эти параметры можно было бы игнорировать, если они были одинаковы для всех кнопок, но поскольку я увеличиваю крайние кнопки, чтобы они заполнили все игровое пространство, то эти параметры могут отличаться. Пример: игровое поле 10x10, кнопок 9, следовательно крайние кнопки должны быть на 1 пиксель больше остальных, чтобы заполнить все игровое поле).
позиция на игровом поле (чтобы можно было менять их позициями других кнопок)
ну и точки x, y  задающие положение кнопки в исходной матрице, чтобы в событие Paint я мог по этим параметрам правильно расположить кнопки.

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

Итак, чтобы переставить кнопки местами, мне нужно поменять у них 5 параметров.
Под оптимизацией в данном случае я имел, что мне хотелось бы получить фрагмент вроде:
swap(this.Location, c.Location)
swap(this.x, c.y)
swap(this.y, c.y)
...

и не писать один и тот же кусок кода для всех 5 переменных.

x, y - это просто открыте переменные, с ними проблем не возникает
Location, Height и Width - это свойства (property), их передать в качестве ссылки в метод swap .Net не разрешает.

Как быть?


Это сообщение отредактировал(а) versus - 20.9.2008, 00:51

versus, взять любую книгу по ооп и внимательно прочитать. У тебя проблемы с дизайном - с одной стороны пытаешься привязать cell к определенному месту, с другой - очень хочешь подвигать их по полю. Что делает метод swap? меняет местами ячейки? Ячейки - это часть игрового поля. Выпиливать их из него только ради того, чтобы поменять местами фишки, действительно трудоемко.

Введи отдельно ячейки (дырки), с фиксированной позицией. Составь из них матрицу. У ячеек будут фиксированные позиции, привязанные к матрице. Для каждой можно узнать 4 соседние. - это то, что у тебя сейчас под Cell подразумевается.
Потом введи сами фишки (Token) - что-то, вообще не имеющее представления о координатах. Возьми 15-ки, выковыряй одну фишку и посмотри. Должен получится соответствующий ей объект - никаких координат, только какая-то надпись на нем.
Потом сделай в Ячейке-Cell свойство, показывающее какая фишка в ней сейчас лежит. И тогда сможешь перекладывать их в 3 строчки, без "оптимизации". Вот примерно так:

Спасибо, помогло.
Сделал у Cell вложенный класс Token с полями text и id. Text - для отображения, ID - идентификатор по которому я отслеживаю правильно ли пользователь выбрал очередной элемент последовательности.
Соответсвенно метод swap у Cell стал таким:



Этого мне вполне достаточно.