bugfix

zhima1234 · zhima1234 · commit ad692b03748a · 2017-05-26T13:50:47.000+08:00
diff --git a/src/logic/rank.go b/src/logic/rank.go
@@ -12,6 +12,7 @@ import (
 	"model"
 	"time"
 
+	"github.com/garyburd/redigo/redis"
 	"github.com/polaris1119/logger"
 	"github.com/polaris1119/nosql"
 	"github.com/polaris1119/times"
@@ -102,9 +103,20 @@ func (self RankLogic) FindMonthRank(ctx context.Context, objtype, num int) (resu
 func (RankLogic) findModelsByRank(resultSlice []interface{}, objtype, num int) (result interface{}) {
 	objids := make([]int, 0, num)
 	viewNums := make([]int, 0, num)
-	for i, length := 0, len(resultSlice); i < length; i += 2 {
-		objids = append(objids, resultSlice[i].(int))
-		viewNums = append(viewNums, resultSlice[i+1].(int))
+
+	for len(resultSlice) > 0 {
+		var (
+			objid, viewNum int
+			err            error
+		)
+		resultSlice, err = redis.Scan(resultSlice, &objid, &viewNum)
+		if err != nil {
+			logger.Errorln("findModelsByRank redis Scan error:", err)
+			return nil
+		}
+
+		objids = append(objids, objid)
+		viewNums = append(viewNums, viewNum)
 	}
 
 	switch objtype {
diff --git a/src/vendor/manifest b/src/vendor/manifest
@@ -293,7 +293,7 @@
 		{
 			"importpath": "github.com/polaris1119/nosql",
 			"repository": "https://github.com/polaris1119/nosql",
-			"revision": "cdbfc63534442f43330efa5800fb98fbc00ace5f",
+			"revision": "5fdf0907446a240593948746a1640f7b677b2fba",
 			"branch": "master"
 		},
 		{

Original file line number	Diff line number	Diff line change
`@@ -293,7 +293,7 @@`
`293`	`293`	`{`
`294`	`294`	`"importpath": "github.com/polaris1119/nosql",`
`295`	`295`	`"repository": "https://github.com/polaris1119/nosql",`
`296`		`- "revision": "cdbfc63534442f43330efa5800fb98fbc00ace5f",`
	`296`	`+ "revision": "5fdf0907446a240593948746a1640f7b677b2fba",`
`297`	`297`	`"branch": "master"`
`298`	`298`	`},`
`299`	`299`	`{`