Jul 29 2008

Single Responsibility Principle(SRP)

Tag: Patterns,PrinciplesM. Cihat Altuntaş @ 2:24 am

Evet lütfen kemerlerinizi bağlayın koltuklarınıza sıkı sıkı yapışın nesneye yönelik programlamanın özüne doğru yolculuğa çıktık.Ufak çaplı sarsıntı yaşayabilirsiniz.Verdiğimiz rahatsızlıktan dolayı özür dileriz. :)

Şimdi bu kadar abarttığıma bakmayın ama konunun önemini belirmek için böyle bir giriş yaptım.Nesneye yönelik programlamanın,tasarımın en temel  ve yazılım tasarımının en önemli prensiplerinden olan Single-Responsibility Principle (SRP) hakkında bilgi verip ufak bir örnekle incelemeye çalışacağız.

Nesneye yönelik programlamanın hatta genel programlama mantığının temelinde yatan metodları nesneleri düşünelim.Neden kullandığımız dillerde bu tarz yapılar mevcut?Yönetimi kolaylaştırmak için.Amaç farklı işleri yapan kavramları birbirinden metodlar, sınıflar kullanarak yönetilmesini kolaylaştırmak için birbirinden ayırmak.Nesneye yönelik programlamada bunu nesneler kullanarak, prosedürel programlamada ise fonksiyonlar fonksiyon modülleri kullanarak yapıyoruz.Ama temel mantık aynı farklı işleri farklı farklı sınıflar, metodlar, katmanlar, kütüphaneler içine koyarak ayırıyoruz.

Tabi bazen farklı farklı işleri gidip aynı sınıf,metod ya da modül içine koyabiliyoruz(biri bizi durdursun).Hatta çoğu zaman bunu bu şekilde yapıyoruz diyebilirim.Etrafta yüzlerce binlerce satırlık sınıflar, metodlar görmemizin sebebi de bu. Tabi böyle yapınca sınıfın kodunu aşağıya doğru çekince en son satıra ulaşmak 10 saniye alıyor neredeyse. Mesela gördüğüm bir sınıf yaklaşık 5000 satırdı aşağıya çekmem bile baya süre alıyordu malesef.

Bertrand Meyer Object-Oriented Software Construction kitabında sınıfların,fonksiyonların,modüllerin nasıl olması gerektiğini aşağıdaki cümleyle çok güzel özetlemiş.

A class has a single responsibility: it does it all, does it well, and does it only.
Classes, interfaces, functions, etc. all become large and bloated when they’re trying to do too many things.

Aslında Single-Responsibility Principle (SRP) tamamen yukarıdaki sözle özetlenebilir. Bir sınıf,fonksiyon vs.. sadece tek bir sorumluluğu yerine getirmelidir  ve yerine getirdiği sorumluluğu iyi yapmalıdır.Birden fazla sorumluluk yerine getirmeye çalıştığı zaman aşırı büyür ve karmaşıklaşır.

Bu prensip aslında yazılım dünyasında uzun zamandır bilinen temel kavramlardan biridir. Kısaca bu prensibi şöyle açıklayabiliriz, kodumuzda, tasarladığımız modüllerde kullandığımız sınıflar,uygulamadaki katmanlarımız,modüllerimiz sadece tek bir sorumluluğunu yerine getirmelidir. Yani sınıfın,modülün sadece kendi ile alakalı işleri yapmasıdır diyebiliriz. Çünkü birden fazla sorumluluğu yerine getiren sınıfların,modüllerin değişmesi için birden fazla neden vardır. Kendi ile alakalı şeyleri yapmayan sınıfların modüllerin aşağıdaki gibi dezavantajları vardır.

    Anlaması zordur Tekrar kullanılabilmesi zordur. Yönetilmesi zordur. Hassas ve sürekli olarak diğer değişikliklerden etkilenen yapıdadır.

