Quellcode Optimierung

[ShouTakumi]

Hallo, ich wollte mal nachfragen, in wie weit man den Quellcode noch "Optimieren" könnte, bzw. ob es gravierende Fehler etc. gibt… schon mal

<?php
require_once("./config.inc.php");
$datetime = date("Y-m-d H:i",(time() + ($time_difference * 3600)));

$connect = mysql_connect($dbserver, $dbuser, $dbpass);
if($connect = false){
    echo "$selectsvr_invalid";
    echo mysql_errno() . ": " . mysql_error() .
    "<br>\n";
}
$selectdb = mysql_select_db($dbname);
if($selectdb = false){
    echo "$selectdb_invalid";
    echo mysql_errno() . ": " . mysql_error() .
    "<br>\n";
}


$query_insert = "INSERT INTO $commtenttab
(newsid,user,comment,datetime)
VALUES
    ('$comment_newsid',
     '$comment_user',
     '$comment_comment',
     '$datetime')";
     
if (mysql_query($query_insert)) {
  print "$comment_valid";

  } else { print "$comment_invalid";
      echo mysql_errno() . ": " . mysql_error() .
    "<br>\n";
}
?>

[TobiaZ]

in erster linie würde ich sagen: korrigier gravierende fehler, wie das "vergleichen" zweier werte.
dann pack das ganze in eine klasse.
danach fällt mir auf, dass du aus welchem grund auch immer variablen in anführungszeichen steckst.
dann würde ich noch die vars aus strings lösen.
weiter würde ich mir ne ganze zeile sparen und statt dessen die mysql-datumsfunktionen ansehen (NOW())
würde auch noch den code vernünftig formatieren


aber eigentlich ist das forum zum besprechen von problemen...

[ShouTakumi]

Original geschrieben von TobiaZ
aber eigentlich ist das forum zum besprechen von problemen... [/B]

Na dann bitte closen...

[TobiaZ]

Na, jetzt möchte ich zumindest noch hören, dass du meine Vorschläge umgesetzt hast.

[ShouTakumi]

danach fällt mir auf, dass du aus welchem grund auch immer variablen in anführungszeichen steckst.

Ich geh mal davon aus, dass ist auf print "$comment_valid"; bezogen

korrigier gravierende fehler, wie das "vergleichen" zweier werte.

Versteh ich nicht

dann pack das ganze in eine klasse.

Bie klassen blick ich nicht wirklich durch...

weiter würde ich mir ne ganze zeile sparen und statt dessen die mysql-datumsfunktionen ansehen (NOW())

Wurde gemacht.

[TobiaZ]

Ich geh mal davon aus, dass ist auf print "$comment_valid"; bezogen

ja!

Versteh ich nicht

Habe ich mir gedacht, wenn ich mir so deine if()-"Abfragen" ansehe.

Bie klassen blick ich nicht wirklich durch...

Kann man lernen

[ShouTakumi]

Original geschrieben von TobiaZ
Habe ich mir gedacht, wenn ich mir so deine if()-"Abfragen" ansehe.

Dann sag doch was net passt
if($selectdb = false){

if($selectdb == false){

[TobiaZ]

dir ists doch aufgefallen

[ShouTakumi]

Peinlich

[Meillo]

Original geschrieben von ShouTakumi
if ($selectdb == false) {

if (!$selectdb) { 

... oder so

[TobiaZ]

if(!mysql_select_db($dbname))

[Meillo]

Original geschrieben von TobiaZ
if(!mysql_select_db($dbname))

full ack
... die orginal Source habe ich gar nicht durchgelesen