6

このコードを単純化する方法を見つけようとしています。すべての if 条件のロジックは基本的に同じなので、重複した if を取り除きたい:

      if "video_codec" in profile:
          self.video_codec = profile["video_codec"]
      if "resolution_width" in profile:
          self.resolution_width = profile["resolution_width"]
      if "resolution_height" in profile:
          self.resolution_height = profile["resolution_height"]
      if "ratio" in profile:
          self.ratio = profile["ratio"]
      if "video_bitrate" in profile:
          self.video_bitrate = profile["video_bitrate"]
      if "profile" in profile:
          self.profile = profile["profile"]
      if "audio_codec" in profile:
          self.audio_codec = profile["audio_codec"]
      if "audio_channels" in profile:
          self.audio_channels = profile["audio_channels"]
      if "audio_bitrate" in profile:
          self.audio_bitrate = profile["audio_bitrate"]

これを 18 行ではなく 3 ～ 4 行で実行できることを願っています。

4

2 に答える 2

10 
for key, value in profile.iteritems():
    setattr(self, key, value)

やりたいことをすればいい

于 2012-07-05T15:48:05.677 に答える
7

すべてのキーと値のペアをprofileから の属性にコピーするだけの場合selfは、次を使用できます。

self.__dict__.update(profile)

profileコピーしたくないアイテムがいくつかある場合は、次を使用できます。

for attr in ("video_codec", "resolution_width", "resolution_height", "video_bitrate", 
             "ratio", "profile", "audio_codec", "audio_channels", "audio_bitrate"):
    if attr in profile: 
        setattr(self, attr, profile[attr])
于 2012-07-05T15:50:23.450 に答える