field.Field 缺失运算方法

[shuqingzai]

GORM Playground Link

Description

在 field.Field 中有求和方法，缺失一些方法

1. 运算方法
2. Between 相关

参考:

1. field:

   gen/field/field.go

   Line 69 in ebb6364

   func (field Field) Sum() Field {

2. int:

   gen/field/int.go

   Lines 70 to 98 in ebb6364

   	// Add ...
   	func (field Int) Add(value int) Int {
   	return Int{field.add(value)}
   	}
   	// Sub ...
   	func (field Int) Sub(value int) Int {
   	return Int{field.sub(value)}
   	}
   	// Mul ...
   	func (field Int) Mul(value int) Int {
   	return Int{field.mul(value)}
   	}
   	// Div ...
   	func (field Int) Div(value int) Int {
   	return Int{field.div(value)}
   	}
   	// Mod ...
   	func (field Int) Mod(value int) Int {
   	return Int{field.mod(value)}
   	}
   	// FloorDiv ...
   	func (field Int) FloorDiv(value int) Int {
   	return Int{field.floorDiv(value)}
   	}

field.Field 应该与其它类型保持同样的方法，以便支持自定义类型

如：使用 decimal.Decimal 作为金额字段类型时，无法运算，自增或自减

UPDATE `user_wallet` SET `frozenBalance` = `frozenBalance` + 10.05 WHERE userId = 311 AND balance > `frozenBalance` + 10.05;

[qqxhb]

如果确定是数值运算，可以把这个字段gen field类型定义成float,或者int

[shuqingzai]

如果确定是数值运算，可以把这个字段gen field类型定义成float,或者int

@qqxhb 你的建议值得商榷

1. 如果数据库已经使用 decimal ，并且涉及一些金额相关的字段，那么你的建议就毫无意义
2. 对于自定义类型有 sum 却没有其它计算方法也不合理，对应 go 使用 decimal 业界一般都是使用 decimal.Decimal 吧
3. 虽然 int 或 float 有一些计算方法，但是有 sum 没有 avg 、max 、min 一类常用的计算方法也是值得商榷的

[shuqingzai]

如果确定是数值运算，可以把这个字段gen field类型定义成float,或者int

@qqxhb 你的建议值得商榷

1. 如果数据库已经使用 decimal ，并且涉及一些金额相关的字段，那么你的建议就毫无意义
2. 对于自定义类型有 sum 却没有其它计算方法也不合理，对应 go 使用 decimal 业界一般都是使用 decimal.Decimal 吧
3. 虽然 int 或 float 有一些计算方法，但是有 sum 没有 avg 、max 、min 一类常用的计算方法也是值得商榷的

@qqxhb 抱歉，我没有仔细阅读代码，avg 等几个参数在 expr 内嵌，但是其他几个点我认为还是值得考虑

[shuqingzai]

UPDATE `user_wallet` SET `frozenBalance` = `frozenBalance` + 10.05 WHERE userId = 311 AND balance > `frozenBalance` + 10.05;

@qqxhb 请问按目前的功能实现，我该如何构建下面的 SQL ??

UPDATE `user_wallet` SET `frozenBalance` = `frozenBalance` + 10.05 WHERE userId = 311 AND balance > `frozenBalance` + 10.05;

[qqxhb]

model上的类型用decimal.Decimal就好
只是改dal上的类型

自定义Filed是否要加这些方法，我们讨论下

[mimicode]

not in 也没有，既然是通用类型的 filed.Field 没有确定类型,应该加上相关方法，有开发者自己决定 用还是不用。不能因为可能会出现误用 而禁止。如果有人认为这样不安全，你可以用更准确的类型，比如field.Int.

[Catlickfish]

@shuqingzai 请问你解决这个问题了吗？能否展示下示例代码呢

[shuqingzai]

@shuqingzai 请问你解决这个问题了吗？能否展示下示例代码呢

@Catlickfish
fork 仓库，自己加方法呗，然后 replace go.mod 的仓库指向