プログラマー目指してます (95ﾚｽ)

プログラマー目指してます http://medaka.5ch.net/test/read.cgi/prog/1760017429/

上下前次1-新
通常表示 512ﾊﾞｲﾄ分割 ﾚｽ栞

---

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

---

ﾒﾓ帳

(0/65535文字)

---

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

---

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

---

Google検索

Wikipedia

---

ぬこの手 ぬこTOP 0.003s