程序代码优化，大家看看

[qyh1211]

类似这样的代码:

if(flagA)
{
 doX();
}
else
{
 if(flagB)
 {
  whlie(flagC)
  {
   doY();
   doX();
  }
 }
 else
 {
  doZ();
  doX();
 }
}

上面的doX()其实是一大段长的代码，我不想写成函数。所以怎么样才能使上面的doX()只出现一次,或者少出现一次也行..不知道行不行..

[太乙]

什么叫只出现一次？？

这跟代码优化没关系吧？？

lz是想减少代码量？？？

函数调用反而会让效率下降（一般情况下）因为涉及到堆栈、断点、现场保存等@！

不懂lz要干嘛？

[lin_style]

if(flagA)
{
 doX();
}
else
{
 if(flagB)
 {
  whlie(flagC)
  {
   doY();
  }
 }
 else
 {
  doZ();
 }
  doX();
  doX();
}

[lin_style]

错了。
因为你
第一个IF配套的ELSE里 doX()必须要执行一次，可以放出来。如果你的doX()和doY()之流关联不大的话，完全可以只执行一次。当然，要看你的doY的函数怎么写了

if(flagA)
/*
 这种就算和doY,doX有关系也可以减少一次。
*/
{
 doX();
}
else
{
 if(flagB)
 {
  whlie(flagC)
  {
   doY();
  }
 }
 else
 {
  doZ();
 }
  doX();
}

[Qlaiaqu]

有意思

[tangshuiling]

如果dox、doy、doxz模块之间无任何耦合性，可单独把dox函数提到if语句块外，if快内的dox统统不要

[lin_style]

啊啊啊 。
没看到while..我错了。。

if(flagA)
{
 //取消
}
else
{
 if(flagB)
 {
  whlie(flagC)
  {
   doY();
   doX();
  }
 }
 else
 {
  //取消
  doZ();
 }
}

doX();//放这

应该行得了吧。。

[lingyin55]

随便写一个，没有检查

if(flagB)
{
  whlie(flagC)
  {
   doY();
   doX();
  }
}
else
{
   if( !flagA )  
   { 
      doZ();
   }
   doX();
}

[Qlaiaqu]

我看来，这个已经是最简化的了

if(flagA)
{
 doX();
}
else
{
 if(flagB)
 {
  whlie(flagC）//这里的循环就导致了doX();不能放到外边去，因为难以保证调用次数，不过你要是贴出全部代码，可能还能改一下，否则变数大了
  {
   doY();
   doX();
  }
 }
 else
 {
  doZ();
  doX();
 }
}

[chin_chen]

if(flagA)
{
	//移出去
}
else
{
	if(flagB)
	{
		whlie(flagC)
		{
			doY();
			doX();
		}
	}
	else
	{
		doZ();
		//移出去
	}
}

if(!flagB)
doX();

[ctan]

[Quote=引用 9 楼 chin_chen 的回复:]
C/C++ code
if(flagA)
{
//移出去
}
else
{
if(flagB)
{
whlie(flagC)
{
doY();
doX();
}
}
else
{
doZ();
//移出去
}
}

if(!flagB)
doX();
[/Quote]

有问题, 如果条件为(flagA)&&(flagB)时, 出现错误!

[chin_chen]

if(flagA)
{
    //移出去
}
else
{
    if(flagB)
    {
        whlie(flagC)
        {
            doY();
            doX();
        }
    }
    else
    {
        doZ();
        //移出去
    }
}

if(!flagC)//刚刚以为是互斥的。那这里用这个就行了撒
doX();

[ctan]

楼上代码都不行,

建议楼主将doX()部分代码写到一个.h文件中, 在需要调用的地方用#include包含进去

[chin_chen]

if(flagA)
{
	//移出去
}
else
{
	if(flagB)
	{
		whlie(flagC)
		{
			myflag=1;//加一个标志符
			doY();
			doX();
		}
	}
	else
	{
		doZ();
		//移出去
	}
}

if(myflag!=1)//这个就可以了
doX();

[chin_chen]

myflag初始为0

[liusichen_0]

这个不能叫代码优化，只能说是简化代码

[KevinYuen]

	if( !a && !b )
	{
		doZ();
	}

	if( !b )
	{
		doX();
	}
	else
	{
		while( c )
		{
			doY();
			doX();
		}
	}

[dongpy]

列出flagA、flagB、flagC的真值表：
A B C doX
0 0 0 1
0 0 1 1
0 1 0 0
0 1 1 1
1 0 0 1
1 0 1 1
1 1 0 1
1 1 1 1
可见除了（!A&&B&&!C），都执行doX，因此简化：
if( A||!B||C )
{
do
{
doX();
if( A||!B )
break;
}whlie(flagC);
}
if( A||!B ）
{
whlie(flagC)
{
doY();
}
}
if( !A&&!B ）
{
doZ();
}

[chin_chen]

再来一个

if(flagA)
{
	myflag=1;//加一个标志符
}
else
{
	if(flagB)
	{
		if(flagC)
		{
			myflag=2;//加一个标志符

		}
	}
	else
	{
		myflag=3;//加一个标志符

	}
}


{


	if(myflag==2)
	{
		doY;
	}
	if(myflag==3)
	{
		doZ();
	}
	if(myflag==1)
		;
	doX();
}while(flagC&&myflag==2)

[KevinYuen]

[Quote=引用 18 楼 chin_chen 的回复:]
再来一个
C/C++ code
if(flagA)
{
myflag=1;//加一个标志符
}
else
{
if(flagB)
{
if(flagC)
{
myflag=2;//加一个标志符

}
}
else
{
myflag=3;//加一个标志符

}
}


{


if(myflag==2)
{
doY;
}
if(myflag==3)
{
doZ();
}
if(myflag==1)
;
doX();
}…
[/Quote]

太多次的判断