Wie vermeidet man doppelten Code in der if/else-Anweisung?

Question

Ich habe zwei genau die gleiche Logik in if/else-Anweisung:

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) 

Gibt es eine Möglichkeit, diese Logik einmal zu tun?

Answer 1

Sie scheinen die Wartezeit Linie auf jeder Bedingung zu tun, nur Ihre Kwargs ändern sich, wo Sie nicht haben comment_time_created Arg für eine bestimmte Bedingung. Dies kann reduziert werden auf:

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

Lassen Sie das dict. Rufen Sie die Methode erst nach dem Ende des äußeren if auf. Vereinfachen Sie den Block, indem Sie die Ausdrücke aufreißen, die allen Verzweigungen gemeinsam sind. So etwas wie dieses:

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)