Doppelt hält besser?

Andynium · 08. Mai 2015 10:27

/lib/classes/class.admintheme.inc.php - ab Zeile 830

Hier klicken, um den Code zum Kopieren zu markieren

$config = cmsms()->GetConfig();
$config = cmsms()->GetConfig();

Klenkes · 08. Mai 2015 10:37

OH! MEIN GOTT!
Mir wird manchmal ganz schlecht von deinen morgendlichen News wink

Man sollte 2 identische Installationen haben, und nur die eine mit deinen Optimierungen verbessern. Und dann den Speedtest machen.

Wenn es wieder Winter wäre und ich mehr Zeit hätte...

Andynium · 08. Mai 2015 10:49

Tja, ich frag mich auch immer wieder, wieso gerade ich das finden muss, wo doch das Dev-Team (angeblich) hunderte Stunden mit Tests etc. zugebracht hat roll .

Klenkes schrieb:

Man sollte 2 identische Installationen haben, und nur die eine mit deinen Optimierungen verbessern. Und dann den Speedtest machen.

Ich formulier es mal ganz vorsichtig so - was glaubst du, womit ich derzeit meine Freizeit verbringe (außer Motorrad putzen) wink ?

Und da gibt es noch eine ganze Reihe Dinge, die es hier nicht als Performance-Tipps erscheinen werden ... einfach, weil es insgesamt, sagen wir mal, etwas "unglücklich" programmiert ist und zentral umgestellt werden muss, was aber den Umfang eines Tipps deutlich übersteigt. Die besondere Herausforderung liegt darin, dass der Endanwender unterm Strich davon nichts merken darf/sollte (außer der Performance wink ).

nicmare · 08. Mai 2015 18:10

setzt du dich echt hin und durchsuchst jede einzelne zeile code von cmsms?

NaN · 08. Mai 2015 19:07

setzt du dich echt hin und durchsuchst jede einzelne zeile code von cmsms?

Anscheinend macht es ja sonst keiner.

Andynium · 08. Mai 2015 21:08

nicmare schrieb:

setzt du dich echt hin und durchsuchst jede einzelne zeile code von cmsms?

Durchsuchen ist gut hmm - da ich kein Coding Pro bin, versuche ich zunächst, es zu verstehen.

Wenn man das jedoch versucht, stolpert man teilweise über ein Code-Chaos, das einem tieferen Verständnis diametral entgegen steht. Kein Wunder, dass bei der Unübersichtlichkeit und fehlenden Systematik so viele Fehler im Code sind.

Offensichtlich hat sich keiner der Devs jemals bei CMSMS DURCHGÄNGIG mit einem Coding Style Guide oder Code Beautifying beschäftigt. Und anstatt wenigstens eine Datei gescheit durchzuformatieren, wenn man schon mal dran arbeitet, hat (gefühlt) jeder nur seinem Mist hinzugefrickelt, natürlich in seinem Stil. Hab mittlerweile knappe 10 Stile ausmachen können.

Für mich intern hab ich sogar eine Hitliste der Code-Lesbarkeit big_smile - will da nicht weiter ins Detail gehen, aber nahezu perfekt lesbar ist der Source von Alby und Goran.

Andererseits stolpert man bei so einer intensiven Beschäftigung mit dem Code quasi automatisch derartige Fehler. Und entdeckt natürlich auch viel schneller Optimierungspotential wink . Eigentliche Dinge, die nach meinem Verständnis zum Tagesprogramm des Dev-Teams gehören - und das, bevor ich neue Funktionen hinzubastele. Die Code-Qualität hat in meinen Augen absolute Priorität.

Da werden im System für gleiche oder ähnliche Funktionen komplett unterschiedliche Lösungsansätze verwendet, anstatt sich auf einen festzulegen. Sorry - aber für mich ist Open Source und konsistenter Code (auch bei vielen Mitwirkenden) kein Ding der Unmöglichkeit.

Und zum dritten ist es natürlich für mich persönlich eine prima Schule PHP, CMSMS und ADOdb betreffend. Hab dabei schon ne ganze Menge erfahren, was ich so bislang noch nicht wusste.

Und wie sagt man so schön, wenn du willst, das etwas gut wird, dann mach es selbst big_smile .

Andynium · 09. Mai 2015 15:00

cyberman schrieb:

Offensichtlich hat sich keiner der Devs jemals bei CMSMS DURCHGÄNGIG mit einem Coding Style Guide oder Code Beautifying beschäftigt.

... was von diesem eher allgemeinen / wenig substantiellen Blafasel nachdrücklich bestätigt wird sad

http://forum.cmsmadesimple.org/viewtopic.php?f=6&t=5866

Forum für CMS/made simple

#1 08. Mai 2015 10:27

Doppelt hält besser?

#2 08. Mai 2015 10:37

Re: Doppelt hält besser?

#3 08. Mai 2015 10:49

Re: Doppelt hält besser?

#4 08. Mai 2015 18:10

Re: Doppelt hält besser?

#5 08. Mai 2015 19:07

Re: Doppelt hält besser?

#6 08. Mai 2015 21:08

Re: Doppelt hält besser?

#7 09. Mai 2015 15:00

Re: Doppelt hält besser?

Fußzeile des Forums