if/else文で重複したコードを避ける方法は？

Question

私がもし/ else文に2つのまったく同じロジックがあります。

if alert.get('comment_time_created') is None: 
here-> args = {'is_comment_visible': 1, 'comment_time_created': current_comment_time} 
     await self._db_alert.update_alert(alert['alert_id'], **args) 
    else: 
     first_comment_time_creation = datetime.strptime(alert['comment_time_created'], '%Y-%m-%dT%H:%M:%SZ') 
     current_comment_time = datetime.strptime(current_comment_time, '%Y-%m-%dT%H:%M:%SZ') 
     if current_comment_time > first_comment_time_creation: 
      await self._db_alert.update_alert(alert['alert_id'], is_comment_visible=1) 
     else: 
here->  args = {'is_comment_visible': 1, 'comment_time_created': current_comment_time} 
      await self._db_alert.update_alert(alert['alert_id'], **args) 

は、かつてこのロジックを実行する方法はありますか？

Answer 1

あなたはすべての条件で待っているようです。ちょうどあなたのkwargsはあなたが特定の条件のためにcomment_time_created argを持っていない場所を変更しています。これは以下のように減らすことができます：

args = {'is_comment_visible': 1} 
if alert.get('comment_time_created') is None: 
    args['comment_time_created'] = current_comment_time 
else: 
    first_comment_time_creation = datetime.strptime(alert['comment_time_created'], '%Y-%m-%dT%H:%M:%SZ') 
    current_comment_time = datetime.strptime(current_comment_time, '%Y-%m-%dT%H:%M:%SZ') 
    if current_comment_time <= first_comment_time_creation: 
     args['comment_time_created']= current_comment_time 

await self._db_alert.update_alert(alert['alert_id'], **args) 

Answer 2

外側のifの終わりの後にのみメソッドを呼び出します。すべてのブランチに共通する式をリッピングしてブロックを簡略化します。 このようなもの：

if alert.get('comment_time_created') is not None:  
    first_comment_time_creation = datetime.strptime(alert['comment_time_created'], '%Y-%m-%dT%H:%M:%SZ') 
    current_comment_time = datetime.strptime(current_comment_time, '%Y-%m-%dT%H:%M:%SZ') 
    if current_comment_time <= first_comment_time_creation: 
     comment_time_created = current_comment_time 

await self._db_alert.update_alert(alert['alert_id'], is_comment_visible = 1, comment_time_created = comment_time_created)