xtDBquery flexibel machen wie xtc_db_fetch_array

[noRiddle]

Vorab wofür es sinnvoll wäre:
In letzter Zeit habe ich Module gebaut oder umgebaut, die bestimmte Artikel in begrenzter Anzahl aus der DB holen, entweder sortiert nach einem DB-Feld wie products_id oder at random, also zufällig.
Bei zufälliger Sortierung ist die Nutzung des Cache und DB-Cache nicht möglich.
Je nach Einstellung im Modul kann ich nun in der Funktion xtc_db_fetch_array() den zweiten Parameter true oder false setzen und somit die Cache-Verwendung steuern. Mit xtDBquery() geht dies jedoch nicht, sodaß man es folgendermaßen umschreiben muß:
Beispiel:

$use_cache = defined('MODULE_BLABLABLA_USE_SOANDSO') && MODULE_BLABLABLA_USE_SOANDSO = 'true' ? false : true;
$use_cache ? $my_query = xtDBquery(SQL) : $my_query = xtc_db_query(SQL);

während man bei xtc_db_fetch_array() schlicht dies machen kann:

$my_array = xtc_db_fetch_array($my_query, $use_cache);

Es wäre also gut auch in der Funktion xtDBquery() einen Steuerparameter zu haben.
Leider müsste man hier den Default auf true setzen, anstatt wie bei xtc_db_fetch_array() auf false, um nicht 1. viel Code ändern zu müssen und 2. irgendwelche Erweiterungen und Module zu brechen wo erstgenannte Funktion verwendet wird.

Gebe zu, mein Vorschlag ist diskussionswürdig.

Gruß,
noRiddle

[noRiddle]

Korrektur:
Meinte natürlich verständlicher anstatt

$use_cache ? $my_query = xtDBquery(SQL) : $my_query = xtc_db_query(SQL);

so

$my_query = $use_cache ? xtDBquery(SQL) : xtc_db_query(SQL);

Gruß,
noRiddle

[Gerhard Waldemair]

Du hast doch bereits die Möglichkeit der Unterscheidung mit

xtc_db_query (ohne Caching)

oder

xtDBquery (mit Caching)

[Torsten Riemer]

Ich sehe darin ehrlich gesagt keinen Vorteil, da wir die Funktionen beide bereits haben und es nur um 27 Zeichen geht, die man einsparen könnte.

$use_cache = defined('MODULE_BLABLABLA_USE_SOANDSO') && MODULE_BLABLABLA_USE_SOANDSO = 'true' ? false : true;
$my_query = $use_cache ? xtDBquery($my_query) : xtc_db_query($my_query);

vs (fiktive Funktion xtDBquery mit zweitem Parameter für Cache):

$use_cache = defined('MODULE_BLABLABLA_USE_SOANDSO') && MODULE_BLABLABLA_USE_SOANDSO = 'true' ? false : true;
$my_query = xtDBquery($my_query, $use_cache);

[noRiddle]

@GTB
Schon klar, aber ich sagte ja, wenn du ein Modul hast wo du sowohl mit als auch ohne Cache haben kannst, eben wegen Einstellungsmöglichkeiten at random oder normale Sortierung.

@Tomcraft
Ja, schon klar, Code-Ersparnis ist nicht das Argument.
Mir ging es eher um konsistente Vorgehensweisen.

Aber ich sagte ja auch "ist diskussionswürdig", war nur so 'ne Idee...

Gruß,
noRiddle

[Volker Strähle]

Und warum nicht xtc_db_query($query, $link='db_link', $chaching=false) ?

[Gerhard Waldemair]

Dann halt so:

$function = 'xtc_db_query';
if (defined('MODULE_BLABLABLA_USE_SOANDSO') && MODULE_BLABLABLA_USE_SOANDSO = 'true') {
  $function = 'xtDBquery';
}
$my_query = $function($my_query);