看看一帮人成天在干什么事

zylthinking

。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。

Here are my comments,
1. Rename n, newText2 with meaningful names. It is one of rules of code convention.
2. Add some comments for your fix.
3. Remove ASSERT(EFalsle).
。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。。

1. 有意义的名称， 妈的也就是一个临时变量，  TInt n = newText.Length(); 非要命名成 int length_of_newtext 才可以吗， 另一个大概应该命名成 copy_of_newtext
2. 公式化的写法， 指导思想是加一行代码写一行注释， 只要不写， 就他妈的有人challenge你
比如 TInt n = newText.Length();  怎么写， 恩 应该写 // assign the length of newText to n, 哦， sorry, 应该是 // assign the length of newText to length_of_newText
操他妈的
3. 移除 ASSERT, 大概闲的疯了

gosapphire

老实说，我觉的提得comments没什么不好…………

pengjianbokobe

楼主要冷静啊

zzyong08

1. 我同意，有意义的名字，也就是多敲了几个字符而已。
2. 我不同意。代码就是最好的注释，冗余的注释，只会导致维护负担，和二义性。
3. 删除 assert 是不适宜的。

tyc611

你再challenge他呗

zylthinking

不在于是不是所谓有意义的命名好或者不好， 而是在这样一些细节上一个个坚持着所谓的职业素养， 比如firstCharOfNewText， 在这种情况下不知道所谓的有意义命名有什么意义。
一个个如此的高素养， 不知道为什么这么多年一个软件仍然 bug 满天飞， 结构看不懂； 到处都是上千行的函数； 费尽力气终于搞明白一点来龙去脉， 做点改动， 以彻底杜绝某方面的错误， 却被叫停， 说是改动太大了， 怕出什么错误， 其判断方法之一是数改动的行数， 如果超过100行， 就有可能； 剩下的合理些。 稍微看看改动的内容， 却绝不考虑这样改的原理， 只是考虑其中部分代码段在程序中的影响， 影响有点大， 也会被叫停。。。。。。。。。。。。。。

最终如下的代码通过了， 这是个见仁见智的问题， 我自己觉得有些幽默   


    TInt lengthOfNewText = newText.Length();
    if (lengthOfNewText == 0)
    {
        ASSERT(EFalse);
        iSpellcaseContext = iCaseMan->CurrentCase();
    }
    else
   {
        TChar firstCharOfNewText(newText[0]);
        if (firstCharOfNewText.IsLower())
        {
                iSpellcaseContext = EAknEditorLowerCase;
        }
        else if (lengthOfNewText > 1)
        {
                TBuf<EMaximumFepWordLength> copyOfNewText = newText;
                copyOfNewText.UpperCase();
                iSpellcaseContext = newText.Compare(copyOfNewText) ?
                                                                                        EAknEditorTextCase : EAknEditorUpperCase;
        }
        else
        {
                iSpellcaseContext = (iCaseMan->CurrentCase() == EAknEditorUpperCase) ?
                                                                                EAknEditorUpperCase : EAknEditorTextCase;
        }
   }

论坛ID

lz淡定阿，谁让他是头呢

bukkake

形式化害人阿，公司里只用软件检查代码形式，然后就出现了把常数6定义为SIX的情况