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

[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

[racoonbear.livejournal.com]

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

[psilogic.livejournal.com]

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

[racoonbear.livejournal.com]

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

[psilogic.livejournal.com]

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

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

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

[racoonbear.livejournal.com]

Спасибо, капитан очкевидность. Но реюз одной строчки может быть не борьбой с копипастой, в затуманиванием смысла. Общая парадигма гласит - выносите в функции или отдельные места (отдельные переменные) то, что по смыслу обрабатывается схоже. А если схожесть случайная, то не надо выносом наверх или в функцию её за уши притягивать.