目前项目的暴强代码风格 + 把它重构了！

3. 1   选中下面的这段代码，然后 alt + shift + m。 起个名字，叫。。。呃。。。。
setTransitionPathOrEndOrSignalAnInstanceIfIsTransition吧。
我想表达的意思是：当 isTransition的时候，setTransitionPath或者signal()或者end()一个instance...

Java代码
if (isTransition)  
        if (!instance.hasEnded()) {  
            if (transitionPath != null 
                    && !"".equals(transitionPath))  
                instance.signal(transitionPath);  
            else 
                instance.signal();  
        } else {  
            instance.end();  
        }  
    } 

--------------
按照你的描述，我会这么写：
proc_inst(struct instance inst, int is_trans)

如果说doProcessIfParentOfTaskIsTaskNode还勉强能看懂
这句话我是无论如何没法从字面上猜出意思来的：isShouldDouldDoProcess

我个人在编码的时候是最不喜欢看到单词简写的方法。而且java代码规范是
骆驼表示法，所以我一直都是这样用的。

如果有维护人员看到我的代码骂我也没关系，我在项目结束之后都会写个
比较详细的文档，注明如果读者有任何问题可以随时跟我联系。


呃。。。我写错了！
我还纳闷呢。。。Dould 是啥意思。。。。
我当时是想命名成： isShouldDoProcess ，也就是 是否应该xx的意思。

改过来了      谢谢提醒！！！！

doProcessIfParentOfTaskIsTaskNode
要是我来写，这个直接写成processTask，方法里面具体是怎么判断的，打开方法看代码就好了。我觉得没必要把一个方法里判断的条件都用方法名描述出来。

那个isShouldDoProcess改成shouldBeProcessed会不会好些？

那么package要是有两个单词怎么整?
下划线?

org.aopalliance.aop (AOP Alliance)
org.apache.commons.beanutils (apache-commons)
org.dom4j.datatype (dom4j)
org.apache.hivemind.methodmatch (Hivemind)
org.hibernate.bytecode.javassistr (Hibernate)
org.springframework.stereotype (Spring)
org.apache.tapestry.scriptaculous (Tapestry)
javax.management.modelmbean (JBoss ?)
org.jfree.chart.imagemap (JFreeChart)
org.acegisecurity.securechannel (Acegi)

我认为，重构中，最重要的一项是：“命名”。修改函数的名称，修改变量的名称，抽取函数、分解函数后，同样需要给新的函数命名。
不理解程序，你如何命名？

如果重构的目的，仅仅是为了保持代码的行为不变，当然只需要考虑“形式化的过程”。

我只想说，你这步就做得很差劲
像这么简单的程序，打开Eclipse或者IDEA就直接开始搞，一边重构一边就能看懂了
重构首先是一个形式化的过程：如果必须把代码看懂了才能重构的话，那么你就会让自己陷入一个悖论，因为最需要重构的代码你是看不懂的
你接着做吧…我只是想告诉你，有时候其实是五十步笑百步而已