Modüller açısından bu prensibi inceleyelim.Katmanlı mimari modüler tasarım modül vb.. kavramlar da aslında bu tasarım prensibinin uygulanmasının bir sonucudur.Örnek olarak 3 katmanlı bir yapıda tasarlanmış bir uygulamada klasik olarak Sunum(Presentation Layer), İş(Business Layer) ve Veri(Data Layer) katmanları bulunsun. Adlarından da anlayabileceğimiz gibi her katman kendine ait sorumlulukları yerine getirir. Biri(Data Layer) veritabanı ile ilgilenirken diğeri(Business Layer) iş kurallarıyla alakalı sorumlulukları yerine getirir. Bir diğeride(Presentation Layer) bilgilerin kullanıcılara sunulması sorumluluğunu yerine getirir. Bu şekilde tasarımın en önemli faydalarından biride bir katmandaki değişikliğin diğer katmanları etkilememesi ve iyi tasarlanmış katmanların tekrar başka uygulamalarda kullanılabilmesidir. Çünkü her katman sadece kendi ile alakalı tek bir sorumluluğu yerine getirir. Böylece değişmesi için tek bir sebep vardır.Bu da o katmandaki ihtiyaçların değişmesi. Örnek olarak sunum katmanındaki bir değişim veri katmanını etkilemez çünkü ikisi de birbirinden bağımsız ayrı sorumlulukları yerine getirirler.Tabi bu sözde anlatması kolay fakat gerçekleştirmesi bir o kadar zor bir kavramdır.

Sınıflar için baktığımızda bu prensip anlatması ve anlaşılması en kolay fakat uygulamaya gelince en çok zorlandığımız prensiplerden biridir.Mesela benim 5000 satırlık işçi için sınıfım işciler ile alakalı iş mantığını yönetmekle sorumlu diyebilirim.Ama bu yönetme içinde veritabanı ile iletişim,import,export özellikleri,iş mantığını,cache yönetimi…. gibi bir sürü ayrı sorumluluğu yerine getiriyor yani birden fazla işi yapıyor ama ben olaya öyle bakmıyorum.Bu yüzden sorumluluk kavramını tam anlamıyla doğru şekilde uygulamak zaman ve deneyim isteyen bir süreçtir. Bunun bazı sebeplerinden biride sorumluluk kavramının karıştırılabilmesi, biraz bulanık bir kavram olmasından da kaynaklanmaktadır.

Mesela bir sınıf için baktığımızda sorumluluk nedir ? Genelde sorumluluk denince sınıfın bir methodu akla gelir fakat gerçekte birden fazla method bir sorumluluğu yerine getiriyor olabilir. Bence sorumluluğun en güzel özeti bir sınıfın değişmesi için bir sebep olarak tanımlayabiliriz. Yani bir sınıfın,metodun.. değişmesi için birden fazla neden varsa o sınıfın birden fazla sorumluluğu yerine getirdiğini söyleyebiliriz.Mesela yukarıda bahsettiğim işçi sınıfı için söyleyecek olursak bu sınıf veritabanına erişim mantığı değiştiğinde değişebilir, ayrıca değişik bir import seçeneği sunmak istediğimizde yine sınıfı değiştirmek gerekir,yeni bir cache algoritması eklemek istediğimizde doğal olarak tekrar bu sınıfı değiştirmek zorunda kalacağız. Bu da bize sınıfın değişmesi için birçok sebep verdi demekki bu sınıfın çok fazla sorumluluğu var onun sırtındaki yükü biraz azaltmamız lazım.

Örneklerin genelde gerçek hayattan olmasına özen göstermeye çalışıyorum. O yüzden daha önceden üzerinde çalıştığım kodun küçük bir kısmını değiştirerek incelemek için aşağıya yazıyorum.

class Contact
{
    private int _contactID;

    public int ContactID
    {
        get { return _contactID; }
        set { _contactID = value; }
    }

    private string _name;

    public string Name
    {
        get { return _name; }
        set { _name = value; }
    }

    private string _number;

    public string Number
    {
        get { return _number; }
        set { _number = value; }
    }

    private ContactType _type;

    public string Type
    {
        get { return _type; }
        set { _type = value; }
    }

    public int ImportExcel(File file)
    {
        CantactDao contactDao = null;
        List<Contact> contactList = null;
        int contactCount = 0;
        try
        {
            contactDao = new ContactDao();
            tr = contactDao.BeginTransaction(System.Data.IsolationLevel.ReadUncommitted);
            contactList = contactDao.GetContactsFromExcel(file);
            if (contactList != null &amp;&amp; contactList.Count > 0)
            {
                for (int i = 0; i < contactList.Count; i++)
                {
                    contactList[i].Type = ContactType.Friend;
                    contactList[i].Save();
                }
                contactDao.CommitTransaction(tr);
                contactCount = contactList.Count;
            }
        }
        catch (Exception ex)
        {
            contactDao.RollbackTransaction(tr);
        }
        return contactCount;
    }

