Metin2 Code Refactor

[0x23]

HERAKLES Otomatik Avlı kalıcı sunucu. 19 Haziran'da açılıyor. Atius & Wizard güvencesiyle hemen kayıt ol, ön kayıt ödülleri aktif. HEMEN TIKLA!

Source dosyalarını bir repo açıp gelen isteklere göre refactor etmeyi düşünüyorum(örneğin char.cpp'deki x fonksiyonu gibi, x classı, x eklentisi vb.), source çok eski olduğundan code smellslerle dolu. Contributor olmak isteyen varsa yorum yazabilir, yeterli katılım olursa bu işi beraber yapabiliriz dediğim gibi tek başıma yapacağım bir iş değil.

Kısaca kendimden bahsedeyim, mid level mühendisim(bilgisayar müh. çıkışlıyım), 4 yıla yakındır savunma sanayiide çalışıyorum ek olarak yüksek lisans yapıyorum. Burdaki herkes gibi çocukluğumda bu oyunu oynadım, bu aralar çok yoğun olmadığım için bakınırken bu forum aklıma geldi, sadece boş zamanlarımda ilgileneceğim gönüllü bir uğraş benim için.

Tabii ki üretimde C/C++ kullanıyoruz ve bu profosyonel düzeyde. Metin2 geliştiricileri tarafından yazılan kodları bu repoya kabul etmeyeceğimi belirtmek isterim, forumda yeterince yerli/yabancı kodlarını inceledim ve %90ı intern bir C++ geliştiricinin bile yazacağı kodlar değil. Kısacası bu dili udemyden veya ezbere öğrenmeyip kural, logic, yazılım mimarileri gibi konulara hakimseniz yorumda belirtebilirsiniz.

 

[[C]Martin]

Güzel bir düşünce kodu yenileyecekseniz tüm server yapısını kodlarını komple değişmesi gerekiyor sadece belli bölümleri değiştirmek uzun vadede iyi değil en temel gereksinim yeni nesil fonksiyonel bir network kütüphanesi yazılması şart metin2 için 2. gereksinim karakter senkronizasyonlarının baştan ele alınması ve eşitlenmesi 3. gereksinim çoklu çekirdek yapısı ve modüler mimari yapısına geçilmesi gerekiyor bir sürü madde var yapılması gereken uzun uğraş isteyen savunma sanayi işlerinde çalıştıysanız buraya ayıracağınız zamanı daha gelişmiş silah ve savunma sistemleri üzerine yazılım geliştirmek sizin için daha avantajlı olur bu eski kodu modern yapıya yükseltmek herşeyi baştan yazmak gibi birşey kodları okuduysanız tüm cpp yapısını o zamanın şartlarıyla iyi olabilir ama günümüz şartlarıyla ne kadar mantıksız ve kusurlu olduğunu farketmişsinizdir.

 

[0x23]

Güzel bir düşünce kodu yenileyeceksek tüm server yapısını kodlarını komple değişmesi gerekiyor sadece belli bölümleri değiştirmek uzun vadede iyi değil en temel gereksinim yeni nesil fonksiyonel bir network kütüphanesi yazılması şart metin2 için 2. gereksinim karakter senkronizasyonlarının baştan ele alınması ve eşitlenmesi 3. gereksinim çoklu çekirdek yapısı ve modüler mimari yapısına geçilmesi gerekiyor bir sürü madde var yapılması gereken uzun uğraş isteyen savunma sanayi işlerinde çalıştıysanız buraya ayıracağınız zamanı daha gelişmiş silah ve savunma sistemleri üzerine yazılım geliştirmek sizin daha avantajlı olur bu eski kodu modern yapıya yükseltmek herşeyi baştan yazmak gibi birşey kodları okuduysanız tüm cpp yapısını o zamanın şartlarıyla iyi olabilir ama günümüz şartlarıyla ne kadar mantıksız ve kusurlu olduğunu farketmişsinizdir.

Geniş açıdan bakacak olursak evet mimarisi zaten çok kötü singleton pattern four gangin bile sorguladığı bir pattern ve CTRP'den dolayı olumsuz gözle bakılıyor, halen çalışmaktayım öyle uğraş çıksın diye düşünmüştüm zaten kısa vadeli bir şey değil ve gönüllü bir iş olduğu için x gün x ayda bitirip geçicem diye düşünmedim. Evet network, sql kesinlikle sıfırdan geliştirilmeli, character class(mob ve karakterler aynı classtan türetilmiş vb.) gibi çok fazla durum var(sourceyi detaylı incelemedim o kadar vaktim olmadı ama 2-3 saat göz gezdirmede bile yüzlerce "bu nedir ya?" dediğim oldu). Aslında demek istediğim buydu zaten character classını refactor edelim desen çok fazla düzeltmen gereken yer olucak, bazen bazı sistemleri sıfırdan geliştirmek refactor etmekten daha kolay oluyor haklısınız. Minimalist bir şeyler yapar, plan kafamızda oturmazsa başlamayız son çare sağlık olsun

 

[[C]Martin]

Geniş açıdan bakacak olursak evet mimarisi zaten çok kötü singleton pattern four gangin bile sorguladığı bir pattern ve CTRP'den dolayı olumsuz gözle bakılıyor, halen çalışmaktayım öyle uğraş çıksın diye düşünmüştüm zaten kısa vadeli bir şey değil ve gönüllü bir iş olduğu için x gün x ayda bitirip geçicem diye düşünmedim. Evet network, sql kesinlikle sıfırdan geliştirilmeli, character class(mob ve karakterler aynı classtan türetilmiş vb.) gibi çok fazla durum var(sourceyi detaylı incelemedim o kadar vaktim olmadı ama 2-3 saat göz gezdirmede bile yüzlerce "bu nedir ya?" dediğim oldu). Aslında demek istediğim buydu zaten character classını refactor edelim desen çok fazla düzeltmen gereken yer olucak, bazen bazı sistemleri sıfırdan geliştirmek refactor etmekten daha kolay oluyor haklısınız. Minimalist bir şeyler yapar, plan kafamızda oturmazsa başlamayız son çare sağlık olsun

Evet çok iyi noktaları görmüşsün game ve db arasındaki gereksiz paket yükünü bir görsen bunu hangi zihinle yazdılar diye düşünürsün. Boş zamanlarında uğraşabilirsin ama ileride bir gün bu altyapıyı kullanmayacaksan zaman kaybı demek oluyor keşke başka birşey üzerine uğraşsaydım dememen için söylüyorum yazılım topluluğu olarak zaman çok değerli. Biraz kodları incele bak network den sonra en büyük sıkıntı item sınıfı sonra character sınıfı geliyor bunların db iletişimine bak ve paketleri loglayıp bir test et trafiği normal oyuncu olarak bunu yüksek bir online ile çarp burada bir nokta var ne kadar karakterler yan yana olursa biri vurduğu anda yakındaki tüm karakterlere bir sürü paket gidiyor ve bu durum birbirini gören oyuncu sayısı ne kadar artarsa paket yükü o kadar artıyor. Benim en nefret ettiğim nokta bir şeyi güncellemek istiyorsun baştan yazmak kodunu ama tüm yapıyı ele alman gerekiyor herşey birbirine bağlı olduğu için ağır işlemlere giremiyorsun kodun testi ayrı bir durum yüksek oyuncuda deneme riski ayrı bir durum çok fazla sınırlamayla karşılaşılıyor.

 

[MT2Dev]

Onlarla dolu değil de kaynak kodunun tamamı koca bir code smells örneği desek çok daha isabetli olabilir. Temeli 2002-2004 yıllarına dayanıyor ve döneminde çalışan senior devleri dahil ekiplerinin çok büyük bir çoğunluğunun C dilinde uzman devler olduğunu düşünüyorum, fonksiyon yapıları, oldukça ezber gözüken kodlama stilleri bana böyle düşündürdü. Pek çoğunun C++ ile herhangi bir bağlantısı olmadığını ve hatta bu projeyi geliştirirken bazı şeyleri öğrendiklerini düşünüyorum, bence bu source 2004 dönemi için bile oldukça özensiz, plansız ve kesinlikle profesyonellikten uzak, aynı dönemin oyunu olan hatta hala dünyanın en büyük MMORPG oyunu olan WoW en büyük örnek, aradaki kaynak farkı, profesyonellik bambaşka düzeyde.

Bahsettiğiniz projenin üstteki arkadaşın da dediği gibi bölüm bölüm yapılabilmesi bence de çok güç, o kadar birbirine bağlı bir kaynak ki örneğin eğer solid prensipleriyle refactor etmek istiyorum derseniz muhtemelen en basitinden yüzlerce yeni class yazmanız gerekir, çok uzun bir iş ve uğraşa değeceğinden de şüpheliyim. Bu kadar detaylı olmasa da bunu belirli bir düzeyde yapan projeler de var örneğin N2Play dosyalarını incelemenizi öneririm, tam bir refactor sayılmayabilir ama güzel iştir.