代码优化5-if条件判断的陷阱

最近看到一个公司底层代码中这样写到

	public boolean match(FeatureSupportDO f) {
		boolean ret = false;
		if (null != f) {
			if (null != f.getFeature(this.key) 
					&& (null == this.value 
							|| this.value.equals(f.getFeature(this.key).getValue()))) {
				ret = true;
			}
		}
		return ret;
	}

这个if中有两次出现了“f.getFeature(this.key)”，那么java会不会执行两次这个方法呢？
我们写了一个测试类来检测一下：

public class Test {
	
	public static void main(String[] args) {
		A a = test.new A("1");
		if(a.getP() != null && (1!=1 || a.getP().equals("1"))) {
			System.out.println(a.getV());
		}
	}
	class A{
		private A a;
		private String p;
		int v = 0;
		public A() {
			
		}
		
		public A(String p) {
			this.p = p;
		}

		public String getP() {
			v ++;
			return p;
		}
		public int getV() {
			return v;
		}
	}
}

输出结果是 "2"
这说明if中多次出现的方法，并不会在编译期进行优化，所以当碰到这种情况时，我们做如下修改：

   public boolean match(FeatureSupportDO f) {
      if (null != f) {
         FeatureDO feature = f.getFeature(this.key);
         if (null != feature 
               && (null == this.value 
                     || this.value.equals(feature.getValue()))) {
            return true;
         }
      }
      return false;
   }

当然如果这个getFeature只是简单的return一个属性，那么这个修改的意义并不是很大。
偏偏这个基础类中的方法并非如此，实现中还有递归调用等处理。