Gönderilen uygulama incelendiğinde, belirtilen gerekliliklerin bir kısmının karşılandığı görülmüştür.

Transkript

1 İsim : İlker **** Soyad : K****** Değerlendirilme tarihi : Karşılıklı görüşme tarihi : Alanı : Backend Java Değerlendirme yorumu: Gönderilen uygulama incelendiğinde, belirtilen gerekliliklerin bir kısmının karşılandığı görülmüştür. Öncelikle, uygulamanın kolaylıkla ayağa kalkmadığı, geliştirme ortamına bağımlı bir konfigürasyona sahip olduğu, bu sebeple uygulama zahmetsizce ayağa kalkmalı teknik isteğinin karşılanmadığı ifade edilmelidir. Geliştiricinin istenen uygulamanın inşası için, seam framework kullanmaya çalışması tarafımızca olumlu karşılanmış, ancak yapılan online görüşmede bu framework e çok hakim olmadığı, ayrıca application server ile birlikte bu tercihin, sistem aynı anda minimum kişiye hizmet verebilecek şekilde tasarlanmalı isteğinin yanlış değerlendirilmesiyle yapıldığı anlaşılmıştır. Geliştirici, söz konusu isteği salt jboss application server kullanarak karşılayabileceğini düşünmüştür. Hibernate gibi bir ORM( object relational mapping ) aracının kullanılması tarafımızca olumlu karşılanmakla birlikte, yukarıdaki istekle paralel, söz konusu aracın sağlayacağı connection pooling vb avantajlardan yararlanılmamış olması, bir eksiklik olarak kayda geçirilmiştir. Uygulamada sisteme sadece kayıtlı kullanıcıların girebilmesi isteği karşılanmış, ancak sisteme kullanıcı kayıt etmek için bir adım atılmadığı görülmüştür. Yine uygulamada validasyon işleminin yer yer ihmal edildiği görülmüş, bu ihmalin yapılmadığı noktalarda, kullanıcıyı bilgilendirmek için gösterilen mesajların user friendly olmadığı tespit edilmiştir. Sisteme giriş anında girilen password ün hash lenmesi ise tarafımızca olumlu bir nokta olarak kayda geçirilmiştir. Önyüzde, uygulamada kullanılan bileşen kütüphanesinin( richfaces ) salt panel ve tab componentlerinin kullanıldığı, geliştiricinin CSS kodu yazmadığı, ya da bir CSS framework ünden yararlanmadığı tespit edilmiştir. Bu açıdan geliştirici, UI/UX noktasında değerlendirme yapabilecek yeterlilikte bir veri ortaya koymamıştır. Yapılan online görüşmede bunun, responsive dizayn isteğinin doğru anlaşılmamasından kaynaklandığı tespit edilmiştir.

2 Uygulama MySQL database kullandığı halde, kullanılan schema için bir sql script in projeye eklenmemiş olması( proje içinde boş, sql uzantılı import adında bir dosya mevcut ) da bir diğer eksiklik olarak kayda geçirilmiştir. Buna mukabil, geliştiricinin uygulamanın konfigürasyonu ve nasıl ayağa kaldırılacağına dair bir metin dosyası hazırlaması, olumlu bir nokta olarak değerlendirilmiştir. Geliştiricinin uygulama genelinde exception handling noktasında attığı adımlar (try catch kullanımı) olumlu karşılanmış, ancak yakalanan hataların kayıt edilmemesi, yine önyüzde kullanıcıyı bu hatalara dair bilgilendirme ihtiyacı hissedilmemesi, eksiklik olarak görülmüştür. Uygulamada dilsel esneklik ve I18N için bir message bundle kullanılması, olumlu bir nokta olarak kayda geçirilmiştir. Loglama yapılmaması ve unit test yazılmaması da, diğer eksiklikler olarak kayda geçmiştir. Geliştiricinin kodun yönetilebilirliği için gerekli olan kod dokümantasyonu konusunda yeterli olduğu görülmüştür. Yapılan incelemede kod içi yorum satır oranının % 5 olduğu tespit edilmiştir. Yine açık kaynak kodlu Apache projeleri baz alındığında yorum satır oranı en düşük %3.6, ortalamada ise % 21.5 dir Bu kişi bizim standartlarımıza göre skoru 4 üzerinden 1.75 olarak belirlenmiştir.

