SpyTools v3.0 (istediğiniz özelliği söyleyin)

Status
Not open for further replies.
Önceki konuna attığım yorumdaki önerilere dikkat edersen bence çok daha iyi olur.

Kanka biraz hatalarından bahsedeceğim. En büyük hatan C# ile tool yapman. C# %100 Nesne tabanlı bir dildir.
1 - UI Bir tasarım varsa aklında WPF ile yapmalısın.
2 - Formların UI amaçlı yapıldığı için Form nesnesi tanımlandığında RAM (Heap) bölgesinde büyük bir bölüm ayırıyor. Sen Formla işin bittiğinde hiç birini Dispose etmemişsin. Bu yüzden her Form açıldığında uygulamanın kullandığı RAM miktarı artıyor. "this.Close();" kullanman Form'un RAM üzerindeki kullandığı adresleri temizlemez. 2. Form açtıktan sonra Form'u kapatma tuşu ayarladığın yerden kapanınca ana Form'undan bir adet daha açılıyor ve uygulama RAM sömürür hale geliyor.
3 - 200'den fazla nesne tanımlamışsın ve her kullanımda sürekli aynı nesnelerden bir tane daha oluşturuyor. Tanımladığın nesnelerden bir kaçı;
pbQp4a.png


Her işlemde nesne tanımladığın metodlar kullandığın için metodların işlemleri bittikten sonra nesneler temizlenmediği için sürekli olarka RAM kullanımı artıyor.

4 - C#'da WebBrowser kullanımı tamamen gereksizdir. Yüksek derecede RAM tüketir ve asla RAM'dan Dispose edilemez. Manuel yapsanda RAM'dan tam çıkarılamaz. O yüzden WebBrowser kullanımını ya azaltmalısın yada tamamen kaldırmalısın.

5 - Aşırı yükleme fonksiyonları kullanmışsın bunlara gerek yok boşuna performans düşürmene gerek yok.
r5bodm.png

Yukarıda using kullanmışsın. using genellikle metod içerisinde tek sefer kullanılıp otomatik işlem bittikten sonra Dispose edilmesi için kullanılır. System.IO.File kütüphanesi nesne oluşturmaz ama System.IO oluşturur. iTalk_Button_13_Click metodu sonlandığında ise otomatik Dispose olur.

6 - Program videonda şu bölümde yaptığın iş tamamen yanlış :) İnsanları kandırmaya gerek yok.
View attachment 1286
Videoda o dakikada Thread kullandığın için bekletmişsin :/
View attachment 1287

Uygulamanı geliştireceğin için bunlara dikkat etmen gerekiyor o yüzden yazdım. Yoksa daha da kötüleşir durum. Son önerim olarak Class kullanmanı öneririm kod karmaşası fazla var. Framework sürümünü güncellemelisin. @MawionZ

🔒 Bu içeriği görmek için giriş yapın

 
Status
Not open for further replies.
136,744Threads
3,308,026Messages
328,583Members
nxthLatest member
Top Bottom