🚀 go-pugleaf

RetroBBS NetNews Server

Inspired by RockSolid Light RIP Retro Guy

Article View: pl.comp.lang.php
Article #219927

Re: !isset - zbędna negacja?

#219927
From: ant
Date: Fri, 31 Jul 2009 12:22
50 lines
2107 bytes
hazet wrote:
> Jest taki kod:
> 
> if(!isset($_GET['styl']))
>      {
>          require('./normalny.php');
>      }
>      else
>      {
>          require('./opcjonalny.php');
>      }
> Jeśli zmienna styl "nie" istnieje require normalny w innym wypadku
> opcjonalny.
> 
> Po co ta negacja ?
> Przecież równie dobrze można:
> 
> if(isset($_GET['styl']))
>      {
>         require('./opcjonalny.php');
>      }
>      else
>      {
>          require('./normalny.php');
>      }
> Jeśli zmienna styl istnieje require opcjonalny w innym wypadku normalny.
> Trochę mnie to zastanawia, widziałem to w wielu kodach i ta negacja
> wydaje się być bez sensu.

JA to widze tak. Negacja
> if(!isset($_GET['styl']))

jest jako pierwsza, bo zapewne w 99.9% przypadków
zmienna $_GET['styl'] nie będzie istniała
i programista chciał, aby ta sytuacja została
obsłużona jako pierwsza. Gdyby miał to zrobić
bez negacji to by było na oko tak:

if(isset($_GET['styl']))
{
    require('./opcjonalny.php');
}
else
{
    require('./normalny.php');
}

Co już wygląda głupio, bo "opcjonalne pierdoły"
są w kodzie jako pierwsze. Generalnie
powodów takiego postępowania są miliony.
Technical Details
Message-ID: <h4unp8$pa6$1@atlantis.news.neostrada.pl>
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.gazeta.pl!newsfeed.pionier.net.pl!newsfeed.neostrada.pl!atlantis.news.neostrada.pl!news.neostrada.pl!not-for-mail
References: <h4s5jk$1rf$1@inews.gazeta.pl>

Show Full Headers
Complete Article Headers
Path: mbox-mo1.pugleaf.net!news-archive.icm.edu.pl!news.gazeta.pl!newsfeed.pionier.net.pl!newsfeed.neostrada.pl!atlantis.news.neostrada.pl!news.neostrada.pl!not-for-mail
From: ant <ant@nowhere.tld>
Newsgroups: pl.comp.lang.php
Subject: Re: !isset - zbędna negacja?
Date: Fri, 31 Jul 2009 14:22:23 +0200
Organization: TP - http://www.tp.pl/
Lines: 51
Message-ID: <h4unp8$pa6$1@atlantis.news.neostrada.pl>
References: <h4s5jk$1rf$1@inews.gazeta.pl>
NNTP-Posting-Host: 79.189.151.28
Mime-Version: 1.0
X-Trace: atlantis.news.neostrada.pl 1249043048 25926 79.189.151.28 (31 Jul 2009 12:24:08 GMT)
X-Complaints-To: usenet@news.neostrada.pl
NNTP-Posting-Date: Fri, 31 Jul 2009 12:24:08 +0000 (UTC)
User-Agent: Thunderbird 2.0.0.17 (X11/20080927)
In-Reply-To: <h4s5jk$1rf$1@inews.gazeta.pl>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit