【Bad】
def GetPage(url) :    //インターネットからページを取得                        
……

【Good】  
def FetchPage(url) :  //もしくはDownloadPage                          
……

var euclidean_norm = function (v) {
      var retval = 0.0;
      for (var i = 0; i < v.length; i += 1) 
            retval += v[i] * v[i]; 
      return Math.sqrt(retval); 
}; 

var euclidean_norm = function (v) {
      var sum_squares = 0.0;
      for (var i = 0; i < v.length; i += 1) 
            sum_squares+= v[i] * v[i]; 
      return Math.sqrt(retval); 
}; 

#define DISALLOW_EVIL_CONSTRUCTORS(ClassName) \ 
      ClassName(const ClassName&); \
      void operator=(const ClassName&); 
   
class ClassName { 
   private: 
      DISALLOW_EVIL_CONSTRUCTORS(ClassName); 
   public: ... 
}; 

#define DISALLOW_COPY_AND_ASSIGN(ClassName) ... 

【Bad】
var start = (new Date()).getTime(); // ぺージの上部
...
var elapsed = (new Date()).getTime() - start; // ページの下部 
document.writeln(" 読み込み時間:" + elapsed + " 秒 "); 

【Good】
var start_ms = (new Date()).getTime(); // ページの上部
...
var elapsed_ms = (new Date()).getTime() - start; // ページの下部 
document.writeln(" 読み込み時間:" + elapsed_ms / 1000 + " 秒 ");

if (debug) { 
    map<string,int> m; 
    LookUpNamesNumbers(&m); 
    Print(m); 
} 

【Good】
evaluation → eval
document   → doc
string     → str

【Bad】
BackEndManager → BEManager

ConvertToString() → ToString()  
DoServeLoop()     → ServeLoop()

results = Database.all_objects.filter("year <= 2011")

【Bad】
#textの最後を切り落として、「...」をつける 
def Clip(text, length):
... 

【Good】
def Truncate(text, max_chars):
…

public double getMean() { 
         // 全てのサンプルをイテレートして、total / num_samplesを返す。
}  
   ... 
} 

【Bad】
DatabaseConnection database_connection;
string error;
assert(ExpandFullName(database_connection, "Doug Adams", &error) == "Mr. Douglas Adams");
assert(error == "");
assert(ExpandFullName(database_connection, " Jake Brown ", &error) == "Mr. Jacob Brown III");
assert(error == "");
assert(ExpandFullName(database_connection, "No Such Guy", &error) == ""); 
assert(error == "no match found"); 
assert(ExpandFullName(database_connection, "John", &error) == ""); 
assert(error == "more than one result"); 

【Good】
CheckFullName("Doug Adams", "Mr. Douglas Adams", ""); 
CheckFullName(" Jake Brown ", "Mr. Jacob Brown III", ""); 
CheckFullName("No Such Guy", "", "no match found"); 
CheckFullName("John", "", "more than one result"); 
void CheckFullName(string partial_name, 
                   string expected_full_name, 
                   string expected_error) {
// database_connectionはクラスのメンバになっている。 
string error;
string full_name = ExpandFullName(database_connection, partial_name, &error); 
assert(error == expected_error);
assert(full_name == expected_full_name); 
} 

【Bad】
CheckFullName("Doug Adams", "Mr. Douglas Adams", ""); 
CheckFullName(" Jake Brown ", "Mr. Jacob Brown III", ""); 
CheckFullName("No Such Guy", "", "no match found"); 
CheckFullName("John", "", "more than one result"); 

【Good】
CheckFullName("Doug Adams" , "Mr. Douglas Adams"  , ""); 
CheckFullName("Jake Brown" , "Mr. Jacob Brown III", ""); 
CheckFullName("No Such Guy", ""                   , "no match found"); 
CheckFullName("John"       , ""                   , "more than one result"); 

