このスクリプトをPythonで書くよりよい方法がありますか？

Question

私はこれがスロピーであることを知っています。私は初心者ですが、これを行うにはより良い方法があります。あなたのコードの最初の数行を見てみると

#!/usr/bin/env python 

""" 
Script to send any pending alerts via email 
""" 

import sys 
from pymongo import MongoClient 
from datetime import datetime 

#### begin 


def send_email(): 
print "Begin GAMER-ALERT-SEND-EMAILS %s " % (datetime.now()). 

if 'score_by_cat' >=9: 
     send_email() 

if 'score_by_cat' >=9: 
     print("Send Email") 
    else: 
     print('Nothing") 




print "Begin GAMER-ALERT-SEND-EMAILS %s " % (datetime.now()) 


client = MongoClient('mongodb://localhost:28057/') 
db = client.factor 

email_rcpt_list = ['example@momo.com'] 
        ['example2@momo.com'] 

#process the alert.. if it is high enough risk and the makes sense (not a security control etc) 

    for alert in db.risk_alerts.find({"$and" : [{'sent': False}]}): 
     if float(alert['metric'])>=9 and alert['category']!='security control': 
      #Call email send here 
      for email in email_rcpt_list: 
       print 'Call email and pass in the email address from email_rcpt_list' 
       print ' Sample alert: Vendor: %s Has an new item %s on %s of elevated risk %s in the category %s link: https://Gamer-dev.momo.net/profile/view/detail/%s' % (alert['vendor_name'],alert['key'],alert['source'],alert['metric'],alert['category'],alert['profile_id']) 
    #mark as sent always 
    db.risk_alerts.update_one({"_id":alert['_id']},{"$set": {"sent":True}}) 

print "End GAMER-ALERT-SEND-EMAILS %s" % (datetime.now()) 

Answer 1

：

私は何をしようとしていることはここでは9

より高いが、コードであることをいつでもスコアの変化を警告メールを送信し、あります

if 'score_by_cat' >=9 

print "Begin GAMER-ALERT-SEND-EMAILS %s " % (datetime.now()) 
timestamp=datetime.utcnow() 

条件文は、（例えば、if、else、for）はコロン（:）で終わらなければなりません。私は2番目の行がインデントされるべきだと信じています（スコアが9以上の場合、電子メールを送信する必要があります）。また、timestamp=datetime.utcnow()はこのコードでは使用されませんでした。したがって、この行を削除することができます。

ここで、コードに誤りがないかどうか再確認してください。

python