Neden birçok PHP Devs, isset () ve / veya PHP'nin empty () gibi benzer savunma işlevlerinden herhangi birini kullanmaktan nefret ediyor?

Stackoverflow üzerinde, bu sorunun her zaman azaldığını görüyorum:

• E_NOTICE? == E_DEBUG, daha sofistike error_handler ile isset () ve @ karakterlerinden kaçınır
• PHP, E_NOTICE açıkken $ _GET için undefind endeksini kontrol etmemeye nasıl ayarlanır?
• PHP'nin PHP günlüğe kaydetmesini engelleme
• Böyle bir PHP 5.3 Bildirimlerini nasıl kapatabilirim?

Pek çok sağlam PHP tavsiyesi sunan Pekka bile , korkunç E_NOTICEcanavara çarptı ve kullanmaktan daha iyi bir çözüm umuyor isset(): isset () ve empty () çirkin hale getiriyor

Şahsen ben kullanmak isset()ve empty()birçok yerde benim uygulamaların akışını yönetmek için. Örneğin:

public function do_something($optional_parameter = NULL) {
    if (!empty($optional_parameter)) {
        // do optional stuff with the contents of $optional_parameter
    }
    // do mandatory stuff
}  

Bunun gibi basit bir snippet bile:

if (!isset($_REQUEST['form_var'])) {
    // something's missing, do something about it.
}

bana çok mantıklı geliyor. Şişmiş gibi görünmüyor, kararlı kod gibi görünüyor. Ancak birçok geliştirici, uygulamalarını E_NOTICEetkinleştiği için ateşledi , birçok sinir bozucu "başlatılmamış dizi dizini" bildirimini keşfetti ve sonra tanımlanmış değişkenleri kontrol etme ve kodlarını "gönderme" umuduyla yüzünü buruşturdu isset().

Başka dillerin işleri farklı şekilde ele aldığını sanıyorum. Tecrübelerden bahseden, JavaScript PHP kadar kibar değildir. Tanımsız bir değişken tipik olarak betiğin çalışmasını durduracaktır. Ayrıca, ( deneyimsizlikten konuşarak ) C / C ++ gibi dillerin derlenmeyi reddedeceğinden eminim.

Peki, PHP devs sadece tembel? (Sizden bahsetmiyorum Pekka, eski bir uygulamayı yeniden düzenlediğinizi biliyorum.) Veya diğer diller tanımlanmamış değişkenleri programcının tanımlanmış olup olmadığını ilk olarak kontrol etmelerini istemekten daha zarif bir şekilde ele alıyor mu?

( E_NOTICETanımsız değişkenlerin yanı sıra başka mesajlar da olduğunu biliyorum , ancak bunlar en çok dile gelenler gibi görünüyor)

Zeyilname
Şimdiye kadar cevaplardan, isset()kod şişirme olmadığını düşünen tek kişi ben değilim . Şimdi merak ediyorum, diğer dillerdeki programcılarla bunu ekleyen sorunlar var mı? Yoksa bu sadece bir PHP kültürü sorunu mu?

Ben kodluyorum E_STRICTve başka bir şey yok.

Boş ve isset kontrolleri kullanmak kodunuzu çirkin yapmaz, kodunuzu daha ayrıntılı hale getirir. Aklımda onları kullanmaktan kaynaklanabilecek en kötü şey nedir? Birkaç karakter daha yazarım.

Bunları kullanmamanın sonuçlarını, en azından uyaranlara çevirir.

Bence bilinmeyen unsurlara ilişkin bildirimler PHP'de bir tasarım hatası. Emin şimdi hatayı düzeltmek mümkündür değilim, ama sanki Demirbaş bir sürü kod üretir if(isset($foo['abc']) && $foo['abc'] == '123')- Bu kod olmamalıdır sahip issetbelirli bir yerde '123' olup olmadığını kontrol etmektir niyet beri, $foove şey yoksa orada olduğunu Kesinlikle '123' değil. İki kat fazla kod yazmanızın tek nedeni PHP'deki bu talihsiz tasarım hatası. Ve PHP'de bildirimler çok pahalı, ne yazık ki, bu yüzden yalnızca bunları devre dışı bırakmak, performansın endişe verici olduğu bir kod seçeneği değildir.

Yani evet, IMHO kodunu çirkin yapıyor ve beni rahatsız ediyor. Ve deneyim eksikliği yüzünden değil - 1998'den beri PHP kullanıyorum ve .php3uzantı olduğunu hatırlıyorum ve "PHP 2 değil" anlamına geliyordu. Belki tembelim :) Ama tembellik - en azından belirli bir tür - bir programcı için bir erdemdir.

Öte yandan, geçerli yazı kullanımı issetve empty- tıpkı yazıdakiler gibi - iyi. PHP'nin isset/emptygerçekten ihtiyaç duyulmadığı yerlerdeki uyarılar konusunda aşırı kıskanç olduğunu düşünüyorum .

PHP'nin web'de yorumlanan ve kullanılan ücretsiz bir dil olarak, neden savunmacı bir şekilde kodlamaları gerektiğinin tam olarak farkında olmayan ve uyarıları başka bir gereksiz hata olarak görmeleri gerektiğinin farkında olmayan, çok yüksek oranda profesyonel olmayan, kodlanmamış bir kodlayıcı olduğuna inanıyorum. .

Bu puanları her zaman küçük geliştiricilerin ve kendi kendini öğreten senaryo kodlayıcılardan duyuyorum:

• Zaten ortaya çıkacaksa neden bir değişkeni başlatmalı?
• Neden kullanmadan önce bir şey olup olmadığını kontrol et, undefined yine de yanlışsa eşittir?
• @ İle önek eklersem kodumu düzeltir, neden işleri karmaşıklaştırır?

