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

[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;
}

[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

[racoonbear.livejournal.com]

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

[3seemingmonkeys.livejournal.com]

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

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

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

Усё.

[sfy-y.livejournal.com]

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

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

[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;
}
}

[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);

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

[boldcat.livejournal.com]

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