0

URLに保存されている画像をローカルフォルダにダウンロードしようとしていますが、curlを使用して次のように試みました。curlを含めるか、個別にダウンロードする必要があるのか​​、それとも私の関数が正常に機能するのかを知りたいのですが。以下の私の実装に明らかな問題があるかどうか知りたいです。SQLの脆弱性を認識しており、プリペアドステートメントに切り替えています。簡潔にするために、コードの関連性のない部分をトリミングしました。

編集:関数はwhileループから外れています。関数の呼び出しをコメントアウトするとページが表示されます。コメントアウトしないと、空白のページしか表示されません。どうしてこれなの

<?php
header("Content-Type: text/html; charset=utf-8");
if (isset($_GET["cmd"]))

  $cmd = $_GET["cmd"];

else

  die("You should have a 'cmd' parameter in your URL");

 $pk = $_GET["pk"];

$con = mysql_connect("localhost","someuser","notreal");

if(!$con)

{

die('Connection failed because of' .mysql_error());

}
mysql_query('SET NAMES utf8'); 

mysql_select_db("somedb",$con);

if($cmd=="GetAuctionData")

{

$sql="SELECT * FROM AUCTIONS WHERE ARTICLE_NO ='$pk'";

$sql2="SELECT ARTICLE_DESC FROM AUCTIONS WHERE ARTICLE_NO ='$pk'";

$htmlset = mysql_query($sql2);

$row2 = mysql_fetch_array($htmlset);



$result = mysql_query($sql);

function savePicture($imageUrl) {
    $ch = curl_init();
    curl_setopt ($ch, CURLOPT_URL, $lastImg);
    curl_setopt ($ch, CURLOPT_RETURNTRANSFER, 1);
    curl_setopt ($ch, CURLOPT_CONNECTTIMEOUT, 0);
    $fileContents = curl_exec($ch);
    curl_close($ch);
    $newImg = imagecreatefromstring($fileContents);
    return imagejpeg($newImg, "./{$pk}.jpg",100);
}

while ($row = mysql_fetch_array($result))
{



$lastImg = $row['PIC_URL']; 


savePicture($lastImg);


<div id='rightlayer'>

<img src='./".$pk.".jpg' width='".$outputWidth."' height='".$outputHeight."'>


</div>

</div>

</div>";



}



}

mysql_free_result($result);
4

2 に答える 2

1

ループが複数回実行されているときにループ内で関数を宣言すると、エラーが発生します。savePictureしたがって、関数は の外側で宣言する必要がありますwhile

于 2009-01-21T13:59:11.157 に答える
0

while ブロックから関数定義を取り出します。

私の意見では、ここで curl を使用するために curl を使用しています。より簡単な方法は、file get contentsを使用することです。

于 2009-01-21T13:56:53.970 に答える