インクルードによる不具合

解決済

質問者：mapple55
質問日時：2007/06/26 14:15
回答数：4件

PHP初心者です。
以下２つのファイルを使って簡単な接続テストを行ってから徐々に
目的のものを作り上げていこうとしたのですが、どうにもうまく動作しません。（２つは同一ディレクトリにある）
・「mysql.php」：DB接続とSQLの発行
・「test.php」：ブラウザ出力（本体：mysql.php呼び出し元）
実行すると「***　ＤＢ/ＳＱＬテスト　***」のみブラウザに出力されるため、試しに「mysql.php」で行うDB接続とSQL発行をすべてtest.phpに直接書いてインクルードせず単体で実行したところ、目的の結果が表示されました。
なので問題はインクルード文にあるのだと思うのですが検討がつきません。
どなたかお力をお貸し下さい。
ちなみに以下のコードは参考にしたテキストの例題をもとに見やすくするため余分な処理やエラー処理などを省いたものです。

■mysql.php
<?php
class MySQL{
var $m_con;
var $m_rows = 0;

function MySQL(){
$this->m_con = mysql_connect("ホスト名", "ユーザ名", "パスワード");
mysql_select_db("データベース名", $this->m_con);
}

function query($sql){
$this->m_rows = mysql_query($sql, $this->m_con);
return $this->m_rows;
}
}
?>

■test.php
<html>
<head>
<meta http-equive="Content-type" content="text/html; charset=utf-8">
<title>SQLtest</title>
</head>
<body>
***　ＤＢ/ＳＱＬテスト　*** 
<?php
include("mysql.php");
$mysql = new MySQL;
$sql = "SELECT * FROM staff";
$mysql->query($sql);
while($row = mysql_fetch_array($this->m_rows)){
echo $row["id"];
echo $row["name"];
echo " ";
}
?>
</body>
</html>

通報する

この質問への回答は締め切られました。

質問の本文を隠す

回答 (4件)

ベストアンサー優先
最新から表示
回答順に表示

No.4ベストアンサー

回答者： tany180sx
回答日時：2007/06/26 22:42

補足の通り書いているのだとすると・・

・コメントアウトが不正
/ -> //

・DBの接続情報が不正
mysql_connect($this->m_HostName, $m_UserName, $m_Password);
-> mysql_connect($this->m_HostName, $this->m_UserName, $this->m_Password);

この回答への補足

あぁ・・・やってしまいました。
我ながらなんて痛いヤツなんでしょう。最低です・・・

ご指摘のとおり、コメント文が不適切だったのが主な原因でした。
ただこのあと修正したものでテストすると「MYSQLの接続に失敗しました」がでました。
iniファイルの情報を直打ちする形で絞っていき、値が正常に読み取れていないと
推測できたのでいろいろ試したところ、「mysql.ini」の文字コードを「utf-8」から「sjis」に変更することで無事接続できました。
なぜ「utf-8」ではいけなかったのかは不勉強なのでわかりませんが、とりあえずはこれで進めることができるようになりました。

なぜエラーメッセージがでないのか、なぜincludeで処理が途切れてしまうのかという点で自分の知識の及ばない範囲の不具合かもしれないと、勝手に舞い上がってしまいました・・・
確かにコメント文はコピペでアップロード前にいい加減に付け足した覚えがあります。
エディタ機能の色分けで気づきそうなものですが、完全な思い込みで盲目になってました。

この問題にご協力頂き、真剣に考え時間を費やして下さった皆様本当に申し訳ありませんでした。
皆様の丁寧なご指摘で問題点以外でもいろいろ参考になりました。
ありがとうございました。

補足日時：2007/06/27 01:07

通報する

- 0
- 件

通報する

この回答へのお礼

ご協力ありがとうございました。
これってありがとうポイントみたいなものはどうやったらつけられるんでしょう・・・？

通報する

お礼日時：2007/06/27 01:19

No.3

回答者： php504
回答日時：2007/06/26 15:47

DB classを作るならDB操作を全てclass内に実装しないとやりにくくないですか。

PEAR/DBとかPDOとか既存のものを使うのはだめでしょうか。

この回答への補足

的外れの返答だったならすいません。
元々のサンプルコードは以下のようなＤＢ操作をまとめてやるものでしたが
問題箇所を探す上でコードを改変し、また質問する上でその方がゴチャゴチャせずに理解して頂けると思ってこちらに載せました。
最初はmysql.iniも同一ディレクトリに用意してテストしていたのですが、うまくいかなかったので直接コード内に書くようにし、さらに関連の薄いコードを削っていきました。
この間、エラーメッセージはなかったように思います。
PEAR/DBとかPDOはまだ知識としてないのでわかりません。
このサンプルのやり方で出来たなら次のステップでもテキストを参考に進められるのでできる限りこのやり方での解を知りたいと思っています。

■mysql.php

