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

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

[lionet.livejournal.com]

Это псевдокод, вестимо.

[psilogic.livejournal.com]

порядок seek/read важен :)

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

[psilogic.livejournal.com]

для файла seek, read не равно read, seek

[3seemingmonkeys.livejournal.com]

выходит надо выполнить
seek
read

или

read
seek

тут или копипаста или goto третьего не дано имхо.

[afa-at-work.livejournal.com]

вынос в методы. и не заморачиваться.

[psilogic.livejournal.com]

На практике - да, но можно рассмотерть как "олимпиадную задачку"...

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

Вывод в методы, или языческое goto

[psilogic.livejournal.com]

О! Насчет языческого я и забыл... :)

[racoonbear.livejournal.com]

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

[psilogic.livejournal.com]

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

[3seemingmonkeys.livejournal.com]

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

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

[psilogic.livejournal.com]

неплохо!

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

Упустили момент, когда ни read, ни 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)

Усё.

[psilogic.livejournal.com]

Ничего не понял...
Одно ясно, что это неверно, т.к. в некоторых случаях может понадобится seek+read или read+seek, а у вас - только read либо только seek

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

[kindex.livejournal.com]

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

[kindex.livejournal.com]

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

[psilogic.livejournal.com]

угу, for как goto :)

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

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

[psilogic.livejournal.com]

:)))

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