Hiç bir zaman kesin olarak yazılmış bir dil yaşamamışlarsa veya açıklanmamış / yetersiz değişkenlerin tuzaklarını deneyimlemişlerse, ikna edici olurlar. Ben genellikle bir saat boyunca kod hata ayıklama zevkini yaşamaya başladıktan sonra genellikle verdiklerini buluyorum.

Diğer güçlü faktör PHP'nin güvenlik ve kod kalitesinden çok verim ile daha fazla ilgilenme eğiliminde olan web endüstrisindeki kullanımıdır.

Evet, tembeller. Yine de çoğu ...

Ne yazık ki, pek çok PHP kodlayıcısının zihniyeti "Eksik değişkenlerin neden olduğu hataları vb. İşlemek için dile dayanarak aynı sonuca daha çabuk ulaşabiliyorsanız, savunmanın bir anlamı yoktur." Biliyorum, onlardan birkaçıyla çalıştım.

Ayrıca, doğru hata yönetimi ve raporlama eksikliği, canlı sunucuları birkaç saatliğine çıkardıklarında, erken bir öğle yemeğine gizemli bir şekilde yönelenlerdir.

Dizileri veri aktarma nesneleri olarak kullanmaktan kaçınarak isset () ve empty () yöntemlerinden kaçınmaya çalışıyorum. Aklı başında varsayılanlara sahip sınırlı bir özellik kümesini kabul edecek ve bu özelliklerin girişini doğrulayacak şekilde yapılandırılabilecek bir sınıf oluşturun. ArrayAccess arayüzünü bile uygulayabilir, böylece onu bir dizi gibi kullanabilirsiniz. Bu aynı zamanda, birileri yönteminize yanlış bir varlık türü iletmeye çalıştığında hataları yakalamak için yöntem imzalarınızda tür ipucu kullanmanıza izin verir.

Sorgulanan sorulardan biri benim, bu yüzden tekrar etmeme izin ver.

Birçok geliştirici isset(), kalite işareti olarak çoğunun varlığını yanlış yapar . Bir güvenilirlik görünümü verir ve bazı insanlar güvenlik özelliği olarak bile düşünür.

Fakat PHP'nin derlenmiş bir dil olmadığını dikkate almalısınız. Dinamik tip sistemli bir betik dilidir. E_NOTICE hataları, yalnızca ada göre hatalardır. Ve eğer issetbirçoğu sadece bildirimleri bastırma niyetiyle kullanılıyorsa , o zaman aslında sadece dile karşı kodlama yapıyorsunuzdur .

Peki, PHP devs sadece tembel?

Gerçekten de, bildirimler ve atılan çok sayıda uyarı görürseniz durum budur. Çok sayıda yenidoğan ihbarı önemsemiyor ve bu genellikle tamamen özürlü olmanın sonucudur error_reporting(0).

Bununla birlikte, diğer geliştiricilerin, yalnızca @ ya da baskı altında bırakılmadıkları için E_NOTICE'ları tanımadıkları için yanılıyorsunuz. En azından bu tutulan bildirimler sorusunun arkasındaki niyetimdi . Kurtulacak bir şey değil, zaman zaman önemli hata ayıklama bilgileri.

Tüm genellemeler gibi, isset'in düşüncesiz kullanımı da optimal kodlamaya yol açmaz. Nerede issetve emptygerekli ve nerede sözdizimsel tuz olduklarını ayırt etmek önemlidir .

Yoksa bu sadece bir PHP kültürü sorunu mu?

Hayır, tanımsız değişken "hata" sadece bir PHP problemi değildir. Bash, TCL ve Perl veya JavaScript tanımsız değişkenlerin kullanımına izin verir. Ancak bu içkin dil özelliği orada kusur olarak görülmez. Tanımlanmamış değerleri kontrol etmek için benzer dil yapıları vardır. Ancak, undef değerlerinin "hata" olarak yanlış tanımlanmadığı için PHP'deki kadar sürekli kullanılmazlar.

İlginç. Ben neredeyse hiç isset () kullanmıyorum. PHP kodum nadiren, ilk etapta bir değişken ayarlanmış olup olmadığını bilmediğim bir durumda. Kullanılan her GET veya POST değişkenine, mevcut değilse, varsayılanı sağlayacak bir işlevle erişilir. İşlev çağrılarındaki varsayılan değerler genellikle açıkça 0 veya boş dizeler olarak ayarlanır.

Ben setset ve bol boş kullanırım. Ama çoğunlukla, birinin parametreleriyle uğraşması durumunda $ _REQUEST işleme gibi bahsettiğiniz yerler. Etrafta uçuşan tüm değişkenler üzerinde kontrolüm olduğu durumlarda, genellikle onlara ihtiyaç duymadığımı anlıyorum.

İsset kullanmaktan hoşlanmıyorum, ancak başkası tarafından yapılan kod kütleleri varsa, o zaman tasarruf lütuf olabilir. Bu soruna yardımcı olmak için aşağıdaki wee kodunu yazdım, eğer isset () isseter kullanmak yerine ($ a, $ b), $ a tanımlanmadığında veya boşsa ya da işlev null değerini döndürdüğünde $ b döndürür. Herhangi bir gelişme, hoş geldiniz:

//returns var or NULL (if undefined)
//$reserve optional second value is returned as $default if undefined
function isseter(&$default,&$reserve=NULL)
{
$default = isset($default) ? $default : NULL;
$reserve = isset($reserve) ? $reserve : NULL;
if ((!$default) && ($reserve)) $default=$reserve;
return $default;
}