3 Adayın genel durum özet tablosu Güçlü yanları Yeni teknolojileri kullanma isteği Kod içi yorum satırı yazma Hata yakalama mekanizması Dokümantasyon Kaizen (iyileştirme) noktaları UI/UX Validasyon Loglama Unit test Temel İstatistikler LOC (Line of Code) 957 # Classes 15 Değerlendirme kriterleri Kriterler Junior Junior+ Senior Principle Gereksinimler Verilen iş isteklerini eksik olarak yerine getirebilir. Sadece verilen görevleri yerine getirir. Büyük resmi anlayarak, probleme kendi bakış açısını da katarak çözer. Çözümü görselleştirerek handover dokümanı hazırlar. UI/UX Hiç bir css kullanmaz. Ekran girdileri için bir validation yapmaz Css kullanır ve validation yapar Ergonomi unsurlarına (UI/UX) dikkat eder. Responsive tasarım yapısı kullanılır. Bu şekilde sistem değişik ortamlarda rahatlıkla kullanılabilir.

4 Tasarım prensipleri kullanımı Uygulamayı modüler yapıda yazmaz. Kodlar içerisinde if else, switch ifadeler fazladır. Encapsulation yaklaşımını kullanarak uygulamayı modüler hale getirir. Temel tasarım prensiplerini Tasarım prensiplerini etkin bir şekilde Yeni istekler uygulamanın mevcut durumunu bozmadan eklenir. Kod iş dilinde yazıldığı için (Domain Driven Design) kodu okuması ve anlaması kolaydır. Hata yakalama mekanizması Sadece hatasız akışa odaklanarak ( happy path ) kod geliştirir. Hata yakalama mekanizması yoktur. Temel düzeyde hata yakalama adımları mevcuttur. Hata yakalığında sadece kaydedilir. Hata durumlarını önceden tespit ederek sistemin hata durumuna düşmesini engellemeye çalışır. Hata fırladıktan sonra sistemin kaynaklarının boşaltılıp tekrardan kullanıma hazır olup olmadığı kontrol edilir. Kodun tüm katmanlarında hata yakalama ve yönetme mekanizmaları mevcuttur Veritabanı etkileşimi Temel düzeyde veritabanı bağlantısı Connection pool ve caching yapılarını uygulama içerisinde İhtiyaca göre ORM (Object Relation Mapping) Tablo index ve transaction yapılarına hakimdir. Veritabanının yükünü hafiflemek için in memory yapılarını Transaction yayılımı, izolasyon seviyelerini bilir.performans ve kötü durum senaryoları için önlemler geliştirir. Distributed Transaction (Two phase commit) çözümlerini bilir ve

5 Loglama Log mekanizmala rını kullanılmaz. Log araçlar kullanarak logları değişik ortamlara yazar. Logların DEBUG, INFO, WARNING, ERROR, FATAL gibi kategorilere göre loglanır. Logları incelemek ve kolaydır. Dilsel esneklik ve i18n Sistem içerisindeki tüm metinsel ifadeler hard coded şekilde kodların içerisinde gömülüdür. Dilsel esneklik ve i18n den bahsilemez. Dil esnekliği sağlanmıştır. Business gerektiğinde uyarı ve/veya hata metinlerini kendisi yazılımcıya bağlı kalmadan değiştirebilir. Bir adım öteye gidilmiş ve sisteme i18n desteği eklenmiştir. Business artık istediği dil için uyarı ve/veya hata metinlerini kendisi yazılımcıya bağlı kalmadan değiştirebilir. Unit Test Unit test yoktur. Unit test yazılmıştır. Testlerin içeriği hep happy path senaryolar hard coded ifadelerle test edilmiştir. Unit testler içerisinde bir çok senaryo kurgulanır. In memory database kullanılarak testler gerçek ortamı mock nesnelerle ile birlikte test edillir. Testleri Behavior driven development yaklaşımına göre senaryolar bazında gerçekleştirir. Saygılarımızla Kodcu.com ekibi

6