[bitte-mal-anschauen] Stil-frage

[david_kri]

Ich wollte euch mal fragen was Ihr von dem Stil der Programmierung haltet. Bitte nehmt mich nicht auseinander
Mein Mini-Template-Versuchssystem:
auszug aus userdata.tpl
==========================================
<tr>
<td>Name</td>
<td><input type='Text' name='name' value='[NAME]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Vorname</td>
<td><input type='Text' name='vorname' value='[VORNAME]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Strasse</td>
<td><input type='Text' name='strasse' value='[STRASSE]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Ort / Stadt</td>
<td><input type='Text' name='ort' value='[ORT]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Postleitzahl</td>
<td><input type='Text' name='plz' value='[POSTLEITZAHL]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Lizenznummer / Vermittlernummer</td>
<td><input type='Text' name='lizenz' value='[LIZENZNUMMER]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Email-Adresse</td>
<td><input type='Text' name='email' value='[EMAIL]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Passwort(Verschlüsselt)</td>
<td><input type='Text' name='passwd' value='[PASSWORT]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Telefonnummer</td>
<td><input type='Text' name='telefon' value='[TELEFON]' class="[TEXTFIELD]"></td>
</tr>

<tr>
<td>Faxnummer</td>
<td><input type='Text' name='fax' value='[FAX]' class="[TEXTFIELD]"></td>
</tr>

=================================================
und die funktion die die Platzhalter ersetzt:

function user_data()
{

    require("settings.php");

    $DB = new DB;

    $DB->ConnectDB("$HOST","$USR_NAME","$PASSWD","$DBNAME");

    $DB->Query("SELECT * FROM `table` WHERE login = '$u_name'");

    $DB->FetchRow();


    $DB1 = new DB;

    $DB1->ConnectDB("$HOST","$USR_NAME","$PASSWD","$DBNAME");

    $DB1->Query("SELECT * FROM `design`");

    $DB1->FetchRow();

    $filename = "./tmpl/userdata.tpl";

    $varr1  = file_get_contents($filename);

    $search  = array( '[NAME]', '[VORNAME]', '[STRASSE]','[ORT]','[POSTLEITZAHL]','[LIZENZNUMMER]','[EMAIL]','[PASSWORT]','[TELEFON]','[FAX]','[USERTABLE_CLASS]','[TEXTFIELD]');

    $replace = array($DB->result[1], $DB->result[2], $DB->result[3], $DB->result[4],$DB->result[5],$DB->result[7],$DB->result[8],'********',$DB->result[10],$DB->result[11],$DB1->result[0],$DB1->result[1]);

    $neu = str_replace($search, $replace, $varr1);

    print $neu;

    $DB->CloseLink();

    $DB1->CloseLink();
    } 

[Uzimaster]

2 Verbindungen zur Datenbank sind doch völlig unnötig. Das kann man auch mit einer machen. (Wenn die gleiche DB ausgelesen wird)

Ich kenn zwar deine Db-Klasse nicht aber das ist auf jedenfall möglich. Eventuell müssen da paar Änderungen vorgenommen werden.

[david_kri]

Es geht auch mehr um das ersetzen der Platzhalter mit dem Packen in Arrays. Das mit der DB-> Es sind tatsächlich 2 Datenbanken.

[Abraxax]

so wie du das ersetzen gemacht hast, ist schon ok.

aber ich würde dir empfehlen nicht nur [platzhalter] zu verwenden. die [] sind schliesslich auch bei arrays innerhalb von formularen wichtig. diese möglichkeit nimmst du dir damit leider. verwendet lieber {{platzhalter}} statt dessen.

ausserdem bist du mit deiner jetzigen form sehr gebunden, was die TAGs innerhalb deines html-codes betrifft. eine schöne template-klasse kann dir da sehr viel nützlicher sein.

BTW
*VERSCHIEB* nach ot