0

次の変数$user_idが設定されています

    //Check if user is logged in
session_start();
   if (!isset ($_SESSION['user_id']))
{
    header("location:login.php");     
} 
 elseif(isset ($_SESSION['user_id']))
  {
$user_id = $_SESSION['user_id'];
  }

そして、同じ関数ファイル内に次のものがあります。

function course_menu()
  {
      $sqlSubscription = "SELECT * FROM subscriptions WHERE `user_id` = '".$user_id."'";
            $subscriptionResult = mysql_query($sqlSubscription);

                while ($rows = mysql_fetch_assoc($subscriptionResult))
                    {
                        $user_id = $rows['user_id'];
                        $course_id = $rows['course_id'];
                        $course_title = $rows['course_title'];

                              if ($data_id == $rows['course_id'])

                              {
                                  echo
                                  '<li>
                                      <a href="chapters.php?course_id=',$course_id,'&course_name=',$course_title,'" class="active">',$course_title,' </a>
                                          </li>';
                              }
                                  else
                                  {
                                      echo          
                                                  '<li><a href="chapters.php?course_id=',$course_id,'&course_name=',$course_title,'">',$course_title,' </a></li>';
                                  }
                    }
  }

問題は、関数を実行しようとするたびに未定義の変数 user_id を取得し続けることです。echo $user_id別のページでand thenindex.phpを使用して言うことができますが、何らかの理由で関数自体がアクセスできませんか?require_once function.phpecho $user_id

それはその範囲外にあるためだと思いますが、そうであれば、どうすればよいか完全にはわかりません。

私の質問は、関数が変数 $user_id を使用できるようにするにはどうすればよいですか?

編集

だからやり始めた

$user_id = $_SESSION['user_id'];
global $conn;

$sqlSubscription = "SELECT * FROM subscriptions WHERE `user_id` = '".$user_id."'";
            $subscriptionResult = $conn->query($sqlSubscription);
            while ($rows = mysqli_fetch_assoc($subscriptionResult))
                {
                    $user_id = $rows['user_id'];
                    $course_id = $rows['course_id'];
                    $course_title = $rows['course_title'];

                          if ($data_id == $rows['course_id'])

                          {
                              echo
                              '<li>
                                  <a href="chapters.php?course_id=',$course_id,'&course_name=',$course_title,'" class="active">',$course_title,' </a>
                                      </li>';
                          }
                              else
                              {
                                  echo          
                                              '<li><a href="chapters.php?course_id=',$course_id,'&course_name=',$course_title,'">',$course_title,' </a></li>';
                              }
                }

これはうまくいくように見えますが、関数を使用して毎回新しい接続を追加したり、$user_id を手動で設定したりするのは少し面倒です。データをプルするためにデータベースへの接続を必要とするいくつかの関数があるため、これを回避する方法はありますか? このタイプのものを構造化するためのより良い方法はありますか? 私は OOP にあまり詳しくありませんが、方向性がわかれば試してみることができます。これは私が使用する別の関数です (そして、データベース接続を必要とする少なくとも別の 5-6 があります)。

function render_dashboard()
{
$user_id = $_SESSION['user_id'];
global $conn;
//Following brings up the number of subscription days left on the user dashboard
$sqlDate = "SELECT * FROM subscriptions WHERE `user_id` = '".$user_id."'" ;
$date = $conn->query($sqlDate);

 while ($daterows = mysqli_fetch_assoc($date))
                    {
                        $course_registered = $daterows['course_title'];
                        $date_time = $daterows['end_date'];
                        $calculate_remaining = ((strtotime("$date_time")) - time())/86400;
                        $round_remaining = round("$calculate_remaining", 0, PHP_ROUND_HALF_UP);
                       // Here we assign the right term to the amount of time remaining I.E DAY/DAYS/EXPIRED
                         if($round_remaining > 1)
                             {
                                 $remaining = $course_registered." ".$round_remaining." "."Days Remaining";
                                 $subscriptionStatus = 2;
                                 echo '<p>',$remaining,'</p>';
                             }

                        elseif ($round_remaining == 1) 
                             {
                                 $remaining = $course_registered." ".$round_remaining." "."Day Remaining";
                                 $subscriptionStatus = 1;
                                 echo '<p>',$remaining,'</p>';
                             }

                        elseif ($round_remaining <= 0) 
                            {
                                  $remaining = $course_registered." "."Expired"." ".$date_time;
                                  $subscriptionStatus = 0;
                                   echo '<p>',$remaining,'</p>';
                            }


                    }

//Check for most recent viewed video
 $sqlVideo = "SELECT `last_video` FROM users WHERE `user_id` = '".$user_id."'" ;
 $videoResult = $conn->query($sqlVideo);
 if ($videoRows = mysqli_fetch_assoc($videoResult))
 {
     $last_video = $videoRows['last_video'];
     $videoLink = "SELECT `chapter_id` FROM chapters WHERE `chapter_title` = '".$last_video."'";
     if ($chapteridResult = mysql_fetch_assoc(mysql_query($videoLink)));
     {
     $chapter_id = $chapteridResult['chapter_id'];
     }

     $videoLink = "SELECT `course_id` FROM chapters WHERE `chapter_title` = '".$last_video."'";
     if ($courseResult = mysql_fetch_assoc(mysql_query($videoLink)));
     {
     $course_id = $courseResult['course_id'];
     }
 }


}
4

2 に答える 2

2

関数course_menu()はあなたの を認識しません$user_id。それはその範囲外にあるためです。

globalこの問題を解決するには、キーワードを使用します。

function course_menu()
  {
   global $user_id;
   // your remaining code .........
于 2013-10-06T09:42:26.200 に答える