プログラマー目指してます (95ﾚｽ)
上下前次1-新

---

92(2): 11/05(水)15:47 AA×


[240|320|480|600|100%|JPG|べ|ﾚｽ栞|ﾚｽ消]
92: [sage] 2025/11/05(水) 15:47:53.24 さらっと見た感じ index.phpやthread.phpで $nameに対しtrimを2重にやってるから $name = trim($_POST['name'] ?? 'No name'); これでいいんじゃね あと $title = trim($_POST['title'] ?? ''); $name = trim($_POST['name'] ?? ''); $content = trim($_POST['content'] ?? ''); if ($title !== '' && $content !== '') { // トランザクション開始 $pdo->beginTransaction(); これは、ifの条件を逆にしたほうがいい。要はtitle contentが正しい値かを見る引数チェックだろ 関数だったらARGVを見る引数チェックに相当する処理だから if ($title !== '' && $content !== '') {　... これの条件を逆にして if ( ... ) { 　// 値が不正な場合のエラーメッセージ等 // exit;等 } // トランザクション開始 $pdo->beginTransaction(); という流れのコードにしたほうがいい 引数チェック・メインの処理で分ける http://medaka.5ch.net/test/read.cgi/prog/1760017429/92
さらっと見た感じ やで に対しを重にやってるから これでいいんじゃね あと トランザクション開始 これはの条件を逆にしたほうがいい要は が正しい値かを見る引数チェックだろ 関数だったらを見る引数チェックに相当する処理だから これの条件を逆にして 値が不正な場合のエラーメッセージ等 等 トランザクション開始 という流れのコードにしたほうがいい 引数チェックメインの処理で分ける

---

上下前次1-新書関写板覧索設栞歴

---

あと 3 ﾚｽあります
ｽﾚ情報 赤ﾚｽ抽出 画像ﾚｽ抽出 歴の未読ｽﾚ AAｻﾑﾈｲﾙ

---

ぬこの手 ぬこTOP 0.025s