-1

このコードはheader.phpにあります。その場でタイトルを作成するためのものです。問題は、ユーザーがarticle.php（記事を読むためのページ）にいるときにのみタイトルを作成しますが、他のページでは「未定義の変数i」エラーを生成することです。何を変更する必要がありますか、またはWebページのタイトルのより良いコードを知っている人はいますか？私はidです。 

   if(isset($i=$_GET['i']) && $i = true ) {  
$i=$_GET['i'];
$tbl_name="yote";
$sql=mysql_query("SELECT * FROM $tbl_name WHERE i='$i'") or die(mysql_error());
while($rows=mysql_fetch_array($sql)){
$title = $rows['title']. "";
}
mysql_close();   

   } else { 

         $title = 'Malloooo - News, Reviews, Analysis and Market Watch From Around The World';     
                 } 
?>

<!-- Header
================================================== -->
<!DOCTYPE html>
<!--[if lt IE 7 ]><html class="ie ie6" lang="en"> <![endif]-->
<!--[if IE 7 ]><html class="ie ie7" lang="en"> <![endif]-->
<!--[if IE 8 ]><html class="ie ie8" lang="en"> <![endif]-->
<!--[if (gte IE 9)|!(IE)]><!--><html lang="en"> <!--<![endif]-->
<head>
<!-- Basic Page Needs
 ================================================== -->
<meta charset="utf-8">
<title><?php echo $title; ?></title>
4

1 に答える 1

0

正直なところ、コードにはかなりの間違いがあります。

  1. if (isset($i=$_GET['i']) ...：それで何をしようとしているのかわかりませんが、それは有効ではありません。
  2. ... && $i = true)：これは値を設定することであり、チェックすることではありません。
  3. あなたはSQLインジェクションに対してひどく脆弱です。PDOを調べます：http：//php.net/PDO
于 2013-02-23T22:40:13.820 に答える