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

Joined
Aug 5, 2018
Credits
0
Rating - 0%
Ö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
 

ZerOne 

EVIL
Joined
Jun 7, 2018
Credits
285
Rating - 0%
Önceki konuna attığım yorumdaki önerilere dikkat edersen bence çok daha iyi olur.
Önerilerin için teşekkürler iyi demişsin fakat sonda md5 hash için çok uğraştım hep hatalar aldığımdan siktir edip sadece görsel koydum
 

Users who are viewing this thread

Top