[Gelistirici] r21707 - trunk/python-modules/pardus-python/pardus

Ozan Çağlayan ozan at pardus.org.tr
4 Nis 2009 Cmt 18:52:05 EEST


Fatih Aşıcı wrote:
> Perşembe 05 Mart 2009 tarihinde, Gürer Özen şunları yazmıştı: 
>> On Thursday 05 March 2009 01:57:01 Fatih Aşıcı wrote:
>>>> +        try:
>>>> +            self.cp.read(db_file)
>>>> +        except:
>>>> +            print "Network configuration file %s is corrupt" % db_file
>>>>          self.__unlock()
>>> Şu an sadece Net.Link betikleri mi iniutils kullanıyor bilmiyorum; ama bu
>>> çözüm ve hatta bu hata mesajı bu modülü kullanmak isteyecek başka
>>> uygulamalara uygun olmayabilir.
>> Bu tür boş except doğru bir yöntem değil. Mesela read yerine red yazsan
>> gene db corrupt diye hata alacaksın. read fonksiyonu IOError veriyordu
>> sanırım, dökümanında yazıyordur, özel olarak onu yakalaman lazım.
>>
>> Bir de kitaplık içinde print kullanmak da güzel bir şey değil. Burada doğru
>> çözüm bir exception tanımlamak, hata durumunda (mesela burada read IOError
>> atabilir), kendi exception'ımızı bu hata mesajıyla raise etmek, üst seviye
>> kodun da bunu yakalayıp devam edip etmeyeceğine, yada hatayı nasıl
>> kullanıcıya sunacağına kendi karar vermesi.
> 
> Bahadır, Ozan? Bunun için söyleyeceğiniz bir şey var mı? Yeni xdm ve zorg için 
> pardus-python release edilmesi gerekiyor. Unutmadan şunu da halletsek?

Konuya yabancıyım, pardus-python ne yapıyor çok da iyi bilmiyorum ancak yukardaki tartışmada
doğru olan yaklaşım Gürer'in söylediği.

Pazartesi Bahadır'la bakalım olmadı?

-- 

Ozan Çağlayan
<ozan_at_pardus.org.tr>



Gelistirici mesaj listesiyle ilgili daha fazla bilgi