在C++論壇裡發了一個與本文題目相同的貼子如下：

下面是一個我在一個應用系統中發現的我認為很醜陋的一段C++程式碼.顯然,程式碼的意圖是將string中的內容變成相應的小寫字母.請大家指出它的問題之所在,並給出改善後的程式碼.我自已已經有了一個答案,只用一條語句就能實現這個功能.待兩日後我給出我的答案.但然,也許你的答案比我的更好.
void IniReader::string_lower(string& str)
{
    int size=str.length();
    string temp;
    char c;
    for(int i=0;i <size;i++)
    {
        c=str.at(i);
        if(c>=65&&c <=90)
        {
            c+=32;
        }
        else
        {
            continue;
        }
        temp=c;
        str.replace(i,1,temp);
    }
}

我給出的改善的程式碼是：

# include  <string>
inline std::string& string_tolower(std::string& str)
{
   std::transform(str.begin(), str.end(), str.begin(), std::tolower);
   return str;
}

將函式的介面做了調整。這個函式沒有涉及到任何的類的變數，顯然不應該屬於類函式。將修改後的string作為返回值，將使得使用起來更加方便。例如可以如下使用：

std::cout << string_tolower("Hello, UPPER string TO LOWER STRING");

函式的實現使用了stl的模版函式transform實現，這就是我說的用一條語句就實現的地方。首先跟貼的那個表示懷疑的朋友現在是否明白了。

海嘯朋友最先給出transform的解答，其他幾個朋友都寫出了transform的改善方法。基本上符合了我的答案。細節的地方做個說明：那個tolower函式最好加上std::的字首修飾，因為有些系統將tolower定義成了巨集，缺少std的修飾說明，有可能會編譯出錯。另外，這個函式由於簡短，適合於放在標頭檔案中，作為inline函式。

朋友們在注重改善實現的時候，忘記了我還還有一句：請大家指出它的問題之所在。

在這裡我舉出下面幾點：

• 那麼temp = c 將構成string類的物件，並且在迴圈體內，將嚴重影響效率。而使用replace進行字元替換，顯然是小題大做。因為這裡要進行的是一個字元的更改，並不是將這個字元替換成一個字串。
• continue語句可以省去。雖然continue語句好於goto語句，但仍然屬於影響正常流程的跳轉語句。
• 使用數字65表示字元A，可讀性差。
• 格式風格的美觀可讀性差。比如 if(c>=65&&c <=90) 密密麻麻的擠壓在一起，很難看。寫成  if ( (c >= 64) && (c <= 90) )會更好看一些。

其實，我還可以舉出一些問題。先慢慢來吧。

我只所以說這個程式醜陋，並非是說它一定要用一句tranform來實現就算漂亮了。如果你還不熟悉STL模版庫。也可以寫成下面這樣：

for ( int i = 0; i < str.size(); i++ )
  str[i] = tolower(str[i]);

如果，你還不知道tolower這個函式，也可以寫成下面：

  int c = str[i];
  if ( (c >= 'A') && (c <= 'Z') )
    str[i] = c + ('a' – 'A');

這裡，str.size()由於是inline函式，放在迴圈判斷上不會影響程式的效率，並不需要像原來程式一樣，設一個size的中間變數。同樣，tolower也是inline函式，也不會增加程式的函式呼叫開銷。

如果看了我這篇的解釋，你還不以為然。那我到真希望你來維護一下我看到的這個系統：那可以延長到兩頁的迴圈體裡，佈滿了continue；那充滿了硬編碼的需要你從一堆程式碼中去尋找含義的魔數；那一個類裡面竟然設計了一大堆毫不相關的函式。實在的，我都有快崩潰的感覺。寫醜陋的程式碼，實際上是在給後來使用和維護程式碼的人挖陷阱。當然，跳進這個陷阱的有可能就是你自己。