включать неиспользованную полную функцию или много включает - PullRequest
Новая
       15

включать неиспользованную полную функцию или много включает

0 голосов
/ 22 февраля 2012

Я нахожусь в процессе создания структуры авторизации PHP, которая стремится быть простой, но быстрой.Моя проблема проста, и я надеюсь, что для ее использования потребуется только одно включение, однако, если оно включено, оно может загружать ненужные функции.Поэтому мой вопрос в том, что лучше, включая все функции, в результате которых используется больше памяти, или файл, который имеет функцию включения этой функции, что приводит к уменьшению ненужных функций, но будет ли он использовать больше ресурсов ЦП?

напримермоя функция входа выглядит следующим образом: 

function login ($submit ='login', $user = 'user', $pass = 'pass') {
    if ($_POST[$submit]){

        $user = $_POST[$user];
    $pass = $_POST[$pass];

    if ($user && $pass){ //if user and pass is enterered

            require("auth_vars.php"); //require MySQL conection settings
            mysql_connect($auth_mysql_server, $auth_mysql_user, $auth_mysql_pass); //connect to MySQL
        mysql_select_db($auth_mysql_db); // select MySQL database

            $pass = md5($pass); // hash password

            $query = mysql_query("SELECT * FROM $auth_mysql_table WHERE user='$user'"); // run query
            $numrows = mysql_num_rows($query);

            if ($numrows == 1){ //check if user exists
                $row = mysql_fetch_assoc ($query);
                $dbid = $row[$auth_mysql_id_row];
                    $dbuser = $row[$auth_mysql_user_row];
                    $dbpass = $row[$auth_mysql_pass_row];

                    if ($pass == $dbpass){ // if password is equal to the one in the database start session
                        //set session information
                        $_SESSION['userid'] = $dbid;
                        $_SESSION['username'] = $dbuser;


                        header("Location:$auth_path_loggedin"); // goto logged in page

                    }
                    else return (3);
            }
            else return (2);

            mysql_close(); // close MySql connection
        }
        else return (1);
}else {
    // If the user clicks the "Log Out" link.
    if(isset($_GET['status']) && $_GET['status'] == 'loggedout') {
        logout();//logout
        return 4;
        //else if the user is denied
    }else if(isset($_GET['status']) && $_GET['status'] == 'denied') {
        logout(); //to keep it clean in case of errors
        return 5;
    }
}}

теперь, если функция входа не нужна, было бы лучше сделать это: 

function login ($submit ='login', $user = 'user', $pass = 'pass') {
    include ("myloginfunction.php");
    login_function($submit, $user, $pass);
}

илион используется для чрезмерного использования ЦП по сравнению с объемом памяти, необходимым для включения всей функции

1 Ответ

1 голос
/ 22 февраля 2012

Ни один подход не намного лучше, чем другой.Вы можете получить больше, переместив создание соединения mysql из функции входа в систему, чтобы все ваши запросы SQL использовали одно и то же соединение, вместо создания нескольких.

...