Author Topic: Coding Standards  (Read 5302 times)

0 Members and 1 Guest are viewing this topic.

Offline DaPhreak

  • Pre-Newbie
  • Posts: 9
    • View Profile
Coding Standards
« on: August 08, 2002, 04:15:38 PM »
Also,

is mir gestern mal aufgefallen :

1.) kann es sein das du leerzeichen benutzt anstatt tabulatoren ??  :?

2.) Ich würde die if-schleifen so schreiben (is bei über 400 Zeilen Code übersichtlicher und trotzdem is die datei kleiner als die Original) :roll:

Code: [Select]

if ($foo == 'bla')
{
    $test = 'foo';
}


3.) In der register.php ist mir aufgefallen, das die url_register Variablen fehlen .. kann das sein ???

Bin mir nicht sicher ob ich die geändert habe ....  :wink:

4.) Kann es sein, dass 4images mächtig viel Ressourcen (im ACP) benötigt ?

Ist mir nur so aufgefallen, da der Server direkt neben mir steht ...

Wäre es nicht besser die Ordnergröße in der DB zu speichern (Thumbnails und Media-Ordner) und beim Start des ACP ausgibt ...

Wäre nicht so belastend für den Server .....  :D  :D  :D  :)

mfg DaPhreak

Offline Jan

  • Administrator
  • 4images Guru
  • *****
  • Posts: 5.024
    • View Profile
    • 4images - Image Gallery Management System
Coding Standards
« Reply #1 on: August 09, 2002, 08:18:47 AM »
1. Auf jeden Fall

2. Ich finde es ger ade so viel übersichtlicher.

3. Ich kann mich nicht erinnern, dass dort jemals eine solche Variable gab ;)

4. Was meinst Du mit ACP? Was meinst Du mit Ordnergröße?

Gruß Jan
Your first three "must do" before you ask a question:
1. Forum rules
2. FAQ
3. Search

Offline DaPhreak

  • Pre-Newbie
  • Posts: 9
    • View Profile
Coding Standards
« Reply #2 on: August 09, 2002, 03:47:20 PM »
Quote from: Jan

1. Auf jeden Fall


Das macht doch die Dateien ziemlich groß ...
(ich red von den Leerzeichen am Anfang einer Zeile)

Quote

2. Ich finde es gerade so viel übersichtlicher.


Bei weniger als 200 Zeilen stimmt das ja auch :D :D :D

aber bei über 600 Zeilen Code ..... :twisted:

Da wird das mit der Zeit auch n bissl unübersichtlich .....

Quote

3. Ich kann mich nicht erinnern, dass dort jemals eine solche Variable gab ;)


Muss es aber geben  :lol: ,

denn ansonsten kann sich ja niemand registrieren ......  :roll:

In deinem Original Template (register_signup.html) steht ja schließlich auch folgendes :

Code: [Select]

<form method="post" action="{url_register}">
  <input type="hidden" name="action" value="register" />
  <input type="submit" value="{lang_agree}" class="button" />
</form>


Demnach musst du die Variable 'url_register' irgendwo vereinbaren / zuweisen ......  :roll:  :P

Quote

4. Was meinst Du mit ACP? Was meinst Du mit Ordnergröße?

Mit ACP meine ich das Admin Control Panel .... ("admin/index.php")

Dort wird ja auf der ersten Seite (Übersicht) die Ordnergröße in (kB/ MB) angezeigt ....

Und das belastet den Server sehr arg ... (zumindest bei mir :D; die Verzeichnisse sind ja schließlich auch mehr als 800 MB groß :D :D)

Wenn er jedes Mal die Verzeichnisgröße checken tut .....

Jetzt klar ???? ;)

mfg DaPhreak

Offline Jan

  • Administrator
  • 4images Guru
  • *****
  • Posts: 5.024
    • View Profile
    • 4images - Image Gallery Management System
Coding Standards
« Reply #3 on: August 09, 2002, 05:03:30 PM »
Naja, es bleibt sicher jedem selbst überlassen wie er seinen Code formatiert. Ich habs halt lieber kürzer, schmaler kompakter...

$url_register wird in page_header.php registriert.

Wenn Du das Auslesen der Dateigröße entfernen willst, suche in admin/home.php nach get_dir_size().

Gruß Jan
Your first three "must do" before you ask a question:
1. Forum rules
2. FAQ
3. Search