[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