Anzeige - [Hier werben / Mediadaten]
(?) Tags raten (?) php (edit)
 
Reply to this topicStart new topicStart Poll
> Richtige If-Kurzform?, PHP if-statement
Thema abonnieren | Thema versenden | Thema drucken
nitrit
Geschrieben am: Sa 1.09.2007, 14:18
Report PostQuote Post

AyomRank 5
**********

Gruppe: Member (aktiv)
Beiträge: 259
Mitglied seit: 8.08.2007
Ich wollte nur kurz wissen, ob die kurzform so richtig funktioniert, zumindest erhalte ich keinen PHP-Fehler.
Im Normalfall benutze ich nämlich die lange Form, möchte aber hier ein wenig Code sparen:

CODE
function dir_exists(){
           if(file_exists($this->dir))
               return true;
               else return false;
       }


--------------------
http://innovative-webmedia.de
Top
PMEmail Poster
Top
 
 
Daniel Cieslar
novinet Websolutions
#2 Geschrieben am: Sa 1.09.2007, 14:22 (+00:03)
Report PostQuote Post

AyomRank 6
************

Gruppe: Member (aktiv)
Beiträge: 759
Mitglied seit: 3.10.2005
Hallo,

diese Form ist absolut richtig, wenn auch unüblich und meiner Meinung nach aus Lesbarkeitsgründen unpraktikabel. Wenn du erstmal 5 ineinandergeschachtelte if-Abfragen hast, wirst dich nach den Klammern sehnen :-)

Gruß, D.


--------------------
...bald verheiratet und mit neuem Namen: ab 06.12.2008 Daniel Steffen

www.calcolo.de - Calcolo 3.0 mit PDF-Rechnungen, Angebote, Mahnungen,...

www.webspiration.de - interaktive Werbemittel: Pagepeel, Curtain, Rotator, Carousel

novinet Software, Design, Consulting - wir erstellen Ihren maßgeschneiderten Internetauftritt
Top
PMEmail PosterUsers WebsiteICQ
Top
 
nitrit
#3 Geschrieben am: Sa 1.09.2007, 14:41 (+00:19)
Report PostQuote Post

AyomRank 5
**********

Gruppe: Member (aktiv)
Beiträge: 259
Mitglied seit: 8.08.2007
Das Stimmt, allerdings bei solchen Funktionen wie oben beschrieben, ist es übersichtlicher als -{ }- wink.gif
Bei größeren Funktionen hast du natürlich recht smile.gif

Danke für die Hilfe übrigens wink.gif


--------------------
http://innovative-webmedia.de
Top
PMEmail Poster
Top
 
Daniel Cieslar
novinet Websolutions
#4 Geschrieben am: Sa 1.09.2007, 14:50 (+00:09)
Report PostQuote Post

AyomRank 6
************

Gruppe: Member (aktiv)
Beiträge: 759
Mitglied seit: 3.10.2005
Nix zu danken, tun wir ja gern... 3,50 EUR kannst auch auf mein Konto überweisen *lach*

Gruß, D.


--------------------
...bald verheiratet und mit neuem Namen: ab 06.12.2008 Daniel Steffen

www.calcolo.de - Calcolo 3.0 mit PDF-Rechnungen, Angebote, Mahnungen,...

www.webspiration.de - interaktive Werbemittel: Pagepeel, Curtain, Rotator, Carousel

novinet Software, Design, Consulting - wir erstellen Ihren maßgeschneiderten Internetauftritt
Top
PMEmail PosterUsers WebsiteICQ
Top
 
Marcel Kruse
Online IS
#5 Geschrieben am: Mo 3.09.2007, 13:56 (+47:05)
Report PostQuote Post

AyomRank 4
********

Gruppe: Member (aktiv)
Beiträge: 66
Mitglied seit: 15.02.2007
QUOTE (nitrit @ Sa 1.09.2007, 15:18)
Ich wollte nur kurz wissen, ob die kurzform so richtig funktioniert, zumindest erhalte ich keinen PHP-Fehler.
Im Normalfall benutze ich nämlich die lange Form, möchte aber hier ein wenig Code sparen:

CODE
function dir_exists(){
           if(file_exists($this->dir))
               return true;
               else return false;
       }

Noch kürzer:

CODE

if(file_exists($this->dir)) ? return true : return false;


--------------------
Preissuchmaschine und Preisvergleich
Top
PMEmail PosterUsers WebsiteICQMSN
Top
 
manuel
#6 Geschrieben am: Mo 3.09.2007, 14:35 (+00:39)
Report PostQuote Post

AyomRank 5
**********

Gruppe: Member (aktiv)
Beiträge: 261
Mitglied seit: 14.01.2004
QUOTE (Marcel Kruse @ Mo 3.09.2007, 13:56)
CODE

if(file_exists($this->dir)) ? return true : return false;


funktioniert nicht

richtiger wäre:
CODE

return (file_exists($this->dir)) ? true : false;


wobei ich die Schreiweise auch noch einigermaßen gut finde.

Eine If-Bedingung ohne Geschweifte-Klammern ist aber nie zu empfehlen.
Die Lesbarkeit wird durch Klammern extrem erhöht, IMMER smile.gif

Das Problem ohne Klammern ist, dass beim lesen des Codes nicht auf den ersten Blick klar ist was gemacht wird.

Schreib es also entweder richtig mit Klammern oder wenn du es ganz kurz brauchst so wie oben mit ? und : . Sonst machst du dir und allen die später den Code lesen müssen, das Leben unnötig schwer.
Top
PM
Top
 
Roger Bobst
calac gmbh
#7 Geschrieben am: Mo 3.09.2007, 15:58 (+01:22)
Report PostQuote Post

AyomRank 5
Group Icon

Gruppe: Experten Entwicklung (Mod)
Beiträge: 426
Mitglied seit: 15.02.2004
das if ist überflüssig, da file_exists bereits einen boolean liefert.

CODE
return file_exists($this->dir);


Gruss
Top
PMEmail PosterUsers Website
Top
 
Thema wird von 0 Benutzer(n) gelesen (0 Gäste und 0 anonyme Benutzer)
0 Mitglieder:
« Tabelle mit bestimmter Sortierung aufrufen | Programmierung Allgemein | Preisvergleich - Programmierung »
Trackback-Url: http://www.ayom.com/track/t/20401
Dieses Thema abonnieren

Topic Options Reply to this topicStart new topicStart Poll

 


> Ähnliche Themen
Themen Titel Autor Views Antworten Letzte Aktion
automatisch immer das richtige Wetter steini 97 1 Fr 10.10.2008, 10:12
Was ist der richtige Job? rocoloco 846 19 So 21.10.2007, 21:46
Was ist der richtige Inet-Anschluss? Jan-Remmer Harms 237 8 Mo 16.07.2007, 16:28
Richtige Schreibweise? japsa 271 3 Do 17.05.2007, 10:45
11 richtige Projekte zu verkaufen Andreas Schroth 561 0 Sa 13.01.2007, 16:44
Suche richtige Keywords...nach realen Abfragen kekskruemel 590 2 So 18.12.2005, 16:54
Richtige Werbung? weissNix 736 0 Do 12.05.2005, 19:43
richtige keywords? ypsie 645 6 Do 21.10.2004, 13:56
Richtige GOOGLE-Optimierung?! ViperNeo 655 6 Mi 13.10.2004, 12:36




Anzeige - [Hier werben / Mediadaten]



Anzeigen


[Hier werben / Mediadaten]