Pythonのコードモジュールの品質を向上させるためのアドバイス

Question

最近、辞書を有効なCSSコードに変換するモジュールを書く考えがありました。私のコードについてのあなたのコメントを聞きたいと思います。 BSDなど ライセンスMIT、

コードは非常に簡単です：

#pss.py 

__version__ = '1.0' 

class PSS: 
    def __init__(self, obj): 
     self.obj = obj 
     self.__data = {} 
     self.__parse(obj) 

    def __repr__(self): 
     return self.__build(self.obj) 

    def __build(self, obj, string = ''): 
     for key, value in sorted(self.__data.items()): 
      if self.__data[key]: 
       string += key[1:] + ' {\n' + ''.join(value) + '}\n\n' 

     return string 

    def __parse(self, obj, selector = ''): 
     for key, value in obj.items(): 
      if hasattr(value, 'items'): 
       rule = selector + ' ' + key 
       self.__data[rule] = [] 
       self.__parse(value, rule) 

      else: 
       prop = self.__data[selector] 
       prop.append('\t%s: %s;\n' % (key, value)) 

インポートモジュール：

#test.py 

from pss import * 

css = PSS({ 
    'html': { 
     'body': { 
      'color': 'red', 
      'div': { 
       'color': 'green', 
       'border': '1px' 
      } 
     } 
    } 
}) 

print(css) 

結果：

html body { 
     color: red; 
} 

html body div { 
     color: green; 
     border: 1px; 
} 

だから私は改善するために、あなたのアドバイスを必要としますコードの品質

Answer 1

1. 引数をタイプチェックしないでください。__init__：発信者があなたよりよく知っているかもしれません。たとえば、 "dict"が必要です。私はdictのサブクラス、またはマッピングプロトコルの他の実装を渡すかもしれません。あなたの現在のコードはそれを拒否します。
2. このような__namesは使用しないでください。迷惑です。 Pythonの文化を学ぶなら、データを隠す必要はありません。 Pythonには本当にデータを隠すための防弾方法がないので試してはいけません。
3. self.__data[i].__len__()は、単にlen(self.__data[i])であり、if len(xx):は、ちょうどif xx:になります。 __len__がlenよりもPythonicだと思った理由は不明です。一般に、ユーザコードはdunderメソッドを呼び出さず、Pythonがあなたのために呼び出すために提供されています。
4. あなたのお名前はあなたの出力と同じではないはずです。 reprはデバッグコンテキストでオブジェクトを表示するためのものです。
5. クラスの出力を取得する方法がわかりません。 Reprは唯一の公的な方法だと思われますが、これは非常に奇妙です。少なくとも__str__メソッドにしてください。
6. prop.insert(prop.__len__(), x)は、ちょうどprop.append(x)です。
7. 文字列の書式は、一緒に文字列を追加するより簡単に、より読みやすいです：私はpylintを使用することをお勧めします'\t%s: %s\n' % (i, obj[i])

Answer 2

、それはあなたのコードスタイルの欠陥を指摘します。それ以外の場合は、コードをできるだけ単純で読みやすいものにしておけば、それは好みの問題です。