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

[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

[psilogic.livejournal.com]

уровень понял :)

[sfy-y.livejournal.com]

Увидел. Так это ошибка by design.

Пойдете с электронным паспортом через контроль - кивните, стэк драйверов там мой. :)

[psilogic.livejournal.com]

"стек драйверов" - какие слова знаете :))))

[sfy-y.livejournal.com]

Так случилось. Я и Кнута читал, и Оллонгрена, и Мартина, и много кого ещё. Когда-то писал ЦУПы (6 штук). В 2005-2006 от скуки написал драйверы для электронных паспортов (по заказу ГУП Атлас).

Шли бы Вы со своими редакторами? :)))

[boldcat.livejournal.com]

Где у Кнута написано, что по можно посмотреть на названия переменных и переписать логику?

[sfy-y.livejournal.com]

А в каком словаре написано, что fat и bold - синонимы?

[boldcat.livejournal.com]

:) :) :) :)
А я никнейм не по словарю подбирал :)
Желаю Вам успехов!

[sfy-y.livejournal.com]

:))) И Вам.