m2wasabi m2wasabi 2012-08-16 01:58:43

[PHP] データベースに・・・・うわぁぁぁぁぁあ このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

例外って何だろう。SQLってなんだろう。エラーとは。そして帰ってこない関数。 先人の知恵をすべて無に返す実装。

// DBに接続
$pdo = new PDO($dsn,$user,$password);

$id = $_POST['id'];

$data = getHogeData($pdo,$id);

$smarty->assign('data',$data);
$smarty->display();

function getHogeData(&$pdo,$id){
	try {
		// データの取得
		$sql = "SELECT * FROM hoge";
		$stmt = $pdo->query($sql);
		while($data = $stmt->fetch()){
			if($data['id'] == $id){
				return $data;
			}
		}
	} catch(exception $e){
		user_error('入力内容が不正です');
		exit;
	}
}

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(10)

#1 math_neko math_neko 2012-08-16 10:28:20  

なまじ SQL に詳しくてクソ長い SQL 文になってるのは見たことあるがこれもこれで酷い

#2 cubmaru cubmaru 2012-08-16 12:57:38  

これ負荷テストの内容によっては通っちゃいますよね、その点でも破壊力が大きい

#3 HarpoonArrow HarpoonArrow 2012-08-17 02:00:41  

なんかじわじわきた

#4 kxyz_0x9a kxyz_0x9a 2012-08-19 11:57:04  

7〜8年前にコボラーのオッサンが書いたソースをメンテした事があったが基本このスタイルだった

#5 fshin2000 fshin2000 2012-08-19 20:32:50  

これ熱いなー。user_errorメソッドの自在ぶりも。

#6 uncle_istvan uncle_istvan 2012-08-19 21:31:25  

もともとファイルを行で探索するコードをDBに転用しましたって感じがする。例外処理は・・・ヒット前提なのかな

#7 m_shige1979 m_shige1979 2012-08-20 13:44:32  

うんこれはひどい

#8 gradou2 gradou2 2012-08-20 23:18:00  

WHERE「・・・。」

#10 BlackDogK8 BlackDogK8 2012-10-06 13:36:59  

これはコボラーの仕業ですな。 10年程前だったかコボラーのおっさんに、同じ様なコード書けって言われた。

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] is禁止令

このエントリーをはてなブックマークに追加

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »

[Java] フルHD推奨。

このエントリーをはてなブックマークに追加

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[PHP] つ、釣り!?

このエントリーをはてなブックマークに追加

ツボって思い出し笑いが止まらなかったので...

class Hoge {
    /*
      * インストラクター
...

鑑賞する »