Roligt!

Kul när man börjar upptäcka saker på nytt. Leker med PDO och annat till mysql.. skriver om det mesta sql frågorna. Plötsligt tar alla frågor mindre plats .. effektivare kod .. roligare helt enkelt! ;)

Tror jag fick funktionen nedan att fungera, vet dock inte om det är 100% rätt

function log_this($log_user_name,$log_action)
{
	$result = DB::getInstance()->query("
        INSERT INTO " . TBL_PREFIX ."_log 
        SET log_user_id = 
        (SELECT user_id FROM " . TBL_PREFIX ."_users WHERE user_name = '" . $log_user_name . "'), 
        log_action = '" . $log_action . "'
        ");
}

Har kört för länge i gamla spår.. nu e det jul igen :P

Om du sen även lägger till prepared statements så blir det ännu enklare, och säkrare.
Kika på prepare() och execute() i PDO.

Oj! Vad smart att göra en funktion av en SQL-fråga?! Funkar ju perfekt om man bara ska spara smågrejer? Eller?

Koolt i alla fall :)

oskars skrev:

Oj! Vad smart att göra en funktion av en SQL-fråga?! Funkar ju perfekt om man bara ska spara smågrejer? Eller?

Inte riktigt det som Fuel syftade på.. Snarare att det är effektivt att jobba mot PDO. Att ha en SQL-fråga inbakad i en funktion är inte särskilt ovanligt. Men visst, du får gärna tycka att det är coolt ;)

Förstår att det inte var det Fuel menade, men jag lärde mig något nytt :)

oskars skrev:

Förstår att det inte var det Fuel menade, men jag lärde mig något nytt :)

Du bör kappsla in så mycket som möjligt i funktioner, för det underlättar återanvänding av kod. Kör du OOP så är det klasser och funktioner (metoder) som gäller. :)