phpBB.de

Miriam hat geschrieben:Zum Speichern der Werte kannst Du sicherlich das Objekt $user heranziehen / benutzen.
Meiner Meinung nach ist das aber wenig hilfreich, denn das Objekt enthält nur Daten des angemeldeten Users und nicht des Users allgemein (also nicht des Posters / Members etc. pp.).
Für eine "globale" Lösung könnte es ein guter Ansatz sein, eine entsprechende Funktion in der /includes/functions.php zu definieren, die die Daten in Abhängigkeit von der (abgefragten) User-ID bereitstellt.

Miriam hat geschrieben:Ob es sinnvoll ist oder nicht, kannst Du nur entscheiden.

Scheibar geht es hier auch um Performance-Fragen: Das User Objekt wird jedes mal erstellt und alle dafür notwendigen SQL Queries werden auch jedesmal ausgeführt. Deiner Idee folgend, würden auch die zusätzlichen SQL Queries für Deine Modifikation jedesmal mit ausgeführt, obwohl das garnicht notwendig ist. Von daher meine ich, daß die Abfrage / Ausführung der Queries performanter ist, wenn sie im Bedarfsfalle ausgeführt werden.
Wenn Du nun noch geschickt Indizes setzt und die Abfragen entsprechend den erwarteten Ergebnissen einschränkst, dann bist Du mit einer separaten Funktion besser beraten.

Ich habe das gerade mal getestet: Ein SELECT in einer DB mit 394285 Einträgen. Es wurden 20727 Einträge zurück gegeben. Insgesamt auerte die Abfrage 0,0007 Sek.
Da sollte der DB Server Deine 20 Ergebnisse relative schnell bereitstellen können.

// MOD Achievements
function get_userachievement($user_id, $achievement_id = false)
{
    if($achievement_id != false)
    {
        $sql = 'SELECT *
                FROM
                    phpbb_userachievements
                WHERE
                    achievement_id='.$achievement_id.'
                AND
                    user_id='.$user_id.'';
        $result = $db->sql_query($sql);

        while ($row = $db->sql_fetchrow($result))
        {
            $db->sql_freeresult($result);
            return true;
        }
        $db->sql_freeresult($result);
        return false;
    }
    else
    {
        $sql = 'SELECT *
                FROM
                    phpbb_userachievements
                WHERE
                    userid='.$user_id.'';
        $result = $db->sql_query($sql);

        $achievements = array();
        while ($row = $db->sql_fetchrow($result))
        {
            $achievements[] = $row['achievement_id'];
        }
        $db->sql_freeresult($result);
        return $achievements;
    }
}
// END Achievements  

ob er ein neues Achievement bekommen hat,

• Code: Alles auswählen

      global $db, $config; // sonst kennt die Funktion das Objekt $db nicht und es funktionieren keine DB Abfragen
      if($achievement_id) // wenn sie nicht false ist, wie vorgegeben, dann kann sie nur noch true sein, wenn eine ID übergeben wurde.
      {
          $sql = 'SELECT user_id
                  FROM ' . USER_ACHIEVEMENTS_TABLE . " 
                  WHERE achievement_id = $achievement_id
                      AND user_id = $user_id";
          // Definiere eine Tabelle in der includes/constants.php, hier: USER_ACHIEVEMENTS_TABLE oder benutze $table_prefix,
          // so umgehst Du das Problem frei wählbarere Tabellenpräfixe.
          // Die Notation und Layout der SQL Query erfolgen nach den Vorgaben in der /docs/coding-guidelines.html
          
          $result = $db->sql_query($sql);
  
          $return = ($db->sql_fetchrow($result)) ? true : false; 
          // Du brauchst keine WHILE Schleife, wenn Du nur überprüfen willst, ob es ein Ergebnis gibt oder nicht. 
          // return beendet die Schleife ohnehin.
          $db->sql_freeresult($result);
          return $return;
      } 

Ach mensch, ich seh schon, viel Arbeit für mich