[Jake.Nu]

Ursprungligen postat av Legendarisk

Hej,
Håller på att snickra ihop ett simpelt inloggnings system i PHP och försöker göra själva kopplingen till databasen så enkel som möjligt.

Jag har flertalet funktioner vid själva inloggningen som behöver connecta till databasen (exempelvis kolla att username finns osv).

Därför har jag gjort en egen funktion för detta i en separat fil som ser ut som följande:

PHP-kod:

function connect_to_db() {

    $db = new mysqli('localhost', 'namn', 'lösen', 'databasen');

    //Check db error
    if (mysqli_connect_errno()) {
        echo 'Sorry, we are experiencing connection problems.';
        exit;
    } 
    
    return $db;
} 


varav de andra funktionerna sedan använder db för att köra queryn.

Det jag undrar är om det är ok i olika avseenden att göra på detta sätt, eller finns det något annorlunda sätt som är att föredra? Tänker på både säkerhetsaspekter och hastighet.

Tack på förhand.

Din kod går utmärkt att bygga vidare på, det som gäller är bara att du inkluderar filen där funktionen finns + inte skapar en ny anslutning för varje fråga.

PHP-kod:

<?php
function connect_to_db() {
    static $db = null;
    if (is_null($db) || !$db->ping()) {
        $db = new mysqli('localhost', 'namn', 'lösen', 'databasen');
        if (mysqli_connect_errno()) {
            echo 'Sorry, we are experiencing connection problems.';
            exit();
        }
    }
    return $db;
}

[Legendarisk]

Ursprungligen postat av Jake.Nu

Din kod går utmärkt att bygga vidare på, det som gäller är bara att du inkluderar filen där funktionen finns + inte skapar en ny anslutning för varje fråga.

PHP-kod:

<?php
function connect_to_db() {
    static $db = null;
    if (is_null($db) || !$db->ping()) {
        $db = new mysqli('localhost', 'namn', 'lösen', 'databasen');
        if (mysqli_connect_errno()) {
            echo 'Sorry, we are experiencing connection problems.';
            exit();
        }
    }
    return $db;
}

Tack för tipset!

Vad innebär det att vi sätter $db till NULL där i början?

Skulle ett alternativ kunna vara att man kör $db->close() i funktionerna som anropar connect_to_db funktionen?

[Conny Westh]

Ursprungligen postat av Legendarisk

Tack för tipset!

Vad innebär det att vi sätter $db till NULL där i början?

Skulle ett alternativ kunna vara att man kör $db->close() i funktionerna som anropar connect_to_db funktionen?

Nej, en variabel som är static behåller sitt värde efter att funktionen avslutas
så att sätta den till null görs bara första gången den används.

Nästa gång funktionen anropas hämtas den befintliga Connection och du får därmed bara en Connection till databasen, vilket spar resurser och exekveringstid.