优化代码

server/dbdata/user.go

@@ -134,20 +134,18 @@ func checkLocalUser(name, pwd, group string) error {
 }
 
 // 用户过期时间到达后，更新用户状态，并返回一个状态为过期的用户切片
-func CheckUserlimittime() []interface{} {
-	u := &User{Status: 2}
-	if _, err := xdb.Where("limittime <= ?", time.Now()).And("status = ?", 1).Update(u); err != nil {
-		fmt.Println(err)
+func CheckUserlimittime() (limitUser []interface{}) {
+	if _, err := xdb.Where("limittime <= ?", time.Now()).And("status = ?", 1).Update(&User{Status: 2}); err != nil {
+		return
 	}
 	user := make(map[int64]User)
-	limitUser := make([]interface{}, 0)
 	if err := xdb.Where("status != ?", 1).Find(user); err != nil {
-		fmt.Println(err)
+		return
 	}
 	for _, v := range user {
 		limitUser = append(limitUser, v.Username)
 	}
-	return limitUser
+	return
 }
 
 var (

server/handler/link_home.go

@@ -21,11 +21,13 @@ func LinkHome(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 	index := &dbdata.SettingOther{}
-	dbdata.SettingGet(index)
+	if err := dbdata.SettingGet(index); err != nil {
+		return
+	}
 	w.WriteHeader(http.StatusOK)
 	if index.Homeindex == "" {
 		index.Homeindex = "AnyLink 是一个企业级远程办公 SSL VPN 软件，可以支持多人同时在线使用。"
-	}	
+	}
 	fmt.Fprintln(w, index.Homeindex)
 }