【PHP】お問い合わせフォームからDBへ格納されたデータの一覧表表示について

Question

・お問い合わせフォームから入力されたデータをDBに格納（myadminsi使用）
→ 格納されたDBの一覧表示をさせる

DBの取得・表示に関するプログラムに関して、
現在出来ているところまでで間違っている部分をご指摘下さるとあり難いです

因みに、DBクラスファイルと一覧表示ファイルは別にし、
require_onceで読み込ませる方法でやりたいです。

■DB_class.php
<?php
//MySQLの接続の為のクラスClass for connecting MySQL
class DB {
//コンストラクタ
var host:localhost
var user:ユーザ
var pass:
var db:test

function __construct($host,$user,$pass,$db) {
	$this->host = $host;
	$this->user = $user;
	$this->pass = $pass;
	$this->db = $db;
	$this->dsn = "mysql:dbname=$db;$host=$host";
}
 
function fetch($sql) {
  try{
        $pdo = new PDO ($this->dsn, $this->user, $this->pass, array(
                PDO::MYSQL_ATTR_INIT_COMMAND => "SET CHARACTER SET 'utf8'"));
        $stmt = $pdo->prepare($sql);
        $stmt->execute();
        $data = $stmt->fetchAll(PDO::FETCH_ASSOC);      
        return $data;
        } catch(PDOException $ei) {     
                echo 'Connection failed:'.$e->getMessage();
                exit();
	}
}
 
function execute ($sql) {
  try{
        $pdo = new PDO ($this->dsn, $this->user, $this->pass, array(
                PDO::MYSQL_ATTR_INIT_COMMAND => "SET CHARACTER SET 'utf8'"));
        $stmt = $pdo->prepare($sql);	//繰り返し
        $stmt->execute();
        $data = $pdo->lastInsertId();	//PDO::lastInsertId — 最後に挿入された行の ID あるいはシーケンスの値を返す
        return $data;
	} catch(PDOException $ei) {	//エラーメッセージ
                     echo 'Connection failed:'.$e->getMessage();
                exit();}
	}
}
?>

以下クラスファイルの読み込み

■DB.php

//クラスの読み込み
require_once "DB_class.php";
//DB(myadmin)へ接続
$conn = new DB($host,$user,$pass,$db);
//テーブルの読み込み
$sql = "SELECT * FROM contact_tbl";
$result = $conn->fetch($sql);

//SELECTしたものをwhileで表示させる
while($row = $sql->fetch()) {
	$id = $row['c_no']);
	$name = $row['c_date']);
	$name = $row['c_name']);
	$name = $row['c_status']);

	echo "$c_no"."<br />";
	echo "$c_date";
	echo "$c_name";
	echo "$status";
}

//テーブルすべてのレコードをカウント
$sql = mysqlquery("SELECT COUNT(*) FROM contact_tbl");
//行数を取得する
$count = mysql_num_rows($re);
//($re)は接続値
$count = 0;
 while($row = mysql_fetch_array($my_Row)){
 if( $count % 5 == 0 ) echo "<tr>";
 <td> ～ </td>
 if( $count % 5 == 4 ) echo "</tr>";
 $count++
 }

Answer 1

とりあえず、コードが滅茶苦茶とかはchokojoriさんに任せるとして、２点ほど・・・
DBクラスにて

$this->dsn = "mysql:dbname=$db;$host=$host";

とありますが、DBクラスにはdsnというプロパティが存在していないように見えます。
また、$host=$hostとしているため、おそらく正常に動作しないと思われます。
正しくは以下のようではないかと思います。

$this->dsn = "mysql:dbname=$db;host=$host"; //$が１つ余計

Answer 2

//テーブルすべてのレコードをカウント
$sql = mysqlquery("SELECT COUNT(*) FROM contact_tbl");
//行数を取得する
$count = mysql_num_rows($re);
//($re)は接続値
$count = 0;
 while($row = mysql_fetch_array($my_Row)){
 if( $count % 5 == 0 ) echo "<tr>";
 <td> ～ </td>
 if( $count % 5 == 4 ) echo "</tr>";
 $count++
 }

ここのところが何をしたいのか、さっぱりわかりません。

SELECT COUNT(*) FROM ...は、1行だけ結果があり、1個のカラムだけがあって、そこに行数が入っています。なのでmysql_num_rowsを使うのはナンセンスですね。わかります？

Comments

[bh_C]

ご回答ありがとうございます！
同じようなことを書いているということでしょうか
また、クラスの方はいかがでしょうか･･･？

[chokojori]

> 同じようなことを書いているということでしょうか

というより、まったく支離滅裂のコードなので、何をしようとしているのかわかりません。何がしたいのかわかれば、正しいコードを例示できますが...

> また、クラスの方はいかがでしょうか･･･？

こちらも、いろいろ感心しませんが、とりあえず動きそうな気がします。
感心しない理由としては...

1. 大量の行をSELECTするクエリをfetchに渡すとOut of memoryになる。
2. そもそもfetchというメソッド名はどうなのか。selectAllあたりが妥当では？
3. PDOを生で使わず、DBクラスを作る理由もよくわかりません。