Pythonクラスの初期化

Question

私はオブジェクト指向設計の初心者です。

私はにPython関数を組み込もうとしています。 （機能は、I asked earlierとして動作します）。

はここに私のコードです：

import struct 
import urllib2 
import StringIO 


class wallpaper: 
    def __init__(self, url): 
     self.url = url 
     self.content_type = '' 
     self.height = 0 
     self.width = 0 
     image = urllib2.urlopen(self.url) 
     data = str(image.read(2)) 
     if data.startswith('\377\330'): 
      self.content_type = 'image/jpeg' 
      jpeg = StringIO.StringIO(data) 
      jpeg.read(2) 
      b = jpeg.read(1) 
      try: 
       while (b and ord(b) != 0xDA): 
        while (ord(b) != 0xFF): b = jpeg.read(1) 
        while (ord(b) == 0xFF): b = jpeg.read(1) 
        if (ord(b) >= 0xC0 and ord(b) <= 0xC3): 
         jpeg.read(3) 
         h, w = struct.unpack(">HH", jpeg.read(4)) 
         break 
        else: 
         jpeg.read(int(struct.unpack(">H", jpeg.read(2))[0])-2) 
         b = jpeg.read(1) 
        self.width = int(w) 
        self.height = int(h) 
      except struct.error: 
       pass 
      except ValueError: 
       pass 


x = wallpaper('http://i.imgur.com/rapwX.jpg') 
print x.url, "\t", x.content_type,"\t", x.height,"\t", x.width 

問題は、私は高さと幅のプロパティを初期化することができないということである（彼らは0等しいです）。

問題はどこですか？

編集：

私が見つけた問題は、ここにsolutionです。あなたのヒントをありがとう。

もう1つ質問：OOPルールに関する限り、私のアプローチは適切ですか？ で幅/高さを計算するか、他の方法（サイズなど）を作成する必要がありますか？

Answer 1

dataに2バイトしか読み取れません。

Answer 2

あなたは、単に例外を無視する場合は、問題を決定することが困難になります。

except struct.error: 
     pass 
    except ValueError: 
     pass 

少なくとも、あなたは、エラーメッセージを印刷する必要があります：あなたはあなたのをデバッグすることができるはず

except struct.error as exc: 
     print str(exc) 
    except ValueError as exc: 
     print str(exc) 

この方法コードを作成し、問題を解決します。

もちろん、私はあなたの実際のコードでインデントが正しいと仮定します。

Answer 3

この条件

ord(b) >= 0xC0 and ord(b) <= 0xC3 

あなたはself.widthとself.heightを割り当てる前に、ループから抜け出す、外側whileの最初の繰り返しで真である場合。たぶん、あなたのコードにあるレベルより2つ下の割り当てをインデントすることを意図したのでしょうか？