【Bad】
#ユーザのメール帳をインポートして、システムのユーザと照合する。 
#そして、まだ友達になっていないユーザの一覧を表示する。 
def suggest_new_friends(user, email_password):
    friends = user.friends()
    friend_emails = set(f.email for f in friends)
    contacts = import_contacts(user.email, email_password) contact_emails = set(c.email for c in contacts)
    non_friend_emails = contact_emails - friend_emails
    suggested_friends = User.objects.select(email__in=non_friend_emails) 
    display['user'] = user
    display['friends'] = friends
    display['suggested_friends'] = suggested_friends
    return render("suggested_friends.html", display) 

【Good】
def suggest_new_friends(user, email_password): 
    #ユーザの友達のメールアドレスを取得する。 
    friends = user.friends()
    friend_emails = set(f.email for f in friends) 
    
    #ユーザのメールアカウントからすべてのメールアドレスをインポートする。 
    contacts = import_contacts(user.email, email_password)
    contact_emails = set(c.email for c in contacts) 
    #まだ友達になっていないユーザを探す。
    non_friend_emails = contact_emails - friend_emails
    suggested_friends = User.objects.select(email__in=non_friend_emails) 
    #それをページに表示する。
    display['user'] = user
    display['friends'] = friends 
    display['suggested_friends'] = suggested_friends 
    return render("suggested_friends.html", display)

【Bad】
// 与えられた subtree に含まれる name と depth に合致した Node を見つける。 
Node* FindNodeInSubtree(Node* subtree, string name, int depth); 

// このデータだとハッシュテーブルよりもバイナリツールの方が40%速かった
// 左右の比較よりもハッシュの計算コストの方が高いようだ

// TODO: もっと高速なアルゴリズムを使う
// TODO: JPEG以外のフォーマットに対応する

NUM_THREADS = 8  # 値は「>= 2 * num_processors」で十分

// メールを送信する外部サービスを呼び出している(1分でタイムアウト) 
void SendEmail(string to, string subject, string body); 

// intはCategoryType。                         
// pairの最初のfloatは’score’。    
// 2つ目は’weight’

//CategoryType -> (score, weight)

【But】
//データをキャッシュに入れる。ただし、先にそのサイズをチェックする

【Good】
//データをキャッシュに入れる。ただし、先にデータのサイズをチェックする
//データが十分に小さければ、それをキャッシュに入れる

【But】
#これまでにクロールした URL かどうかによって優先度を変える。 

【Good】
#これまでにクロールしていない URL の優先度を高くする。 

【But】
// 'src' の先頭や末尾にある 'chars' を除去する。 
String Strip(String src, String chars) { ... } 

【Good】
// 実例: Strip(“abba/a/ba”, ”ab”)は”/a/”を返す　 
String Strip(String src, String chars) { ... } 

【Bad】
// listを逆順にイテレートする
for (list<Product>::reverse_iterator it = products.rbegin(); it != products.rend(); 
++it) 
DisplayPrice(it->price); 
...

【Good】
// 値段の高い順に表示する
for (list<Product>::reverse_iterator it = products.rbegin(); it != products.rend(); 
++it) 
DisplayPrice(it->price); 
...

【Bad】
Connect(10, false); 

【Good】
// 引数にコメントをつけて関数を呼び出す
Connect(/* timeout_ms = */ 10, /* use_encryption = */ false);

【But】
// このクラスには大量のメンバがある。同じ情報はデータベースにも保管されている。ただし、
// 速度の面からここにも保管しておく。このクラスを読み込むときには、メンバが存在してい 
// るかどうかを先に確認する。もし存在していれば、そのまま返す。存在しなければ、データベー
// スから読み込んで、次回のためにデータをフィールドに保管する。 

【Good】
// このクラスの役割は、データベースのキャッシ層である。

【But】
// 所在地から余分な空白を除去する。それから「Avenue」を「Ave.」にするなどの整形を施す。 
// こうすれば、表記がわずかに違う所在地でも同じものであると判別できる

【Good】
// 所在地を正規化する (例: “Avenue” -> “Ave.”)。