goctopus / silk

yet another golang orm
MIT License
4 stars 4 forks source link

Collection: modify collection test & splice #51

Closed chenhg5 closed 5 years ago

chenhg5 commented 5 years ago

修改 Splice 方法,感觉第三个参数没必要去掉了,原 splice 方法会修改调用collection的值,感觉这样设计不太好。现在修改成深拷贝。

chenhg5 commented 5 years ago

原方法我返回的删除后的切片,这里也错了 想问个问题,n := c.value ,如果 n 的 cap 没有变化的话就是在对 c.value 进行操作吗? 另外我又看了下,splice() 方法应该是返回删除的切片,同时原切片也进行了删除操作

嗯嗯,我没看懂,他说的是immutable的这里

但是 splice这个方法又改了原来的值,这设计感觉有点问题。现在暂时想不出什么场景,我觉得还是按照不修改原来值去做,后面有特定需求再改。

brucegao77 commented 5 years ago

好的