Basitliği göz önüne almadan yorumlamıştım ben; cevabınız, düzenlemeleriniz ve çalışmanız için teşekkürler

erginkeles adlı üyeden alıntı: mesajı görüntüle
Ben bu sitede hiç yıkıcı yorum yapmadım. Başkalarından gelen yorum ve eleştirileri de küçük düşürücü içeriğe sahip olmadıkları ve hakaret içermedikleri sürece hep yapıcı olarak aldım. extends hatasını düzelttim ve tekrar upload ettim.

Ama burada "basit" kelimesine vurgu yapmak isterim Bu sınıf son derece basit bir sınıftır ve geliştirmeye de açıktır.

Metodların hepsinin public olmasının birinci nedeni basitlik. Geliştirilmeye daha elverişli olması. Bu sınıf basit bir sınıf. İsteyen indirir doğrudan kullanır. İkincisi, ki ben bunu tercih ederim, kendi ihtiyaçlarına göre düzenleyip kullanır. sayfala() metodu ile geri döndürdüğüm aktifSayfa, toplamVeri gibi verilere aslında öncelikle ilgili metodu çağırarak ulaşmak istedim. İki şekilde de bu bilgilere ulaşılabilmek için tüm metodları public olarak bıraktım.

Sınıf aslında doğrudan kullanılmasını istediğim bir sınıf değil. Doğrudan da kullanılabilir ama, ihtiyaca göre düzenlensin istiyorum. Gelişsin. Örneğin bir facebook.php sınıfını kafanıza göre düzenleyemezsiniz. Resmi bir sınıftır ve son haline kadar herşeyi tamamlanmıştır. Ama bu sınıf yapı olarak da yapabilecekleri olarak da son derece basit bir sınıf.

SHOW SQL konusunda da haklısınız, veritabanına user atarken sorulur genelde. Ama yine dediğim gibi, bunu indirip doğrudan kullanan arkadaşlar, bu sorgunun çalıştırılabilir olması gerektiğini göz önüne almalılar. "Sorun olacaksa kullanmayın kardeşim" gibi bir yaklaşım yerine ben "Kendinize göre düzenleyip kullanın, gelişmesine katkıda bulunun" gibi bir yaklaşım sergilemeyi daha doğru buluyorum.

Şöyle diyelim Bu paylaştığım, bu sınıfın ilk versiyonu olsun. 1.1 versiyonunda aldığım eleştirilerle metodları public yapmaktan vazgeçeyim. Ama eleştiri yapılmayacak yada yapılan eleştiriler çok azalacak aşamaya gelene kadar private a geçmeyi düşünmüyorum

Neden ihtiyacım olmayan 47 sütun da dönüyor sorunuz da gayet mantıklı. Bunu da göz önüne alacağım. ayar değişkeninde bunu da isteyebiliriz. Yaptığınız 4 eleştiri de, en baştan beri aklımda olan, benim de düşündüğüm şeyler, buna emin olabilirsiniz.

Son olarak negatif sayfa sayısı ile ilgili de bir değişiklik yaptım. aktifSayfa() metodu içinde tek satır ile bunu engelledim. Ama burada da aynı şeyleri söyleyeceğim. Ben demiyorum ki bu sınıfı kullanan GET ile aldığı değeri doğrudan sınıf ayar değişkenindeki aktifsayfa indisine atasın. Kendisi kendi yazdığı kodlar arasında en azından GETten gelen değerle ilgili birkaç mantıksal doğrulama yapmalı değil mi?

Eleştirileriniz için teşekkür ederim. Evet bu sınıf geliştirilebilir. Yapacağınız daha eleştiri varsa neden eleştiri yapıyorsunuz demeyeceğimden emin olabilirsiniz.