rabbit-slide-kou-schoo-readable-code-2015-04 2015.4.3.0 → 2015.4.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 375f4283c021659fa0aa9cb050066b834f92e408
-  data.tar.gz: 5578a7088b61491271240141fd4d615131389516
+  metadata.gz: 23cc1c9aec53ccaba058272cdd69c2fecc4b952b
+  data.tar.gz: 64951a479946b23e222366824c9eaeb68a821b37
 SHA512:
-  metadata.gz: c20f9ac2013920950e0211b64d1b1c461cbb2c8ca1285442fb3b359953a618c3fa834e595b63f2a3faf455912f07e892eb1c3f576b130ec0e9b66fb31d1a3b3b
-  data.tar.gz: 0755f1997a03ab369158ddf0bb7b04b940bae5891a19f9c915b9490cbc1a7d3455233dc87411090b2787572d12a5b97d79af7d06d84ebed6ff1e9984f735fed6
+  metadata.gz: 281337d0b6c01dcafc7b5a8a4f174e0f2affb14f9dc7881bc8dba2b3aadeccfe8bc174e1e332e32a8628003ae6110913d9936bcd75f44de54b0649080e207807
+  data.tar.gz: 69822379c200cbc7275e67aab0083d08d405a3868c2abfd4977c1051d7041f0bf044b2ed6daf19f47fcdf26d6c9411ae5c827b397ff50f3142dbab3942ef0216

data/chapter-7.rab

@@ -255,6 +255,7 @@
 
   # coderay c
 
+  /* A            | B            */
   if (a == b) {   | if (a != b) {
     /* ケース1 */ |   /* ケース2 */
   } else {        | } else {
@@ -264,6 +265,20 @@
 (('tag:center'))
 どちらがリーダブル？
 
+= 7.2 例：処理の順番
+
+  # coderay c
+
+  /* A            | B            */
+  if (a == b) {   | if (a != b) {
+    /* ケース1 */ |   /* ケース2 */
+  } else {        | } else {
+    /* ケース2 */ |   /* ケース1 */
+  }               | }
+
+(('tag:center'))
+A（左）の方がリーダブル
+
 = 7.2 指針
 
   * 否定形は肯定形にして書く
@@ -297,7 +312,7 @@
 
   # coderay cpp
 
-  // 関心が引く条件「expand_all」を優先
+  // 関心を引く条件「expand_all」を優先
   // （肯定形になったのはおまけ）
   if (url.HasQueryParameter("expand_all")) {
     for (int i = 0; i < items.size(); i++) {
@@ -364,10 +379,11 @@ ifとreturnの使い方\n
 
 = 番外：パスとリーダブル
 
-  * 処理を実行するときの流れ
-  * コードを読むときの流れ
-    * 読みやすい流れにすれば\n
-      リーダブルになる！
+  * パス
+    * 処理を実行するときの流れ
+    * コードを読むときの流れでもある
+  * 読みやすい流れ
+    * →リーダブル！
 
 = 番外：例（1）
 
@@ -392,8 +408,9 @@ ifとreturnの使い方\n
       # コメント追加しないの！？
       report_access_error
     else
-      # ネストされているけど正常系の処理か
-      # ifの方と同列の処理かと思った
+      # else（その他）の方の処理だけど
+      # このメソッドにとって大事な処理か
+      # ifの方と同列の大事さの処理かと思った
       comment = Comment.create(text)
       post.add(comment)
     end
@@ -406,12 +423,12 @@ ifとreturnの使い方\n
   # コメントを追加するメソッドだな！
   def add_comment(post, user, text)
     if post.hidden_from?(user)
-      # ネストされているし特別な処理なんだな
+      # すぐにreturnしているし特別な処理なんだな
       report_access_error
       return
     end
 
-    # 通常はコメントが追加されるんだな！
+    # 通常はコメントを追加するんだな！
     comment = Comment.create(text)
     post.add(comment)
   end
@@ -443,11 +460,13 @@ ifとreturnの使い方\n
   def prepare_database(path)
     if not File.exist?(path)
       # なかったら作るのか
+      # あれ、作るのも普通の「準備」なような…
       return Database.create(path)
     end
 
     # あったら開くのか
-    # あれ、どっちも正常系じゃない？
+    # やっぱりどっちも「準備」だよなぁ
+    # なにか特別な意図があるんだろうか…
     Database.open(path)
   end
 
@@ -469,7 +488,8 @@ ifとreturnの使い方\n
 = 番外：例（2）まとめ
 
   * 正常系同士の扱いを変えない
-    * 変えると異常系と勘違いしてしまう
+    * 変えると意図があると勘違いする
+    * 「Aの処理は何か特別なのかも…」
   * なんでもガード節にしない
     * 特別なケースや異常系のときだけ
 
@@ -482,10 +502,21 @@ ifとreturnの使い方\n
 
 = 実際の改善にチャレンジ！
 
-    TODO
+  # coderay javascript
 
-  * 誤解されない名前に改善して投稿
-  * よい投稿に「いいね！」して応援
+  // 読みやすい制御フローにして投稿・よい投稿に「いいね！」して応援
+
+  function positiveIntegerSum(n) { // n以下の正の整数の合計を返す
+    if (0 >= n) {
+      return -1; // エラー
+    } else {
+      var total = 0;
+      for (; n > 0; n--) {
+        total += n;
+      }
+      return total;
+    }
+  }
 
 = まとめ（1）
 
@@ -514,7 +545,6 @@ ifとreturnの使い方\n
   # blockquote
   # title = 7章 制御フローを読みやすくする p. 89
 
-  鍵となる考え\n
   行数を短くするよりも、他の人が理解するのにかかる時間を短くする。
 
 = これから（1）

data/config.yaml

@@ -6,7 +6,7 @@ tags:
 - readable-code
 - schoo
 presentation_date: 2015-04-03
-version: 2015.4.3.0
+version: 2015.4.3.1
 licenses:
 - CC BY-SA 4.0
 slideshare_id: 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rabbit-slide-kou-schoo-readable-code-2015-04
 version: !ruby/object:Gem::Version
-  version: 2015.4.3.0
+  version: 2015.4.3.1
 platform: ruby
 authors:
 - Kouhei Sutou
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-26 00:00:00.000000000 Z
+date: 2015-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rabbit