Özellikle bazı metotların public olması anlamsız geldi bana ama tabi ki görüş meselesi
Naçizane bir iki öneride bulunabilirim belki.
* SQL SHOW sorgusu kullanmışsınız, ancak buna izin verilmeyen veritabanlarında sorun yaşar. Bir çok kişi güvenlik için show,drop,alter,create gibi kritik sorgulara izin vermez.
* 50 sütunlu bir tablomdan sadece id, ad ve soyad sütunlarını içeren bir liste yapmak istediğimde, açıkcası 47 sütunu niye çekiyorum diye düşünürüm. sayfalamaSorgusu() isimli metod ile içeride sorguyu yazmış ve bir takım parametreleri dışarıdan verdirmişsiniz. Ancak dediğim gibi bir problem var, istediğim sütunları çekebilmek isterim.
* Negatif sayfa sayısı verildiğinde sistem çatlar.
* Lütfen yorumlarımı yapıcı olarak algılayın.
* Geliştirmek isterseniz daha başka yorumlar da yapabilirim.
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.
Düzeltirim.
