Программерам

[psilogic]

Задачка: как оптимизировать такой код, чтобы убрать копи-пасты?
Напрашивается заведение двух функций... а, может, кто-нибудь знает другие трюки?

if (undoCase)
{
	if (op & OpBefore)
	{
		uf.read(&hb, sizeof(hb));
		*hNew= hb;
		++hNew;
		++i;
	}
	if (op & OpAfter)
	{
		uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass redo data
		++hOld;
	}
}
else
{
	if (op & OpBefore)
	{
		uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass undo data
		++hOld;
	}
	if (op & OpAfter)
	{
		uf.read(&hb, sizeof(hb));
		*hNew= hb;
		++hNew;
		++i;
	}
}

Upd: Порядок вызова seek/read важен - там в файле два последовательных блока, наличие/отсутствие которых определяется флагами OpAfter/OpBefore.

Comments

[lionet.livejournal.com]

if(!((op & opBefore) ^ undoCase) && !(op & opAfter)) {
uf.read(&hb, sizeof(hb));
*hNew= hb;
++hNew;
++i;
}
if(!((op & opAfter) ^ undoCase) && !(op & opBefore)) {
uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass undo data
++hOld;
}

[lionet.livejournal.com]

Это псевдокод, вестимо.

[psilogic.livejournal.com]

порядок seek/read важен :)

[3seemingmonkeys.livejournal.com]

както так может

bef = op & opBefore
aft = op & opAfter
if (undoCase & bef | !undoCase & aft)
{
uf.read(&hb, sizeof(hb));
*hNew= hb;
++hNew;
++i;
}
else
{
uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass redo data
++hOld;
}

[afa-at-work.livejournal.com]

вынос в методы. и не заморачиваться.

[http://users.livejournal.com/_stilgar/]

Вывод в методы, или языческое goto

[psilogic.livejournal.com]

О! Насчет языческого я и забыл... :)

[psilogic.livejournal.com]

для файла seek, read не равно read, seek

[psilogic.livejournal.com]

На практике - да, но можно рассмотерть как "олимпиадную задачку"...

[racoonbear.livejournal.com]

Ты не указал цель оптимизации - уменьшить зависимость от условных переходов, или увеличить скорость, или уменьшить количество строк кода \ вызовов, или улучшить читаемость или...

[psilogic.livejournal.com]

Убрать дублирующий код, не увеличивая общий объем кода.

[3seemingmonkeys.livejournal.com]

выходит надо выполнить
seek
read

или

read
seek

тут или копипаста или goto третьего не дано имхо.

[psilogic.livejournal.com]

похоже на то

[afa-at-work.livejournal.com]

один фиг достаточно. дальнейшая - читабельность ухудшит и сильно.

[3seemingmonkeys.livejournal.com]

можно убрать 1 дубль :)

if (s = !u & bef) {
seek
}
read;
if (!s) {
seek;
}

[psilogic.livejournal.com]

неплохо!

[racoonbear.livejournal.com]

звучит как подмена цели средством

[psilogic.livejournal.com]

возможно... но как упражнение - почему нет

[sfy-y.livejournal.com]

// op&OpBefore ==!(op&OpAfter) accepted


(undoCase&(!op&OpAfter))?(uf.read(&hb, sizeof(hb)),*hNew++= hb,++i;) :
uf.seek(sizeof(GHarmonicBase), SEEK_CUR),++hOld)

Усё.

[psilogic.livejournal.com]

Ничего не понял...
Одно ясно, что это неверно, т.к. в некоторых случаях может понадобится seek+read или read+seek, а у вас - только read либо только seek

[sfy-y.livejournal.com]

А совсем тупо - просто макрики или инлайны:

inline seek(){...}
inline read(){...}

[sfy-y.livejournal.com]

Там в начале дисклеймер, (предполагается, что программер умеет именовать переменные)

[psilogic.livejournal.com]

хрен поймешь этот дисклеймер

но OpBefore == 1, OpAfter == 2, а op == некая комбинация битов, включая OpBefore и OpAfter

[sfy-y.livejournal.com]

Тогда делать макрики и не париться.

Но одновременно before и after - не по моим мозгам. :)))


Скачайте как-нить сафроутер (ну, оттуда, например,http://as.fatal.ru/projects/route/index.htm ) - так программил я.

[psilogic.livejournal.com]

Это код из обработки undo/redo. В случае операции
1) удаления надо сохранить то, что было ДО удаления (для undo),
2) вставки надо сохранить то, что будет ПОСЛЕ вставки (для redo),
3) изменения надо сохранить то, что было ДО (для undo) и будет ПОСЛЕ (для redo) изменения
так и получается одновременно :)