net-developers.de/forum

[Torsten]

Hallo Simon.

Ich lese jetzt schon eine Weile dein schönes MVC-Tutorial und sehe dabei immer Codekonstrukte, die du vielleicht einfach umdrehen kannst. Ein Beispiel aus der Config-Klasse:

Zeilennummern einblenden / ausblenden | Anzeigen / Verstecken | Alles auswählen

1.  
2.     public function readINI($path)
3.     {
4.         if (file_exists($path))
5.         {
6.             $ini = parse_ini_file($path);
7.             foreach ($ini as $key => $value)
8.             {
9.                 $this->set($key, $value);
10.             }
11.         }
12.         else
13.         {
14.             throw new FW_Exception("Die ini-Datei existiert nicht!");
15.         }
16.     }
17.  

Das kannst du einfacher so schreiben und es ist mindestens genauso verständlich.

Zeilennummern einblenden / ausblenden | Anzeigen / Verstecken | Alles auswählen

1.  
2.         if (!file_exists($path))
3.         {
4.             throw new FW_Exception("Die ini-Datei existiert nicht!");
5.         }
6.         $ini = parse_ini_file($path);
7.         foreach ($ini as $key => $value)
8.         {
9.             $this->set($key, $value);
10.         }
11.  

Vielleicht ist das für dich von Interesse.

[Simon]

Hi,

danke für deinen Beitrag!

Ja, da hast du Recht. Ich finde es aber nicht so schlimm, die "längere" Variante zu benutzen. Ist wahrscheinlich einfach nur Geschmackssache!

MfG
Simon

[Torsten]

Schlimm ist das natürlich nicht. Beim Lesen sind mir eben manche Verschachtelungen aufgefallen, die nicht hätten sein müssen. Darum hab ich das mal erwähnt.