PHP mysql abfrage optimieren

[Blackbenji]

hallo,

ich habe in einer datenbank verschiedene datensätze die sich vorallem durch " id "'s unterschieden.

eine der id's sind die account kategorien. der user hat die möglichkeit entweder für einen account alle einträge anzeigen zu lassen oder sich alle anzuzeigen.

mein problem ist nun die sql abfrage.

$db_result = $this->database->runSQLQuery("SELECT * FROM ".GLOBAL_VALUES_DB." WHERE `user_id` = '".$_SESSION['id']."' AND `enter_date` LIKE '".$enter_date."%' AND `account_id` = '".$_SESSION['account']."' ");

das setzen der session erfolgt dann wie folgt:

	            if(!isset($_SESSION['account'])) {
		            if($_GET['account']) {
			            $_SESSION['account'] = $_GET['account'];
		            } else {
			            $_SESSION['account'] = $globals->set_account();
		            }
	            }

das aufrufen der classe:

	public function set_account() {
		return ("*");
	}

wenn ich account=1 oder account=2 setze, funktioniert es.
das problem ist nun dass default: * nichts bringt. kann mir jemand hier helfen?

 

[Yuuri]

account_id = '*' funktioniert nicht. Lass die Prüfung einfach weg. Platzhalter sind übrigens % und _ und funktionieren nur bei Strings mit LIKE.

 

[Blackbenji]

genau, das * bringt nichts. sagte ich ja bereits.
wenn ich die prüfung weglasse, werden mir aber alle einträge angezeigt und ich kann nicht filtern.

 

[Yuuri]

Du willst doch mit Sicherheit alle Einträge unabhängig der account_id oder? Dann lass AND account_id = xy weg, dann bekommst du alle unabhängig der account_id.

 

[Blackbenji]

da wären wir wieder bei meinem ersten post:

der user hat die möglichkeit entweder für einen account alle einträge anzeigen zu lassen oder sich alle anzuzeigen.

 

[Yuuri]

Naja sicher.

Mit Account-Filter:

SELECT *
FROM global_values
WHERE
	`user_id` = $_SESSION['id'] AND
	`enter_date` LIKE '$enter_date%' AND
	`account_id` = '".$_SESSION['account']."'

Ohne Account-Filter:

SELECT *
FROM global_values
WHERE
	`user_id` = $_SESSION['id'] AND
	`enter_date` LIKE '$enter_date%'

$this->database->runSQLQuery("SELECT * FROM ".GLOBAL_VALUES_DB."
WHERE `user_id` = '".$_SESSION['id']."' AND
`enter_date` LIKE '".$enter_date."%' ".($alle_filtern ? :
    '' :
    "AND `account_id` = '".$_SESSION['account']."'") );

 

[Blackbenji]

kannst du vielleicht noch kurz erläutern was der letzte code-block zu sagen hat? im speziellem

.($alle_filtern ? :
'' :
"AND `account_id` = '".$_SESSION['account']."'") );

sowas hab ich bislang noch nicht gesehen gehabt.

 

[Yuuri]

Das ist eine verkürzte If-Abfrage.

if( $alle_filtern )
  $var .= '';
else
  $var .= "AND `account_id` = '".$_SESSION['account']."'"

 

[Blackbenji]

hmm,
das scheint bei mir nicht zu funktionieren.

$db_result = $this->database->runSQLQuery("SELECT * FROM ".GLOBAL_VALUES_DB." WHERE `user_id` = '".$_SESSION['id']."' AND `enter_date` LIKE '".$enter_date."%'. ($alle_filtern ? : '' : AND `account_id` = '".$_SESSION['account']."' ) ");

wenn ich die zeile 1zu1 bei dir rauskopiere, zeigt er mir diverse fehler an.

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '. ( ? : '' : AND `account_id` = '1' )' at line 1 (SELECT * FROM calculator_values WHERE `user_id` = '1' AND `enter_date` LIKE '2012-04-01%'. ( ? : '' : AND `account_id` = '1' ) )

 

[Yuuri]

Du musst den String beenden für die Abfrage. Dir fehlt das Anführungszeichen in $enter_date,"%'".($alle_filtern.

 

[Blackbenji]

wenn ich die zeile 1zu1 übernehme kommt es zu fehlern die ich nicht ganz verstehe:

1. expected )
2. expected simicolon
expected statement

 

[booyakaxx]

kompakte Schreibweise der if ist falsch. So ist es richtig:

$alle_filtern?' ':"AND 'account_id' = '".$_SESSION['account']."'"