1

我正在尝试将$user_id变量从文件返回到if(isset(...))语句。我将执行不同的 if 语句，但只是尝试echo输出$user_id变量来测试我正在设置页面。

<?php 
include 'core/init.php';
include 'init.image.php';
protect_page();
include 'includes/overall/overall_header.php';

if(isset($_GET['username']) === true && empty($_GET['username']) === false){
$username       = $_GET['username'];


if(user_exists($username) === true){
$user_id        = user_id_from_username($username);
$profile_data   = user_data($user_id, 'first_name','last_name','email', 'username');
?>

    <h1><?php echo $profile_data['first_name']; ?>'s Yor Page</h1>

<div id="navWrapper">
    <ul>
        <li>
            <a href="#"><img src="uploads/profile/blank_profile.gif" width="150" height="150" id="blank_profile"></a>
        </li>

        <nav>
            <ul>
                <li>
                    <a href="<?php echo $profile_data['username'];?>?action=albums">Albums</a>
                </li>
                <li>
                    <a href="<?php echo $profile_data['username'];?>?action=music">Music</a>
                </li>
            </ul>
        </nav>
    </ul>
</div>

<?php
if(isset($_GET['action']) && $_GET['action']=='albums'){

$albums = get_profile_albums($user_id);

if(empty($albums)){
echo 'No Albums';
}else{

foreach($albums as $album){
if (empty($album['image'])) {
    $album['image'] = 'uploads/profile/blank_profile.gif';
}
?>
 <p><?php echo $album['name'],' (', $album['count'], ')'?> <br />
  <a href="<?php echo $profile_data['username'];?>?action=album_id=<?php echo $album['id'];?>">
    <img src="uploads/thumbs/<?php echo $album['id'];?>/<?php echo $album['image'];?>" />
  </a><br />
  <?php echo $album['description'];?>...<br />
</p>
<?php
}
}

}
if(isset($_GET['action']) && $_GET['action']=='album_id=$album['id']'){
echo $user_id;
}



if(isset($_GET['action']) && $_GET['action']=='music'){
echo'<h1>Music</h1>';
}

}else{
    echo 'Sorry, that user doesn\'t exist';
}
}else{
header('Location: index.php');
exit();
}

include 'includes/overall/overall_footer.php';
?>
4

2 回答 2

3

单引号不解析字符串，但双引号可以。例如：

$album_id = 1;
echo 'album_id=$album_id ';
echo "album_id=$album_id";

会导致album_id=$album_id album_id=1

因此，$_GET['action']=='album_id=$album_id'检查是否$_GET['action']等于'album_id=$album_id'而不是$album_id评估的内容。

您的支票应该更多地遵循以下原则：

if (/* ... */ && $_GET['action']=="album_id=$album_id") {

编辑：建议。

您应该检查$_GET['action'] 一次并存储$action。从那里创建一个if/else if/else语句，检查每种可能的操作类型（音乐、专辑、专辑等）。如果某种类型的操作需要额外的用户提供的数据，请将其包含在相应的 if 块中。例如：

 $action = $_GET['action'];
 // URL: /script.php?action=albums
 if ($action == 'albums') {
     // ...
 }
 // URL: /script.php?action=album&album_id=12
 else if ($action == 'album') {
     if (!empty($_GET['album_id'])) {
          $album_id = $_GET['album_id'];
          echo "User with id $user_id is trying to access album with id $album_id";
     }
     // ...
 }
 // ...

不再$_GET['action']=="album_id=$album_id"

于 2012-08-02T00:35:22.087 回答
0

您正在正确传递变量。

在以下行中我更关心的是：

if(isset($_GET['action']) && $_GET['action']=='album_id=$album_id'){

'album_id=$album_id'看起来不正确，这可能是您的 if 没有触发的原因...看起来您正在尝试使用$album_id用单引号括起来的变量，并且不会替换该值，而是保持$album_id原样。我不确定这是否是您的意图，但如果不是，则可能是您的问题的根源。

于 2012-08-02T00:16:45.597 回答