    public int Save()
    {
        ContactDao contactDao = null;
        try
        {
            contactDao = daoFactory.GetContactDao();
            tr = contactDao.BeginTransaction();
            contactDao.Insert(this);
            contactDao.CommitTransaction(tr);
        }
        catch (Exception ex)
        {
            contactDao.RollbackTransaction(tr);
        }
        return this.ContactID;
    }
}
class ContactDao
{
    public void Insert(Contact contact)
    {
//veritabanı erişim kodları ile veritabanına Contact sınıfının eklenmesi
    }

    public void Update(Contact contact)
    {
        //veritabanı erişim kodları ile Contact sınıfının güncellenmesi
    }

    public void Delete(Contact contact)
    {
        //veritabanı erişim kodları ile sınıfın veritabanından silinmesi
    }

    public Contact GetByID(int contactID)
    {
        //veritabanı erişim kodları ile sınıfın veritabanından alınması
    }

    public List<Contact> GetContactsFromExcel(File file)
    {
        Contact contact = null;
        List<Contact> contactList = null;
        System.Data.OleDb.OleDbConnection connection = null;
        System.Data.OleDb.OleDbCommand selectCommand = null;
        try
        {
            string connectionString = @"Provider=Microsoft.Jet.OLEDB.4.0;Data Source=" + File.FilePath +
                                      ";Extended Properties=Excel 8.0";
            string selectQuery = "SELECT * FROM [Contacts$]";
            connection = new System.Data.OleDb.OleDbConnection();
            connection.ConnectionString = connectionString;
            connection.Open();
            selectCommand = new System.Data.OleDb.OleDbCommand(selectQuery, connection);
            System.Data.OleDb.OleDbDataReader dataReader = selectCommand.ExecuteReader();
            if (dataReader != null &amp;&amp; dataReader.HasRows)
            {
                contactList = new List<Contact>();
                while (dataReader.Read())
                {
                    contact = new Contact();
                    contact.Name = dataReader["Name"];
                    contact.Number = dataReader["Number"];
                    //....diğer alanların alınması ve kontrolü
                    contactList.Add(contact);
                }
            }
        }
        catch (Exception ex)
        {
            throw ex;
        }
        return contactList;
    }
}

Şimdi yukarıdaki kodun yapısını biraz inceleyelim. Bir SMS programımız var SMS programının özelliklerinden biri dışarıdan excel içindeki kişi listesini programımıza import edebiliyoruz.Genelde çoğu programda bu tarz özellikleri sizde kullanmıştırsınız.Kodda gördüğünüz gibi bir adet Contact sınıfımız var bu sınıf programımızda kullandığımız kişileri temsil ediyor.Bu sınıfın metodlarına bakarsak gördüğünüz gibi bir adet excel dosyasından import eden ImportExcel metodumuz var. Ayrıca veritabanına bilgileri kayıt eden bir adet Save metodumuz var.Yani ActiveRecord tasarım kalıbını uygulayarak kendini veritabanına kayıt ediyor. ContactDao adında DAO(Data Access Object) tasarım kalıbını uygulayan bir adet sınıfımız var.Bu sınıfımızda Contact sınıfımızı veritabanına ekleme,güncelleme, silme ve Excel dosyasından veri alma işlemini yapıyor. Şimdide UML olarak static sınıf diyagramına bakalım.

Yukarıdaki Uml diyagramını çizmemin tek amacı görsel olarak sınıflar arasındaki ilişkileri görmek kodu okumaktan daha kolay olmasıdır. Şimdi yukarıdaki diyagram yardımıyla sınıfları daha yakından incelemeye başlayalım.Öncelikle sınıfların birbirine olan bağımlılığına(Dependency) bakalım. Etrafta uçuşan okları görüyorsunuz görüntü pek iç açıcı değil açıkçası. Contact sınıfımız üzerindeki ImportExcel metodu sayesinde File sınıfına bağımlı. Ayrıca Save metodunu kullandığında da Transaction oluşturduğu ve CantactDao sınıfının metodunu çağırdığı için bu sınıfa ve ADO.NET sınıflarına da bağımlı.

