| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 0000110 | Branch 0.2.1-FINAL | Feature Request | public | 2009-03-09 07:09 | 2012-11-28 19:31 | ||||||||
| Reporter | Quix0r | ||||||||||||
| Assigned To | Quix0r | ||||||||||||
| Priority | none | Severity | feature | Reproducibility | always | ||||||||
| Status | assigned | Resolution | open | ||||||||||
| Platform | PHP5-Latest | OS | Linux | OS Version | 2.6 | ||||||||
| Product Version | 0.2.1-FINAL | ||||||||||||
| Target Version | 0.2.1-FINAL | Fixed in Version | |||||||||||
| Summary | 0000110: Code-Stil - Was neu ist, was alt ist | ||||||||||||
| Description | ALT: if ($bla == $blub) { // Do something } else { // Do something else } NEU: if ($bla == $blub) { // Do something } else { // Do something else } | ||||||||||||
| Additional Information | Variablen oder Array-Elemente werden immer mit z.B. isset() abgefragt. Es werden Funktionen, wie is_array() usw. verwendet. Zum Entwickeln bitte immer error_reporting(E_ALL | E_STRICT) verwenden. | ||||||||||||
| Tags | No tags attached. | ||||||||||||
| Attached Files |
| ||||||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||||||||
|
|||||||||||||||||||||
 Relationships |
| Notes |
|
|
2009-03-09 11:14 |
Hm ist das nicht umgekehrt oder hab ich an den falschen Stellen bis jetzt gearbeitet? Ich finde ja die Schreibweise if ($bla == $blub) { // Do something } else { // Do something else } sowieso übersichtlicher. |
|
Quix0r (administrator) 2009-07-26 18:49 |
Bitte nach dem Stil "NEU" aufbauen, das ist kompakter und die Uebersichtlichkeit geht noch. Ich muss dazu sagen, dass es an einigen Stellen mit if-elseif-elseif-else-Blocken mit dem alten Stil sehr lang wird und dadurch unuebersichtlich wird, beim neuen Stil ist jedoch die oeffnende Klammer bei sehr langen if-Abfragen nicht sichtbar. :( Nun, die Entscheidung sollte bleiben. Sonst stuelpen wir staendig den Code um und kommen nicht zum Fixen/Fertigstellen. |
|
Quix0r (administrator) 2010-03-02 17:43 |
DIe langen boolischen Ausdruecke koennten auch in Funktionen in wrapper-functions.php verpackt werden. Dann haetten wir das Problem mit dem Verschwinden der Klammer bei ueberlangen Ausdruecken nicht. Das sollte dann aber in einem seperaten Ticket gepflegt werden. |
| Notes |
| Issue History |
|||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2009-03-09 07:09 | Quix0r | New Issue | |
| 2009-03-09 07:09 | Quix0r | Sticky Issue | No => Yes |
| 2009-03-09 11:14 |
|
Note Added: 0000310 | |
| 2009-07-26 18:49 | Quix0r | Note Added: 0000349 | |
| 2009-11-25 22:56 | Quix0r | Fixed in Version | => 0.2.1-FINAL |
| 2009-11-29 03:04 | Quix0r | Relationship added | parent of 0000154 |
| 2010-03-02 17:40 | Quix0r | Fixed in Version | 0.2.1-FINAL => |
| 2010-03-02 17:43 | Quix0r | Note Added: 0000435 | |
| 2010-05-28 18:27 | Quix0r | Build | SVN-HEAD => 0.2.1-FINAL |
| 2010-05-28 18:40 | Quix0r | Product Version | => 0.2.1-FINAL |
| 2010-07-01 04:16 | Quix0r | Sticky Issue | Yes => No |
| 2012-06-17 15:10 | Quix0r | Assigned To | => Quix0r |
| 2012-06-17 15:10 | Quix0r | Status | new => assigned |
| 2012-11-28 18:31 | Quix0r | Relationship added | related to 0000133 |
| 2012-11-28 18:33 | Quix0r | Relationship added | related to 0000148 |
| 2012-11-28 19:31 | Quix0r | Relationship added | related to 0000218 |
| 2012-11-28 19:32 | Quix0r | Relationship deleted | related to 0000218 |
| Issue History |