<?php
/===============================================
/ ■：MySQLクラス
/===============================================
class MySQL{
/-----------------------------------
/ □：変数宣言
/-----------------------------------
var $m_con;
var $m_HostName = "";
var $m_UserName = "";
var $m_Password = "";
var $m_Database = "";
var $m_rows = 0;
/-----------------------------------
/ □：コンストラクタ
/-----------------------------------
function MySQL(){
$filename = "mysql.ini";
if (!file_exists($filename)){
die("mysql.iniファイルが見つかりません");
}else{
$fp = fopen($filename,"r");
if (!$fp){
die("mysql.iniファイルが見つかりません");
}else{
$this->m_HostName = trim(fgets($fp));
$this->m_UserName = trim(fgets($fp));
$this->m_Password = trim(fgets($fp));
$this->m_Database = trim(fgets($fp));
}
fclose($fp);
}
//MYSQLへ接続
$this->m_con = mysql_connect($this->m_HostName, $m_UserName, $m_Password);
if (!$this->m_con){
die("MYSQLの接続に失敗しました");
}
if(!mysql_select_db($this->m_Database, $this->m_con)){
die("データベースの選択に失敗しました");
}
}

/-----------------------------------
/ SQLクエリ処理
/-----------------------------------
function query($sql){
$this->m_rows = mysql_query($sql, $this->m_con);
if (!$this->m_rows){
die("MYSQLでエラーが発生しました {$sql}".mysql_errno().";".mysql_error());
}
return $this->m_rows;
}

/-----------------------------------
/ SQLクエリ処理
/-----------------------------------
function fetch(){
return mysql_fetch_array($this->m_rows);
}

/-----------------------------------
/ SQLで変更された行数を取得
/-----------------------------------
function affected_rows(){
return mysql_affected_rows();
}

/-----------------------------------
/ 列数取得
/-----------------------------------
function cols(){
return mysql_num_fields($this->m_rows);
}

/-----------------------------------
/ 行数取得
/-----------------------------------
function rows(){
return mysql_num_rows($this->m_rows);
}

/-----------------------------------
/ 結果の開放
/-----------------------------------
function free(){
mysql_free_result($this->m_rows);
}

/-----------------------------------
/ MYSQLをクローズ
/-----------------------------------
function close(){
mysql_close($this->m_con);
}

/-----------------------------------
/ SQLエラー情報（No.+ msg）
/-----------------------------------
function errors(){
return mysql_errno().": ".mysql_error();
}

/-----------------------------------
/ SQLエラーNo.
/-----------------------------------
function err_no(){
return mysql_errno();
}
}
?>

■test.php

<?php
require_once("mysql.php");
?>
<html>
<head>
<meta http-equive="Content-type" content="text/html; charset=utf-8">
<title>SQLtest</title>
</head>
<body>
***　ＤＢ/ＳＱＬテスト　***
<?php
$mysql = new MySQL;
$sql = "SELECT * FROM staff";
$mysql->query($sql);
while($row = $mysql->fetch()){
echo $row["id"];
echo $row["name"];
echo " ";
}
?>
</body>
</html>

補足日時：2007/06/26 17:00

通報する

- 0
- 件

通報する

この回答へのお礼

ご協力ありがとうございました。

通報する

お礼日時：2007/06/27 01:16

No.2

回答者： tany180sx
回答日時：2007/06/26 14:38

> while($row = mysql_fetch_array($this->m_rows)){

$this が間違いでは？

display_errors と error_reporting の設定を変更して
エラーを表示すればエラーが出ませんか？

この回答への補足

$thisの部分は書き換えましたが結果は変わりませんでした。

アドバイス頂いたエラー表示に関してはよくわからないのですが、
（info.phpで設定を調べたところdisplay_errors=on , error_reporting=81　でした）
もともと各処理ではdieを使ってエラー表示するようにしていましたが、
実行しても何も表示されませんでした。

実行すると「***　ＤＢ/ＳＱＬテスト　***」が表示されるだけで
ソースを見ても以下のようにインクルード直前で途切れています。

<html>
<head>
<meta http-equive="Content-type" content="text/html; charset=utf-8">
<title>SQLtest</title>
</head>
<body>
***　ＤＢ/ＳＱＬテスト　***

補足日時：2007/06/26 15:39

通報する

- 0
- 件

通報する

No.1

回答者： yambejp
回答日時：2007/06/26 14:37

とりあえず

while($row = mysql_fetch_array($this->m_rows)){
は
while($row = mysql_fetch_array($mysql->m_rows)){
じゃないですかね？

この回答への補足

早速ありがとうございます。

指摘の箇所を修正して実行してみましたがやはり結果は変わりませんでした。
mysql_fetch_arrayの部分はもともとmysql.php内で関数化されているのを
呼び出すコードだったのですが、なるべく問題箇所をDB接続くらいにまで
絞って示したかったので自分で中途半端に書き換えてしまったものでした。
すいません。

補足日時：2007/06/26 15:34

通報する