Öncelikle kötülüklerin anası olan bağımlılık neden bu kadar kötü kısaca bahsedelim. :) Bağımlılığı fazla olan sınıfları yeniden kullanmanız (Reuse) çok zordur. Çünkü kullanmak istediğiniz sınıf diğer  birçok sınıfa bağımlıdır onunla birlikte diğer sınıflarıda projede dahil etmeniz gerekir. Tabi bu da o kadar kolay değildir.Çünkü diğer sınıflarında birçok diğer sınıfa bağımlı olduğu düşünürsek küçük bir sınıfı yeniden kullanmak istediğinizde bütün projeyi diğer projeye dahil etmeniz gerekir. Diğer bir kötü yanı bağımlı olduğu sınıflarda meydana gelen hataların o sınıfı kolaylıkla etkilemesidir. Bu konuda daha geniş bilgi için Dependency Inversion prensibine bakabilirsiniz.Ayrıca gördüğünüz gibi Contact sınıfı,ContactDao sınıfı kendi ile alakalı olmayan metodlar ile gittikçe büyüyor. Sınıfların büyüdükçe yönetilmesinin çok zor olduğunu biliyorsunuz. Benim gibi 5000 satırlık kodun içinde değişiklik yapmanın hatanın ne olduğunu bulmanın nasıl bir kabul olduğunu bilirsiniz.

Şimdi bu kadar şeyden bahsettik bunların Single Responsibility Principle ile alakasının ne olduğunu merak etmiş olabilirsiniz haklı olarak. Kısaca burada ki bağımlılığın sınıfların gittikçe büyümesinin sebebi sınıfların birden çok sorumluluğunun olmasıdır. Contact sınıfımız hem sistemimizde bir kavramı temsil ederken hemde onu sisteme import etme işlemini yapıyor. Ayrıca ContactDao sınıfımız hem veritabanı ile alakalı Insert,Update… gibi işlemleri yaparken ayrıca Excel dosyasından kayıtları okuma işlemini yapıyor. Bu sınıfın değişmesi için birçok neden sıralayabiliriz.Import mantığımız değişir, Contact sınıfını değiştiririz. İş mantığımız değişir, Contact sınıfını değiştiririz. Excel dosyasından okuma mantığımız değişir, ContactDao sınıfımız değişir.Kullanılan teknolojiyi ADO.NET yerine başka birşey kullanmak isteriz iki sınıfımızda değişmek zorunda kalır.Yani değişime karşı sınıflarımız oldukça kırılgan yapıdalar.

Şimdi fazla sorumluluğu olan sınıfların yüklerini biraz hafifletelim. Bunu yapmak için bütün yapılan sorumlulukları liste halinde yazalım.Bu listeyi sorumlulukları atamada kullanacağız.

    Kişilerin sistemnde temsil edilmesi (Contact sınıfı) Kişilerin veritabanı ile alakalı işlerin yapılması (ContactDao sınıfı) Kişilerin import edilerek sisteme kayıt edilmesi (Contact sınıfı) Excel dosyasından satırların okunarak kişi nesnesine çevrilmesi (ContactDao sınıfı)

Gördüğünüz gibi 4 tane sorumluluk 2 tane sınıf arasında paylaştırılmış. Biz kodumuzu değiştirerek her sorumluluğu tek bir sınıfa atayacağız yani Single Responsibility prensibini uygulayacağız. Bunun için öncelikle bu sorumluluklara iyi birer isim bulmaya çalışalım. Dikkat edin bunu sezgi yoluyla yapıyorum.

    Öncelikle kişilerin sistemde temsil edilmesi Contact sınıf bu aynen kalacak. Kişilerin veritabanı ile alakalı işlerinin yapılması zaten bu işi yapan sınıfımız var ContactDao adı üzerinde Data Access Object Kişilerin import edilmesi buna ContactImporter diyelim Kişilerin Excel dosyasından okunması buna da ContactExcelReader olsun.

Sınıflarımızın adlarını belirledikten sonra fazla sorumluluğu olan sınıflardan kodların yeni sınıflara taşınması işlemi var. Bu işlemi yaparken tabiki kes-yapıştır yapmanızı tavsiye etmiyorum. Birçok istenmeyen hata ile karşılaşabilirsiniz. Bunu Refactoring, ve Unit Testing desteğiyle en güvenli şekilde yapabilirsiniz. Konumuzu fazla dağıtmamak için değinmiyorum amacımız Single Responsibility Prensibini açıklamak.O yüzden benim şuanda kes-yapıştır yapmama fazla aldırmayın.Lafı fazla uzatmadan yeni kodlarımızı aşağıya yazalım.

internal class Contact
{
    private int _contactID;

    public int ContactID
    {
        get { return _contactID; }
        set { _contactID = value; }
    }

    private string _name;

    public string Name
    {
        get { return _name; }
        set { _name = value; }
    }

