Переменные из функции не определены

Question

Я помещаю весь сценарий разбиения на страницы в функцию, чтобы я мог использовать его больше раз. Код длинный, но есть только одна часть, с которой у меня проблемы.

После вызова функции paginate($connection, "categories"); я использую

$sql = "SELECT * FROM categories ORDER BY cat_name LIMIT $start, $limit";

etc..

и я получаю эти ошибки.

Примечание: неопределенная переменная: начало Примечание: неопределенная переменная: ограничить вас есть ошибка в вашем синтаксисе SQL; проверьте руководство, которое соответствует ваша версия сервера MySQL для правильный синтаксис для использования рядом с '' в строке 1

Он не распознает переменные, которые у меня есть в функции, которую я вызвал над запросом. Я никогда раньше не помещал столько кода в функцию, поэтому не уверен, что мне нужно делать. Я попытался вернуть эти неопределенные переменные (в функции), но это не сработало. Я также пытаюсь избежать использования глобальных переменных - я слышал, что они устарели.

Ниже приведена актуальная функция

function paginate($connection, $tableName) {

//Pagination
$targetpage = "http://localhost/website/all_categories.php";    
$limit = 4; //sets how many rows to display on each page

//count rows
$sql = "SELECT COUNT(*) as num FROM $tableName";
$total_pages = $connection->query($sql) or die(mysqli_error($connection)); 
$row = $total_pages->fetch_assoc();
$total_pages = $row['num'];

//if there's no page number, set it to the first page
$stages = 3;
$page = isset($_GET['page']) ? $_GET['page'] : 0;
$start = empty($page) ? $start = 0 : $start = ($page - 1) * $limit;

// Initial page num setup
    if ($page == 0){$page = 1;}
    $prev = $page - 1;  
    $next = $page + 1;                          
    $lastpage = ceil($total_pages/$limit);      
    $LastPagem1 = $lastpage - 1;                    


    $paginate = '';
    if($lastpage > 1)
    {   

        $paginate .= "<div class='paginate'>";
        // Previous
        if ($page > 1){
            $paginate.= "<a href='$targetpage?page=$prev'>previous</a>";
        }else{
            $paginate.= "<span class='disabled'>previous</span>";   }



        // Pages    
        if ($lastpage < 7 + ($stages * 2))  // Not enough pages to breaking it up
        {   
            for ($counter = 1; $counter <= $lastpage; $counter++)
            {
                if ($counter == $page){
                    $paginate.= "<span class='current'>$counter</span>";
                }else{
                    $paginate.= "<a href='$targetpage?page=$counter'>$counter</a>";}                    
            }
        }
        elseif($lastpage > 5 + ($stages * 2))   // Enough pages to hide a few?
        {
            // Beginning only hide later pages
            if($page < 1 + ($stages * 2))       
            {
                for ($counter = 1; $counter < 4 + ($stages * 2); $counter++)
                {
                    if ($counter == $page){
                        $paginate.= "<span class='current'>$counter</span>";
                    }else{
                        $paginate.= "<a href='$targetpage?page=$counter'>$counter</a>";}                    
                }
                $paginate.= "...";
                $paginate.= "<a href='$targetpage?page=$LastPagem1'>$LastPagem1</a>";
                $paginate.= "<a href='$targetpage?page=$lastpage'>$lastpage</a>";       
            }
            // Middle hide some front and some back
            elseif($lastpage - ($stages * 2) > $page && $page > ($stages * 2))
            {
                $paginate.= "<a href='$targetpage?page=1'>1</a>";
                $paginate.= "<a href='$targetpage?page=2'>2</a>";
                $paginate.= "...";
                for ($counter = $page - $stages; $counter <= $page + $stages; $counter++)
                {
                    if ($counter == $page){
                        $paginate.= "<span class='current'>$counter</span>";
                    }else{
                        $paginate.= "<a href='$targetpage?page=$counter'>$counter</a>";}                    
                }
                $paginate.= "...";
                $paginate.= "<a href='$targetpage?page=$LastPagem1'>$LastPagem1</a>";
                $paginate.= "<a href='$targetpage?page=$lastpage'>$lastpage</a>";       
            }
            // End only hide early pages
            else
            {
                $paginate.= "<a href='$targetpage?page=1'>1</a>";
                $paginate.= "<a href='$targetpage?page=2'>2</a>";
                $paginate.= "...";
                for ($counter = $lastpage - (2 + ($stages * 2)); $counter <= $lastpage; $counter++)
                {
                    if ($counter == $page){
                        $paginate.= "<span class='current'>$counter</span>";
                    }else{
                        $paginate.= "<a href='$targetpage?page=$counter'>$counter</a>";}                    
                }
            }
        }

                // Next
        if ($page < $counter - 1){ 
            $paginate.= "<a href='$targetpage?page=$next'>next</a>";
        }else{
            $paginate.= "<span class='disabled'>next</span>";
            }

        $paginate.= "</div>";       


}
 echo $total_pages.' Results';
 // pagination
 echo $paginate;


}//end function

Answer 1

(Примечание - это не обязательно проблема, но ...)

$start = empty($page) ? $start = 0 : $start = ($page - 1) * $limit;

должно быть

$start = empty($page) ? 0 : ($page - 1) * $limit;

Answer 2

Вы устанавливаете $start и $limit внутри функции, но вы не возвращаете эти переменные.Переменные внутри функции имеют свои собственные локальные scope , и поэтому не могут быть доступны извне функции.

Вам необходимо вернуть переменные , которые вам нужны внефункция.Поскольку вам требуется более одной переменной, вы можете вернуть их в ассоциативный массив.Например, если вы поместите это как последнюю строку вашей функции:

return array( 'start' => $start, 'limit' => $limit );

... вы можете вызвать функцию следующим образом:

$result = paginate($connection, "categories");

... и использоватьрезультат как это:

$sql = 'SELECT * FROM categories ORDER BY cat_name LIMIT ' .
    $result['start'] . ', ' . $result['limit'];

Answer 3

edit : вы явно определяете $start внутри своей функции.Если он определен до вашего оператора SQL, он, очевидно, будет доступен.Однако я сомневаюсь, что ваша функция нумерации страниц будет очень полезна, если вы не передадите ее $start или $end.Вы также можете определить значения по умолчанию с помощью этого синтаксиса:

function paginate($connection, $tableName, $start=0, $end="ALL") {

original следует

Функции имеют свою собственную область;Вы должны передать функции переменные, определенные вне ее:

function paginate($connection, $tableName, $start, $end) {

Единственным исключением являются суперглобальные $GLOBALS, $_GET, $_POST, $_COOKIE, $_REQUEST, $_SESSION,$_FILES, $_ENV и $_SERVER.

Вы также можете получить доступ к функциям, определенным вне любой функции, через суперглобальный массив $GLOBALS или добавив global $var; в начале реализации функции.Однако вам не рекомендуется делать это, за исключением (возможно) директив конфигурации области приложения.