PHPで unexpected end of file エラーを解消したいです。

Question

前提・実現したいこと

下記ＰＨＰを実行するとエラーが出るので、解消したいです。

発生している問題・エラーメッセージ

Parse error: syntax error, unexpected end of file in /home/abc/storelocator_sqli.php on line 50

５０行目はソースコード最終行の　?> 　の部分になります。

該当のソースコード

<?php
// require("phpsqlsearch_dbinfo.php");
// Get parameters from URL
$center_lat = $_GET["lat"];
$center_lng = $_GET["lng"];
$radius = $_GET["radius"];
// Start XML file, create parent node
$dom = new DOMDocument("1.0");
$node = $dom->createElement("markers");
$parnode = $dom->appendChild($node);

/* PHPでデータベースに接続 */

$dbhost = 'samdbhost';
$dbuser = 'hoge';
$dbpass = 'fuga';
$dbname = 'samdb';
$conn = mysqli_connect($dbhost, $dbuser, $dbpass, $dbname);
mysqli_set_charset($conn, "utf8");
if(! $conn ) {
die('Could not connect: ' . mysqli_error());
}
$db_selected =mysqli_select_db( $db_link, 'samdb');
if (!$db_selected) {
 die ("Can\'t use db : " . mysql_error());

$query  = sprintf("SELECT id, name, address, lat, lng, ( 3959 * acos( cos( radians('%s') ) * cos( radians( lat ) ) * cos( radians( lng ) - radians('%s') ) + sin( radians('%s') ) * sin( radians( lat ) ) ) ) AS distance FROM markers HAVING distance < '%s' ORDER BY distance LIMIT 0 , 20",  
  mysql_real_escape_string($center_lat),
  mysql_real_escape_string($center_lng),
  mysql_real_escape_string($center_lat),
  mysql_real_escape_string($radius));
$result = mysqli_query($query);
$result = mysqli_query($query);
if (!$result) {
  die("Invalid query: " . mysql_error());
}
header("Content-type: text/xml");
// Iterate through the rows, adding XML nodes for each
while ($row = @mysql_fetch_assoc($result)){
  $node = $dom->createElement("marker");
  $newnode = $parnode->appendChild($node);
  $newnode->setAttribute("id", $row['id']);
  $newnode->setAttribute("name", $row['name']);
  $newnode->setAttribute("address", $row['address']);
  $newnode->setAttribute("lat", $row['lat']);
  $newnode->setAttribute("lng", $row['lng']);
  $newnode->setAttribute("distance", $row['distance']);
}
echo $dom->saveXML();
?>

試したこと

■paiza.IOに問題のPHPを記載すると下記表示が出ます。

画像のＸ印にカーソルを当てると、unexpected $EOF と表示されます。

■サクラエディタをつかっているので、このエディタの問題なのか？

補足情報（FW/ツールのバージョンなど）

「PHPでMySQLに接続できません。」https://teratail.com/questions/183477
を解消した後、今回の障害に出会っています。

Answer 1

IDEで開発しましょう。構文エラーであれば実行前に教えてくれます。

以下はEclipseの例

if (!$db_selected) {の閉じ忘れが原因ですね。
IDE使えば{と打った後にEnterすると}が自動的に補完されるので、意図的に削除しない限りは閉じ忘れが一気になくなると思います。

あと、今回のようにXML表示をする場合、特に画面出力がないのであれば閉じ?>は不要です。
というかXML表示に限らず、画面出力がその後にないのであれば最後の閉じ?>は不要です。

• モダンPHPアンチパターン#ファイルの最後に?>を書く

不要な?>を書いてはいけない理由は明確で、<?php … ?>の外にあるコードは出力されてしまうからだ。つまり、誰かが?>の直後に余分な改行を入れてしまったら、わかりにくいタイミングで意図しない改行が出力されることになる。

Comments

[dreamers]

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

Answer 2

if (!$db_selected) {

die ("Can't use db : " . mysql_error());

閉じカッコが抜けてるんでは

Comments

[dreamers]

閉じカッコを付けると、問題は解消されました。
必ずしも、指摘された50行目に問題あるわけではないのですね。
ありがとうございました。

[y_waiwai]

unexpected end of file とか、予期せぬEOF、とかのエラーはどっかでカッコ抜けてる、と思ってたらいいです