    private string _number;

    public string Number
    {
        get { return _number; }
        set { _number = value; }
    }

    private ContactType _type;

    public string Type
    {
        get { return _type; }
        set { _type = value; }
    }
}
class ContactImporter
{
    private ContactDao contactDao = new ContactDao();
    private CntactExcelReader contactReader = new ContactExcelReader();

    public int ImportFromExcel(File file)
    {
        List<Contact> contactList = null;
        int contactCount = 0;
        try
        {
            tr = contactDao.BeginTransaction(System.Data.IsolationLevel.ReadUncommitted);
            contactList = contonctReader.GetContactsFromExcel(file);

            if (contactList != null &amp;amp;amp;amp;amp;amp;amp;amp;&amp;amp;amp;amp;amp;amp;amp;amp; contactList.Count > 0)
            {
                for (int i = 0; i < contactList.Count; i++)
                {
                    contactList[i].Type = ContactType.Friend;
                    contactList[i].Save();
                }

                contactDao.CommitTransaction(tr);
                contactCount = contactList.Count;
            }
        }
        catch (Exception ex)
        {
            contactDao.RollbackTransaction(tr);
        }
        return contactCount;
    }
}
class ContactDao
{
    public int Save(Contact contact)
    {
        try
        {
            Transaction tr = this.BeginTransaction();
            this.Insert(contact);
            this.CommitTransaction(tr);
        }
        catch (Exception ex)
        {
            this.RollbackTransaction(tr);
        }

        return contact.ContactID;
    }

    public void Insert(Contact contact)
    {
//veritabanı erişim kodları ile veritabanına Contact sınıfının eklenmesi
    }

    public void Update(Contact contact)
    {
//veritabanı erişim kodları ile Contact sınıfının güncellenmesi
    }

    public void Delete(Contact contact)
    {
//veritabanı erişim kodları ile sınıfın veritabanından silinmesi
    }

    public Contact GetByID(int contactID)
    {
//veritabanı erişim kodları ile sınıfın veritabanından alınması
    }
}
class ContactExcelReader
{
    private GetContactsFromExcel(File file)
    {
        Contact contact = null;
        IList<Contact> contactList = null;
        System.Data.OleDb.OleDbConnection connection = null;
        System.Data.OleDb.OleDbCommand selectCommand = null;

        try
        {
            string connectionString = @"Provider = Microsoft.Jet.OLEDB.4.0; Data Source =" + File.FilePath +
                                      ";Extended Properties = Excel 8.0";

            string selectQuery = "SELECT*FROM[Contacts$]";

            connection = new System.Data.OleDb.OleDbConnection();
            connection.ConnectionString = connectionString;
            connection.Open();
            selectCommand = new System.Data.OleDb.OleDbCommand(selectQuery, connection);
            System.Data.OleDb.OleDbDataReader dataReader = selectCommand.ExecuteReader();
            if (dataReader != null &amp;amp;&amp;amp; dataReader.HasRows)
            {
                contactList = new List<Contact>();

                while (dataReader.Read())
                {
                    contact = new Contact();
                    contact.Name = dataReader["Name"];
                    contact.Number = dataReader["Number"];
                    //....diğer alanların alınması ve kontrolü
                    contactList.Add(contact);
                }
            }
        }
        catch (Exception ex)
        {
            throw ex;
        }

        return contactList;
    }
}

Yeni sınıflarımıza bir de UML diyagramı ile bakalım.

Şimdi yukarıda ki diyagram yadımıyla kodumuzun son haline bakalım. Gördüğünüz gibi artık Contact sınıfımız ne File,ContactDao gibi sınıflara ne de herhangi bir ADO.NET teknolojisine bağımlı.Fazla sorumluluğu üzerinden alınca diğer sınıflardan bağımsız hale geldi. Diğer sınıflara baktığımızda sadece kendi işlerini yapan ve gereksiz bağımlılıklardan kurtulmuş sınıflar var.ContactExcelReader sadece Excel dosyasından Contact okuma işini yapıyor,ContactImporter liste halinde verilen kişileri veritabanına import ediyor. ContactDao sadece veritabanı ile alakalı işlemleri yapıyor.Artık sınıfların değişmesi için tek sebep var. Kodun anlaşılması ve yönetilebilmesi daha kolay.

