第38回関西PHP勉強会

1. インデントを減らそう
@tasaeda
2018-02-14 第38回関西PHP勉強会

2. 自己紹介
• 朝枝 知之（ASAEDA Tomoyuki）
• Webサイト／サービス制作会社勤務
• Twitter: @tasaeda

3. 発端
• プログラムが不慣れな人に徐々にでも
いいから成長してほしかった

4. ……… (元から書かれているコード)
$tagList = [];
for ($i = 0; $i < count($data); $i++) {
if (!empty($data[$i]['tag'])) {
$tagList[] = $data[$i]['tag'];
} else {
$tagList[] = 'none';
}
}
……… (元から書かれているコード)
例1

5. foreach ($list as $id => $item) {
// ...
if (isset($item->no)) {
foreach ($item->no as $d => $t) {
if ($t['st'] && $t['et']) {
// ...
if ($etc) {
foreach ($etc as $etcK => $etcV) {
if ($etcK) {
$birthday = $etcValue;
}
}
}
}
// ...
}
// ...
}
}
例2

6. • コードは数行〜数十行
• 普段のやりたい処理は 配列｜if｜for で
ほぼ満たせる

7. • これらの単純な組み合わせでは処理で
きないか、書き方がわからなくなると
こちらに投げられる
• この繰り返しでは全体の成長がないの
で、できる限り自分で書くように仕向
けると…

8. $SQL = "SELECT id, score, score_day2, position FROM `".$tblName."` WHERE
tour_id = ".q($tour_id)." && hole != '-' ORDER BY '".$order_score."' ASC";
$RS = mysql_query($SQL);
$ROWS = mysql_num_rows($RS);
for ($i=0; $i<$ROWS; $i++) {
$id = vh(mysql_result($RS,$i,0));
$score = vh(mysql_result($RS,$i,1));
if ($i > 0) {
$score_prev = vh(mysql_result($RS,$i-1,1));
$z = ($score === $score_prev) ? $z : $i+1;
} else {
$z = $i+1;
}
$SQL = "UPDATE `".$tblName."` SET position = '".$z."' WHERE id = ".q($id);
if (!mysql_query($SQL)) {
vmsg("再設定が失敗しました。
","index","ntype=".$data['ntype']."&id=".$data['id']."&tid=".$data['tour_id']);
}
}
mysql_query("UPDATE `".$tblName."` SET position = 0 WHERE hole = '-'");
… to be continued …

9. システムの近代化を説いてみる
• 「一度簡単なフレームワークを触ってみようか！」
→なに書いてるか全然わからないです
• 「プログラム不要なCMSはどうかな？」
→やりたいカスタマイズができないのでイヤです
• 「Pinocoを使おう！HTMLも自然に書けるよ」
→forループ書いたらエラーでます！
※一部受け入れられました

10. なぜ響かないか
• 構造化しなくてもなんとか理解できる規模(数百〜
1000行程度)までのコードがほとんど
• 忙しい／興味がない
• それなりに回っているのでこれ以上の勉強の必要性
を感じない
…とはいえ「手に負える規模のレベル」は引き上げたい

11. 最低限お願いしたこと
• コードのインデントはなるべく2段までに抑える
→ if｜for をある程度抑制
• array系の関数を紹介 → 配列への対応
• continue を紹介 → for/ifへの対応
• php.net をブックマークに入れてもらう

12. ……… (元から書かれているコード)
$tagList = [];
for ($i = 0; $i < count($data); $i++) {
if (!empty($data[$i]['tag'])) {
$tagList[] = $data[$i]['tag'];
} else {
$tagList[] = 'none';
}
}
……… (元から書かれているコード)
Before

13. ……… (元から書かれているコード)
$tagList = array_map(function ($item) {
return $item['tag'] ?: 'none';
}, $data);
……… (元から書かれているコード)
After

14. foreach ($list as $id => $item) {
// ...
if (isset($item->no)) {
foreach ($item->no as $d => $t) {
if ($t['st'] && $t['et']) {
// ...
if ($etc) {
foreach ($etc as $etcK => $etcV) {
if ($etcK) {
$birthday = $etcValue;
}
}
}
}
// ...
}
// ...
}
}
Before

15. foreach ($list as $id => $item) {
// ...
if (!isset($item->no)) {
continue;
}
foreach ($item->no as $d => $t) {
if ((!$t['st'] && !$t['et']) && !$etc) {
continue;
}
// ...
foreach ($etc as $etcKey => $etcValue) {
if (!$etcK) {
continue;
}
$birthday = $etcValue;
// ...
}
}
// ...
}
After

16. 感想
• 「インデントを減らすようにする」とより論理的に
考えるクセがついた模様
→「上から下へ」だけではない処理の流れを意識
→関数・クロージャを使うことでスコープを意識
※「行数を減らすようにする」だと漠然すぎたみたい
• 色々な関数を php.net で調べるようになった
リファクタリングが（多少なりとも）しやすくなった(^^)v

17. 完
ありがとうございました。