2012-01-03 18 views
0

ループ内の関数を呼び出して、それに引数を渡して、選択した項目をデータベースに格納するコードを用意しました。私は正しい引数を関数に渡しているとは思わないので、見てみることはできますか?引数を関数に渡す

<?php 

function welcome($grill){ 
$link = mysql_connect('localhost', 'sc2brsting', '1A2'); 
if (!$link) { 
    die('Could not connect: ' . mysql_error()); 
} 
echo 'Connected successfully'; 

mysql_select_db('sc2bring1', $link); 

$sql = "INSERT INTO `sc2broad_tesing1`.`Persons` (`re`) VALUES ($grill)"; 

mysql_query($sql); 

mysql_close($link); 
} 

?> 

<?php 

$grass=0; 
while($grass<500){ 
$file = $DOCUMENT_ROOT . "website.com"; 
$doc = new DOMDocument(); 
@$doc->loadHTMLFile($file); 


$elements = $doc->getElementsByTagName('a'); 

for ($i=106; $i<=204; $i=$i+2) 
    { 
    $grill = $elements->item($i)->nodeValue . " "; 
welcome($grill); 

    } 
$grass=$grass+24; 
} 

?> 

た問題イムは、変数$グリルは、データベースへに挿入するときは、変数$grillをエスケープされていない機能

+1

あなたは私は私の答え – Dau

+0

に直面している問題は、あなたが 'welcome'機能になっているgrill'' $の値が何であるかを記述することができるもの、あなたのコード '$ elements-> item($ i) - > nodeValue'をチェックしてください。このシンセックスで得られる' $ grill'の値 – Shawn

+0

を更新 – Dau

答えて

1

を通過していないことです。

これはMySQLエラーを引き起こすので、引数が関数に渡されないという印象を与えます。

行は次のようになります。

$sql = "INSERT INTO `sc2broad_tesing1`.`Persons` (`re`) VALUES ('".$grill."')"; 
+0

と同じように表示されます: '' $ grill ''を使用する必要はありません $ sql = "INSERT INTO' sc2broad_tesing1' .'Persons'( 're')VALUES( '$ grill')"; – redmoon7777

+0

それは私が書いたものです:-) @Jakubはそれを編集しましたが、それは良い習慣であるので大丈夫です。 – konsolenfreddy

+0

申し訳ありません、私は編集しました。エディタで色の強調表示が良くなったため、その方法を使用しました(いつも私にはきれいに見えます)。 – Jakub

関連する問題