Single Responsibility prensibini her zaman uygulamak çok faydalı olmayabilir. Mesela çok küçük bir projede uygulamak sınıf sayısını arttırdığı için sistem daha komplex olur.Mesela yukarıda ActiveRecord tasarım kalıbını uygulayan Save metodunu gidip ContactDao sınıfı üzerine taşıdık.Fakat bu tasarım kalıbı küçük çaplı, iş mantığının yoğun olmadığı çoğu projede başarı kullanılabiliyor.Ama çoğu durumda uygulamanız için  bu prensibi uygulamak uygulamanın yönetimi bakımı hata oranı için oldukça önemli rol taşıyor.


Jul 16 2008

Refactoring-Consolidate Duplicate Conditional Fragments

Tag: Code Smells,RefactoringM. Cihat Altuntaş @ 2:16 pm

Evet bu aralar parserlar ile boğuşuyorum kendi çapımda ufak bir dil yapmaya çalışıyorum. Programlamlama dillerini kullanması kolay ama yapması gerçekten zormuş ilk olarak onu söyleyebilirim. Örnek bir parser bulup nasıl yapıldığı hakkında fikir edinmek için onu inceliyordum.Kodu çok karmaşık ve anlaması çok zor olduğu için önce testlerin sağlamlığından emin olana kadar Unit Test hazırladım. Ardından kodu daha okunabilir,anlaşılabilir olması,ve code smell’lerden kurtulmak için refactoring yaparım.Bunu yaparken aşağıdaki gibi bir kod ile karşılaştım.Aşağıdaki kodu öncelikle biraz inceleyin bir problem görebilecekmisiniz?

        private double atom() throws InterpreterException {
            double result = 0.0;

            if (tokenizer.getTokenType() == TokenTypes.NUMBER){
                try{
                    result = Double.parseDouble(tokenizer.getToken());
                }
                catch (NumberFormatException exc){
                    throw new InterpreterException("Syntax Error :" + exc.getMessage());
                }
                tokenizer.obtainNextToken();
            }
            else if (tokenizer.getTokenType() == TokenTypes.VARIABLE){
                result = getValueOfVariable(tokenizer.getToken());
                tokenizer.obtainNextToken();
            }
            else
                throw new InterpreterException("Syntax Error");

            return result;
        }

Şimdi yukarıdaki koda baktığınızda ilk başta büyük bir problem gözünüze çarpmıyo olabilir benimde çarpmıyordu çünkü. Fakat biraz daha incelediğimde beni rahatsız eden bir kötü kod barındırdığını farkettim. Şimdi if-else yapılarını beraberce inceleyelim. İlk if yapısına baktığımızda Token tipinin numara olup olmadığını kontrol ediyor. Eğer numaraysa Tokeni double çevirip sonucu üretiyor ve bir sonraki tokeni alıyor. İkinci else if yapısına baktığımızda tokenin değişken olup olmadığını kontrol ediyor. Ardından değişken ise değerini alıp sonucu üretiyor ve burada da bir sonraki tokeni alıyor.Son olaral eğer hiçbiri değilse hata fırlatıyor.Yani değişken ya da numara değilse hata fırlatıyor.

Şimdi buradaki problem biraz zor farkedilse de if-else yapıları içerisinde tekrarlayan kod içeriyor.Yukarıda açıklamada farkettiyseniz Eğer token değişken ya da numara ise bir sonraki tokeni alma işlemi ikisinde de yapılıyor.Son else yapısında bu yapılmamış aslında son else ilk başta yapılması gereken bir hata kontrolü.Hani hata kontrolünü başa alırsak bu tekrar daha bariz bir şekilde ortaya çıkacaktır. Kodu aşağıdaki gibi düzenleyip hata kontrolünü başa alalım. Ve tekrar içeren yerleri işaretleyelim.

        private double atom(){
            double result = 0.0;

            if (tokenizer.getTokenType() != TokenTypes.NUMBER &amp;&amp; tokenizer.getTokenType() != TokenTypes.VARIABLE)
                throw new InterpreterException("Syntax Error");

            if (tokenizer.getTokenType() == TokenTypes.NUMBER){
                try{
                    result = Double.parseDouble(tokenizer.getToken());
                }
                catch (NumberFormatException exc){
                    throw new InterpreterException("Syntax Error : " + exc.getMessage());
                }
                tokenizer.obtainNextToken();
            }
            else if (tokenizer.getTokenType() == TokenTypes.VARIABLE)
            {
                result = getValueOfVariable(tokenizer.getToken());
                tokenizer.obtainNextToken();
            }

            return result;
        }

