Noroc adlı üyeden alıntı: mesajı görüntüle
Eline saglik. Bir kac onerim olacak.
  • Dosya adini shipy.php yerine Shipy.php olarak kullanman
  • Composer paketi haline getirmen daha iyi olacaktir. Kurulumu daha kolay olur. Bu sekilde namespace de ekleyebilirsin.
  • Bazi if statement'lar icin suslu parantez kullanilmamis, kullanilmasi kodun okunabilirligini arttirir.
  • Opsiyonel olarak methodlara return type eklenebilir.
Önerileriniz için teşekkürler. Yaptığım sınıflarda nedendir bilinmez, namespace kullanmıyorum. Composer paketi hâline getirebilirim, zaten 2 adet sınıfım Composer paketi hâlindeydi anacak buna eklemeyi istemedim başta. Çünkü amacım sınıfı sadece Shipy'e gönderip belki siteleri üzerinde farklı bir PHP entegrasyon alternatifi olarak sunmalarını önermekti. Sınıfın satır sayısını az tutmaya çalıştım. Gereksiz süslü parantezler sanki biraz daha karıştırıyor kodu. Zaten tek satırlık şeyler olduğundan kullanmadım süslü parantezleri. Yine de düzenlemeyi düşüneceğim.

Düzenleme: Composer paketi hâline getirdim.