Mysqlの実行コードが無限ループしてしまう。 - PHPプロ！Q&A掲示板

2504

0P

Mysqlの実行コードが無限ループしてしまう。

質問日時 / 2010年2月6日 01:18 回答数 / 3件

Questioner: memory Tweet

キーワード / キーワードが設定されていません

正しい書き方のはずなんですが、無限ループしてしまいます。

<?php
class config{
public function connection_to_mysql ($ret = true, $port = '3306') {
$connection = mysql_connect($this->database['IP'] . ':' . $port, $this->database['HOST'], $this->database['PASSWORD']) or die (mysql_error());
mysql_select_db($this->database['NAME'], $connection) or die (mysql_error());
mysql_query('SET NAMES UTF8');
if ($ret == true) {
return $connection;
}
}
public function mysql_attach_basic ($first, $table, $where, $set = '', $objecting = false, $limit = 1) {
$self = '';
$first = strtoupper($first);
switch ($first) {
case 'SELECT':
case 'DELETE':
$usewhere = '';
$asterisk = '';
if ( $where != '' ) {
$usewhere = 'WHERE ';
}
if ( $first == 'SELECT' ) {
$asterisk = ' * ';
}
$self = $first . $asterisk . ' FROM `' . $table . '` ' . $usewhere . $where;
break;
case 'INSERT':
$self = $first . $asterisk . ' INTO `' . $table . '` (' . $where . ') VALUES (' . $set . ')';
break;
case 'UPDATE':
$add = '';
$usewhere = '';
if ( $where != '' ) {
$usewhere = 'WHERE ';
}
if ( $set != '' ) {
$add = 'SET ' . $set;
}
$self = $first . ' ' . $add . '`' . $table . '` ' . $usewhere . $where;
break;
}
$query = $this->mysql_attach_escape($self);
$db = mysql_query($query, $this->connection_to_mysql());
if ( $objecting == true ) {
$db = mysql_fetch_object($db);
}
$this->mysql_attach_action($con = null, $query = null);
return $db;
}
private function mysql_attach_escape($query) {
$query = preg_replace('/"(.*?)"/', '"' . mysql_real_escape_string('$1', $this->connection_to_mysql()) . '"', $query);
$query = preg_replace("/'(.*?)'/", '"' . mysql_real_escape_string('$1', $this->connection_to_mysql()) . '"', $query);
return $query;
}
private function mysql_attach_action($con = null, $query = null) {
if ($con != null) {
if ($query != null) {
@mysql_free_result($query);
}
@mysql_close($con);
}
}
}
?>

というコードで、使用方法は、

<?php
$config = new config();
$config->init();
$a = $config->mysql_attach_basic('SELECT', 'accounts', '', '', true);
while ($b = $a) {
}
?>

という風にやっています。

以前のプログラムもこんな感じに書いたのですが、それは実行できています。
相違点がまったくもって分かりません。
以前のプログラム

<?php
class server_system {
function attach_mysql() {
$connect = mysql_connect($this->ip, $this->database_id, $this->database_password);
mysql_select_db($this->database_name, $connect);
mysql_query('SET NAMES UTF8');
return $connect;
}
function attach_mysql_query_escape($query) {
$query = preg_replace('/"(.*?)"/', '"' . mysql_real_escape_string('$1') . '"', $query);
$query = preg_replace("/'(.*?)'/", '"' . mysql_real_escape_string('$1') . '"', $query);
return $query;
}
function attach_mysql_query($query, $unreturn = false) {
$connect = $this->attach_mysql();
if ($unreturn == false) {
$set = mysql_query($this->attach_mysql_query_escape($query), $connect) or die($this->debug(mysql_error(), 1));
} else {
mysql_query($this->attach_mysql_query_escape($query), $connect) or die($this->debug(mysql_error(), 1));
}
$this->attach_mysql_action($connect);
if ($unreturn == false) {
return $set;
}
}
function attach_mysql_action($con = null, $query = null) {
if ($con != null) {
if ($query != null) {
@mysql_free_result($query);
}
@mysql_close($con);
}
}
}
?>

この質問への意見の募集は締め切られ、ポイントは既に配分されました。
意見を投稿することはできますが、ポイントを受け取ることはできません。

この質問に返信

ツリー一覧

Mysqlの実行コードが無限ループしてしまう。memory

A01kaitauとりあえず、「無限ループ」ってありますけどループし

A01-1memory返信ありがとうございます。クラスについては、ま

A01-1-1memory自己解決しました。一度変数に入れてから、実行する

回答一覧

A01
answererkaitau [2月6日 13:01]

とりあえず、「無限ループ」ってありますけどループしてる箇所や、
再帰関数が返ってこないような類の場所はありません。
大量のデータのテーブルに対して複雑なクエリをかければ実行時間が
非常に遅くなることはままありますが、どうみてもそんな現場の
雰囲気でもありませんね。

おそらく、呼び出し側の

$a = $config->mysql_attach_basic('SELECT', 'accounts', '', '', true);
while ($b = $a) {
}

のループの中の処理の方が問題なのでしょう。
まずそちらを調べた方が良いかと。

---- 本題ここまで ----

ところで、率直に言ってクラスの設計がかなりよろしくないかと。
多分覚えたてなんだと思うけど、こういう「自分ルール」的な
パラメータの用法をすると拡張する時とかバグを直す時に苦しみます。
PEARのライブラリのソースをよく読んで勉強した方が良いと思います。
SQLの組み立ても、単一のメソッドで済むほど簡単な世界ではありません。

なにより、最低限エスケープを何のためにするか知ってください。
$query全体にかけても意味ないから。

A01-1
replyermemory [2月6日 15:54]

返信ありがとうございます。

クラスについては、まだ作り途中のもので、一時的にデバッグを行った結果です。
まだ改善点があるようなので、肝に銘じておきます。

エスケープについてですが、特殊文字など、ソフトウェア側が解釈するためまたは、文字化けの引き金となる文字を処理するためだと独学で学びました。
海外のフォーラムで、文字列に対してmysql_real_escape_stringをするといいということが書いてありましたので、こういった処理をしました。

もう一度見直してみます。
返信ありがとうございました。

A01-1-1
replyermemory [2月6日 18:13]

自己解決しました。
一度変数に入れてから、実行すると、できました。

お手数おかけしました。

<<質問一覧へ