Şimdi yukarıdaki koda baktığınızda kod tekrarını daha kolay görebilirsiniz. Tekrar olan yerleri kırmızı içinde görüyorsunuz.Şimdi tekrarlardan kurtulup kodu daha iyi hale getirmek için aşağıdaki gibi tekrar refactoring yapıyoruz.

private double atom(){
    double result = 0.0;

    if (tokenizer.getTokenType() != TokenTypes.NUMBER &amp;&amp; tokenizer.getTokenType() != TokenTypes.VARIABLE)
        throw new InterpreterException("Syntax Error");

    if (tokenizer.getTokenType() == TokenTypes.NUMBER){
        try{
            result = Double.parseDouble(tokenizer.getToken());
        }
        catch (NumberFormatException exc){
            throw new InterpreterException("Syntax Error : " + exc.getMessage());
        }
    }
    else if (tokenizer.getTokenType() == TokenTypes.VARIABLE){
        result = getValueOfVariable(tokenizer.getToken());
    }
    tokenizer.obtainNextToken();
    return result;
}

Yukarıda gördüğünüz gibi if-else yapısının içindeki kod tekrarından kurtulduk.Bu şekilde şartlı yapıların içindeki tekrar eden kodu tek bir yere alıp tekrardan kurtulma işlemine Consolidate Duplicate Conditional Fragments diyoruz. Burada ismi fazla önemli değil aslında yaptığı işlemi anlamanız yeterli.Bu kodu aslında biraz daha düzenleyebiliriz. İlk olarak gözüme çarpan hata kontrolündeki uzun if cümlesi oluyor. Bu uzun ifade daha önce yazdığım Decompose Conditional kullanılarak daha basit ve anlaşılabilir hale getirilir.Onuda size bırakıyorum. Kolay gelsin…


Jul 13 2008

Interface mi, Abstract mı?

Tag: Yazılım MühendisliğiM. Cihat Altuntaş @ 1:44 pm

Genellikle Nesneye Yönelik Programlamaya yeni başlayanların kafasındaki büyük soru işaretlerinden bazıları "Interface ile Abstract arasındaki farklar nelerdir?, Ne zaman Interface ne zaman Abstract sınıfları kullanmalıyız ? …." gibi sorulardır.

Acı ve komik bir anımdan bahsetmeden olmaz. Okulda Java hocama neden Interface kullanırız diye sorduğumda bana "Genelde pek kullanılmaz. Fakat benim daha önce çalıştığım bilmem ne işyerinde bi abi interface kullanarak yazılım geliştirirdi. Biz 2 haftada yapıyorsak o bir haftada bitirirdi" demişti. Tabi daha sonra interface,abstract gibi kavramların nesneye yönelik programlamada ne kadar önemli kavramalar olduğunu öğrenince neden o abinin 1 haftada bitirdiğini anladım. :)

Interface ve Abstract kavramları nesneye yönelik programlamanın en temel ve önemli kavramlarından biridir.Önceki yazılarımızda Interface nedir ne zaman kullanılır?, ve Abstract nedir ne zaman kullanılır? ile bu kavramların nerede nasıl kullanıldığına ayrı ayrı konular altında değinmiştik. Eğer okumadıysanız öncelikle eski yazıları okumanızı tavsiye ederim. Aslında iki yazıyıda dikkatlice okuduğunuzda hangi durumda hangisi kullanılabilir diye karar verebilirsiniz.  Bu yazıda da genel bir özet yapıp bu kavramlar hakkında son düşüncelerimi ve kişisel tercihimi yazmak istedim.

Öncelikle daha önceki yazılarda Abstract sınıfların genellikle IS-A(dır,dir) ilişkilerinde,kalıtım(inheritance) özelliğini kullanarak kod tekrarını azaltmak için kullanıldığını söylemiştik. Interface sınıfların ise daha çok CAN-DO(yapabilir..) tarzı ilişkilerde değişen kavramları uygulamadan soyutlamak için kullanıldığını söylemiştik. Birde bu iki kavramın avantaj ve dezavantajlarına bakalım.Sonuçta Abstract bir sınıfın bütün metodlarını abstract yaparak onu da aynı bir interface gibide kullanabiliriz.

