- Mesaj
- 172
- Çözümler
- 11
- Beğeni
- 264
- Puan
- 804
- Ticaret Puanı
- 0
Kaynak kodlarından gözüme çarpan (veya okurken kendi kendime söylenip hatta yanına foruma yazamayacağım tarzda yorum satırları ekleyip sonra sildiğim ) saçmalıklar;
Özellikle cmd dosyalarında inanılmaz amatör bir veri kullanım şekli var, adam MINMAX ile sadece 0 ve 3 arasında seçilebilecek bir str_to_number fonksiyonu yazmış mesela ama countu int ile tanımlamış ? Ya arkadaş, bu kadar kısa bir aralığı kapsayan şeyi int ile tutmanın manası nedir ? Bu tek başına bir tüketime yol açmaz belki ama src komple bunlarla dolu, hepsini alt alta koyduğun da illaki bellek tüketimine olumsuz etki eder.
Cmd_general içindeki shutdown komutunun fonksiyonunda, CH'nin NULL olup olmadığını kontrol ediyor eğer NULL ise sys_err yazdırıyor ama sys_err içinde ch->GetName kullanmış ? CH nullsa adını nasıl alacaksın sevgili Koreli abi, kimse bunu görüp ya biz napıyoruz demedi mi ?
new xxx ile yeni tanımlama yaptığınız fonksiyonların sonunda delete ile bunu silmek çok mu zordu ? Şuan smart pointerlar bu işi halleder ama siz bunlar olmadan (o yıllarda yoktu) fonksiyonu yazmışsınız kimse de iş bittikten sonra biz bunu silmiyoruz acaba memory leak yaratır mıyız dememiş.
Örnek olarak bonus;
Özellikle cmd dosyalarında inanılmaz amatör bir veri kullanım şekli var, adam MINMAX ile sadece 0 ve 3 arasında seçilebilecek bir str_to_number fonksiyonu yazmış mesela ama countu int ile tanımlamış ? Ya arkadaş, bu kadar kısa bir aralığı kapsayan şeyi int ile tutmanın manası nedir ? Bu tek başına bir tüketime yol açmaz belki ama src komple bunlarla dolu, hepsini alt alta koyduğun da illaki bellek tüketimine olumsuz etki eder.
Cmd_general içindeki shutdown komutunun fonksiyonunda, CH'nin NULL olup olmadığını kontrol ediyor eğer NULL ise sys_err yazdırıyor ama sys_err içinde ch->GetName kullanmış ? CH nullsa adını nasıl alacaksın sevgili Koreli abi, kimse bunu görüp ya biz napıyoruz demedi mi ?
new xxx ile yeni tanımlama yaptığınız fonksiyonların sonunda delete ile bunu silmek çok mu zordu ? Şuan smart pointerlar bu işi halleder ama siz bunlar olmadan (o yıllarda yoktu) fonksiyonu yazmışsınız kimse de iş bittikten sonra biz bunu silmiyoruz acaba memory leak yaratır mıyız dememiş.
Örnek olarak bonus;
C++:
int iSocketCount = 0; // neden int ?????
str_to_number (iSocketCount, arg2);
if (!iSocketCount || iSocketCount >= ITEM_SOCKET_MAX_NUM)
{
iSocketCount = 3; // bir üst satırda ITEM_SOCKET_MAX_NUM kullanmışsın burda niye sabit kodladın hasta mısınız siz kafayı yememe az kaldı
}