Three areas of code that I think can be optimized

[guowei-gong]

我不确定这些点修改后，可以让 qmgo 变的更好，如果可以，我很乐意修改并 pr。

1. Qmogo 中的 util.go 中的 SplitSortField 函数中，if len() 影响了代码缩进。代码应通过尽可能先处理错误情况/特殊情况并尽早返回或继续循环来减少嵌套。减少嵌套多个级别的代码的代码量。

修改前：

修改后：

func SplitSortField(field string) (key string, sort int32) {
	key = field
	sort = 1

	if len(field) == 0 {
		return key, sort
	}

	switch field[0] {
	case '+':
		key = strings.TrimPrefix(field, "+")
		sort = 1
	case '-':
		key = strings.TrimPrefix(field, "-")
		sort = -1
	}
	return
}

2. Qmogo 中的 cursor.go 中的 Next 函数中，err 判断可以简化。尽量缩小变量作用范围。

修改前：

修改后：

func (c *Cursor) Next(result interface{}) bool {
	if c.err != nil {
		return false
	}
	if c.cursor.Next(c.ctx) {
		if err := c.cursor.Decode(result); err != nil {
			c.err = err
			return false
		}
		return true
	}
	return false
}

3. Qmogo 中的 session.go 中的 StartTransaction 函数中，返回值做了多余的判断。不必要的 if else 判断。

修改前：

修改后：

func (s *Session) StartTransaction(ctx context.Context, cb func(sessCtx context.Context) (interface{}, error), opts ...*opts.TransactionOptions) (interface{}, error) {
	transactionOpts := options.Transaction()
	if len(opts) > 0 && opts[0].TransactionOptions != nil {
		transactionOpts = opts[0].TransactionOptions
	}
	return s.session.WithTransaction(ctx, wrapperCustomCb(cb), transactionOpts)
}

期待您的回复 :-)

[jiangz222]

• 1和2没有什么问题
• 3，其实改变了StartTransaction的行为，可以斟酌下
  其实可以直接提pr，在pr里讨论即可