Öncelikle bence en büyük fark Abstract sınıfların tekli kalıtım(inheritance ) kullanması Interface sınıfların ise çoklu kalıtıma(multiple inheritance) izin vermesidir.Bildiğiniz gibi bir sınıfı başka bir sınıftan,ya da abstract bir sınıftan türettiğiniz zaman başka bir sınıftan daha türetme imkanınız olmuyor. Fakat interface sınıflarda durum daha farklı. Bir sınıfı istediğiniz kadar interfaceden türetebilirsiniz. Bu bakımdan interface sınıflar abstract sınıflardan oldukça daha esnek diyebiliriz. Fakat interface sınıflara baktığımızda abstract sınıflardan daha yavaş çalıştığı için hız bakımından devavantajı var diyebiliriz.

Şimdi hız mı esneklik mi diye bana soracak olursanız tabiki tercihim esneklikten yana olacaktır. Bu soruyu 2-3 sene önce sorsaydınız önce hız derdim büyük ihtimalle :)

Kendi kişisel tercihim olarak eğer soyutlamak istediğim yapıda ilişki CAN-DO ise ve hiç ortak kod yoksa sadece interface kullanırım. Fakat ortak kod içeriyorsa öncelikle bir interface(dikkat edin yine interface koyarım) koyup ardından ortak kodu kullanan bir abstract sınıf koyup onu o interfaceden türetir ve diğer sınıfları interface sınıfını kullanır hale getiririm.Burada interface i koymasakta olur fakat koymamın sebebi esnekliği arttırması ve özellikle Test Driven Development açısından Mock objelerin oluşturulması için interface kullanan sınıfların test edilmesinin çok daha kolay olması.Mock objelere ayrı bir konu başlığı altında değiniriz fazla uzatmayalım.

Eğer soyutlamak istediğim yapı IS-A ilişkisi ise sadece Abstract sınıf kullanırım. Abstract sınıfların sevmediğim yanı kalıtım(inheritance) diyebilirim.Kalıtım ilişkisi herzaman bağımlılığı(coupling) fazla olan bir ilişkidir. Yani türettiğiniz sınıfa tamamen bağımlı oluyorsunuz. Türettiğiniz sınıftaki herhangi bir değişiklik alt sınıfları etkiliyor. Fakat interface sınıflarda bu tarz bir problem daha az interface içindeki bir metodu değiştirmedikçe hiç bir sınıf bundan etkilenmiyor.Geliştirdiğiniz sınıflar tamamen plug-in mantığı ile interface sınıflar sayesinde hiç problemsiz değiştirilebiliyor. Bu yüzden kalıtıma her zaman şüpheyle yaklaşıyorum. Zaten Object OOP nin insanlar tarafından uzun yıllar kullanıldıktan sonra da aynı fikre varılmış. Kalıtımın bu problemli yapısı görüldüğü için daha sonra kalıtım yerine birleşim(Composition over Inheritance) kullanmak çoğu durumda tercih edilen bir yöntem.OOP için oldukça önemli olan bu konuyuda başka bir yazı konusu olarak ileriye bırakıyorum.

Konu ile ilgili yazıyı yazdığım gün konu ile alakalı çok güzel bir webcast e denk geldim. Buradan sizde izleyebilirsiniz. Konu Interface vs Abstract sınıflar. Burada konuşmacı benden daha katı olarak tamamen interface kullanılması gerektiğini savunuyor.Kısmen katılsamda bu şekile katı bir interface kullanımı her zaman doğru olmayabilir. Ayrıca konu ile ilgili daha derin bilgi edinmek için okumaya fırsat bulamadığım Interface Oriented Design kitabını okuyabilirsiniz. Ayrıca Wikide de  kısa bir bilgi verilmiş(Interface Based Programming).

Bu konuda şuanda aklıma gelenler bu kadar.Dediğim gibi konu bir blog yazısına sığmayacak kadar detaylı. En azından biraz ışık tutabilmiştirim. Ne kadar örneği bol tutmaya çalışsamda kendiniz yazmadıkça projelerinizde kullanmadıkta biraz havada kalabilir. O yüzden eller klavyeye diyorum…


Jul 13 2008

Bedava E-Book , Foundations of Programming

Tag: KitaplarM. Cihat Altuntaş @ 12:30 pm

Blogunu ve yazı serisini severek takip ettiğim Karl Seguin Foundations of Programming altında daha önceden yazdığı serilerini e-book formatında toplamış. Kitapta gerçekten çok faydalı bilgiler bulacağınıza emin olabilirsiniz. Özellikle, OOP,DDD,TDD,.NET hakkında çok güzel bilgileri vermiş. Kitabı yukarıdaki linkten indirip okuyabilirsiniz.