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

[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]

А catch - не канает?

Любой подобный наворот говорит о неправильном дизайне. Просто ловить ошибки и отрабатывать.

[psilogic.livejournal.com]

да при чем тут ошибки?
обычный undo/redo в редакторе

[sfy-y.livejournal.com]

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

[psilogic.livejournal.com]

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

[sfy-y.livejournal.com]

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

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

[psilogic.livejournal.com]

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

[sfy-y.livejournal.com]

Где там у Вас в коде выполняется seek+read или read+seek?

[sfy-y.livejournal.com]

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

[sfy-y.livejournal.com]

Ну, ох 2 раза.

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

[psilogic.livejournal.com]

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

[psilogic.livejournal.com]

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

[sfy-y.livejournal.com]

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

[sfy-y.livejournal.com]

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

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

[psilogic.livejournal.com]

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

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

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

[psilogic.livejournal.com]

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

[sfy-y.livejournal.com]

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

[sfy-y.livejournal.com]

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

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

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

Копипаста плоха тем, что поддерживать ее намного труднее, поэтому от нее нужно избавляться по возможности.

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

Упустили момент, когда ни read, ни seek не нужны.

[3seemingmonkeys.livejournal.com]

ну еще проверки поставить елси такой случай возможен (мне было видение что ето не надо обрабатывать)

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

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

Итого, копипасту уполовинили, читаемость кода - тоже :)

[kindex.livejournal.com]

for (int i = 0; i <= 1; i++)
{
if (!!i != !!undoCase && (op & OpBefore))
{
uf.read(&hb, sizeof(hb));
*hNew= hb;
++hNew;
++i;
}
if (!!i == !!undoCase && (op & OpAfter))
{
uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass redo data
++hOld;
}
}

[kindex.livejournal.com]

!! - это приведение к bool.

[kindex.livejournal.com]

Вкралась ошибка - OpBefore и OpAfter используется неправильно.
первый (op & OpBefore) надо заменить на (op & (undoCase ? OpBefore : OpAfter)), и второй & аналогично.

[os80.livejournal.com]

Я ваш богомерзкий язык знаю плохо, но, если все "логические" переменные считать нормальными булевскими, то будет что-то вроде этого:

void OrFirstOrSecond(boolean First, boolean DoOrNot) {
if (First && DoOrNot) {
uf.read(&hb, sizeof(hb));
*hNew= hb;
++hNew;
++i;
}
else if (DoOrNot) {
uf.seek(sizeof(GHarmonicBase), SEEK_CUR);//pass redo data
++hOld;
}
}

И Ваш код:
OrFirstOrSecond(undoCase, op && OpBefore);
OrFirstOrSecond(!undoCase, op && OpAfter);

(никогда в жизни такого говна в своём коде не напишу, но как решение олимпиадной задачи - почему нет?