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

[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

[sfy-y.livejournal.com]

А в чём разница?

[psilogic.livejournal.com]

а при чем тут ошибки? :)

[sfy-y.livejournal.com]

Упс. Ну, совсем по-быстрому подробно не объясню.

Любая нештатная ситуация легко обрабатывается катчем. А лучше книжки почитать.

[psilogic.livejournal.com]

В этом коде нет никакой нештатной ситуации. Просто юзер в редакторе захотел выполнить команду undo или redo.

[sfy-y.livejournal.com]

Ой, мама. Всё, уровень понял.

[sfy-y.livejournal.com]

Ну, ох 2 раза.

catch команду Undo {
делать Undo
}

[psilogic.livejournal.com]

Команда Undo не является нештатной ситуацией.

[sfy-y.livejournal.com]

Ну божешка ты ж мой. Просто event-driving. А по жизни - является.

[psilogic.livejournal.com]

а catch не является событием (event)

кроме вас, остальные комментаторы суть ухватили

продОлжите строить из себя умного, или, может, хватит? :)

[sfy-y.livejournal.com]

Сути не было. Была ошибка в дизайне.