Если оператор не работает в функции

Question

Не уверен, что это распространенная проблема или нет, но я столкнулся с ней. При попытке поместить оператор if/else в функцию, он просто не работает. Но когда do/else без функции, он работает (Итак, я знаю, что if/else верен).

У меня есть 2 отдельные документы, Home.php и Functions.php вот что у меня внутри этих файлов»

Home.php (РНР)

<?php 
$db = new PDO("mysql:host=localhost;dbname=TEST;charset=utf8","Xero",""); 
include("Functions.php"); 

menu(); 
?> 

И тогда мы имеем Functions.php (РНР)

<?php 
$db = new PDO("mysql:host=localhost;dbname=TEST;charset=utf8","Xero",""); 

function menu() { 
    echo " 
    <div id='menu'> 
     <div id='user-menu'> 
    "; 
     // Check AutoLogin-User then : 
     $getAutoUser = $db->prepare("SELECT * FROM `users-PHP_FNC` WHERE `ip`='" . $_SERVER['REMOTE_ADDR'] . "'"); 
     $getAutoUser->execute(); 
     if ($autoUser = $getAutoUser->fetch(PDO::FETCH_ASSOC)) { 
      echo " 
      " . $autoUser['user'] . " 
      "; 
     } 
     // If no AutoLogin-User was found: 
     else { 
      echo " 
      Test 
      "; 
     } 
    echo " 
     </div> 
    </div> 
    "; 
} 
?> 

Не знаете, почему это не работает. Не разрешено ли иметь какие-либо выражения в функции PHP?

Answer 1

На самом деле, вы можете быть уверены, что if ... else заявлений Внутри функции.

Таким образом, проблема заключается в другом ... Но есть еще одна вещь, которую вы должны сделать:

// Replace this 
if ($autoUser = $getAutoUser->fetch(PDO::FETCH_ASSOC)) 

// By this 
if ($autoUser == $getAutoUser->fetch(PDO::FETCH_ASSOC)) 

Почему?

Потому что, когда вы говорите if ($something = $somethingElse) вы говорите PHP разные вещи в определенном порядке:

первый => присвоить текущее значение $ somethingElse до $ что-то

второй => тест если $ something и $ somethingElse имеют одинаковое значение

2-й становится очевидно истинным из-за 1-го.

Чтобы обойти 1-й шаг, просто используйте == вместо =.

Answer 2

$db не входит в сферу применения. Вы должны передать его в Я хотел бы сделать мой код больше как:.

// connect.php - should be in a separate secure folder I'll call restricted 
<?php 
function db(){ 
    return new PDO('mysql:host=localhost;dbname=TEST;charset=utf8', 'Xero', ''); 
} 
?> 

// index.php 
<?php 
include_once 'restricted/connect.php'; 
function menu(){ 
    $db = db(); 
    echo "<div id='menu'><div id='user-menu'>"; 
    // Check AutoLogin-User then : 
    $getAutoUser = $db->prepare("SELECT * FROM `users-PHP_FNC` WHERE ip='{$_SERVER['REMOTE_ADDR']}'"); 
    $getAutoUser->execute(); 
    if($autoUser = $getAutoUser->fetch(PDO::FETCH_ASSOC)) { 
    echo $autoUser['user']; 
    } 
    else{ 
    // If no AutoLogin-User was found: 
    echo 'Test'; 
    } 
    echo '</div></div>'; 
}