定时邮件改进：改成同步发送

polaris1119 · polaris1119 · commit 63ef6631996e · 2015-03-03T21:34:58.000+08:00
diff --git a/websites/code/studygolang/src/server/studygolang/background.go b/websites/code/studygolang/src/server/studygolang/background.go
@@ -37,7 +37,7 @@ func ServeBackGround() {
 	c.AddFunc("@daily", service.GenSitemap)
 
 	// 给用户发邮件，如通知网站最近的动态，每周的晨读汇总等
-	c.AddFunc("0 0 9 * * 1", service.EmailNotice)
+	c.AddFunc("0 0 4 * * 1", service.EmailNotice)
 
 	c.Start()
 }
diff --git a/websites/code/studygolang/src/service/mail.go b/websites/code/studygolang/src/service/mail.go
@@ -7,12 +7,12 @@
 package service
 
 import (
+	"bytes"
 	"html/template"
 	"net/smtp"
 	"strings"
 	"time"
 
-	"bytes"
 	. "config"
 	"logger"
 	"model"
@@ -92,11 +92,12 @@ func EmailNotice() {
 
 	var (
 		lastUid = 0
+		limit   = "500"
 		users   []*model.User
 	)
 
 	for {
-		users, err = userModel.Where("uid>?", lastUid).Order("uid ASC").FindAll()
+		users, err = userModel.Where("uid>?", lastUid).Order("uid ASC").Limit(limit).FindAll()
 		if err != nil {
 			logger.Errorln("find user error:", err)
 			continue
@@ -121,9 +122,11 @@ func EmailNotice() {
 				continue
 			}
 
-			go func(content, email string) {
-				SendMail("每周精选", content, []string{email})
-			}(content, user.Email)
+			SendMail("每周精选", content, []string{user.Email})
+
+			if lastUid < user.Uid {
+				lastUid = user.Uid
+			}
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -37,7 +37,7 @@ func ServeBackGround() {`
`37`	`37`	`c.AddFunc("@daily", service.GenSitemap)`
`38`	`38`
`39`	`39`	`// 给用户发邮件，如通知网站最近的动态，每周的晨读汇总等`
`40`		`- c.AddFunc("0 0 9 * * 1", service.EmailNotice)`
	`40`	`+ c.AddFunc("0 0 4 * * 1", service.EmailNotice)`
`41`	`41`
`42`	`42`	`c.Start()`
`43`	`43`	`}`