m2wasabi 2012-08-15 16: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 2012-08-16 01:28:20  

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

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

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

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

なんかじわじわきた

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

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

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

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

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

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

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

うんこれはひどい

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

WHERE「・・・。」

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

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

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

Twitter認証

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

[Java] is禁止令

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

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

// Mod yamada Start

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

鑑賞する »

[Java] 内部的にunicodeだから日本語つかえ...

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

これを書いた先輩に「英語で書きましょうよ...

class 会員 {
    private int 会員番号;
    p...

鑑賞する »

[Java] フルHD推奨。

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

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

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

鑑賞する »