更新不存在的条目的 Golang 数据库操作

huangapple
go评论129阅读模式
英文:

golang database updating a non-existing entry

问题

我有一个用于更新数据库条目的函数，代码如下所示。逻辑是，更新一个不存在的条目会返回错误。不知道是否有更好的方法来处理错误？谢谢。

  1. func (adb *AppDB) UpdateTicket(t Ticket) error {
  2.     result, err := adb.db.NamedExec(`UPDATE ticket SET detail=:detail, start_time=:start_time, end_time=:end_time, priority=:priority WHERE id=:id;`, &t)
  3.     if err != nil {
  4.         return err
  5.     }
  6.     nRows, err := result.RowsAffected()
  7.     if err != nil {
  8.         return err
  9.     }
  10.     if nRows == 0 {
  11.         return fmt.Errorf("Ticket:%s does not exist for update", t)
  12.     }
  13.     return nil
  14. }

我有一个函数用于更新数据库条目。逻辑是，如果更新一个不存在的条目，会返回错误。我觉得代码在处理错误时有点冗长。是否有更好的方法来处理这个问题？谢谢。

英文:

I have a function to update a database entry as follows. The logic is that updating a non-exisitng entry returns error. Somehow I find the code a bit verbose on error handling. Is there a better way to do this? Thanks.

  1. func (adb *AppDB) UpdateTicket(t Ticket) error {                                   
  2.     result, err := adb.db.NamedExec(`UPDATE ticket SET detail=:detail, start_time=:start_time, end_time=:end_time, priority=:priority WHERE id=:id;`, &t) 
  3.     if err != nil {                                                                
  4.         return err                                                                 
  5.     }                                                                              
  6.     nRows, err := result.RowsAffected()                                            
  7.     if err != nil {                                                                
  8.         return err                                                                 
  9.     }                                                                              
  10.     if nRows == 0 {                                                                
  11.         return fmt.Errorf("Ticket:%s does not exist for update", t)                
  12.     }                                                                              
  13.     return nil                                                                     
  14. }

答案1

得分: 0

如果你想让代码更简洁易于维护，我强烈推荐使用 ORM。你可以尝试使用 https://github.com/jinzhu/gorm。

对于许多场景来说，命名返回可能会减少冗余，但这并不是最佳实践。

  1. func (adb *AppDB) UpdateTicket(t Ticket) (err error) {
  2.     var result ResultType
  3.     var nRows int
  4.     if result, err = adb.db.NamedExec(`UPDATE ticket SET detail=:detail, start_time=:start_time, end_time=:end_time, priority=:priority WHERE id=:id;`, &t); err != nil {
  5.         return
  6.     }
  7.     if nRows, err = result.RowsAffected(); err != nil {
  8.         return
  9.     }
  10.     if nRows == 0 {
  11.         err = fmt.Errorf("Ticket:%s does not exist for update", t)
  12.     }
  13.     return
  14. }
英文:

If you need to make it less verbose and easy to maintain I would highly recomend using ORM. You may try https://github.com/jinzhu/gorm

Named return might make it less verbose, but this is not a best practise for lot of scenarios

  1. func (adb *AppDB) UpdateTicket(t Ticket) (err error) {
  2. 	var result ResultType
  3. 	var nRows int
  4. 	if result, err = adb.db.NamedExec(`UPDATE ticket SET detail=:detail, start_time=:start_time, end_time=:end_time, priority=:priority WHERE id=:id;`, &t); err != nil {
  5. 		return
  6. 	}
  7. 	if nRows, err = result.RowsAffected(); err != nil {
  8. 		return
  9. 	}
  10. 	if nRows == 0 {
  11. 		err = fmt.Errorf("Ticket:%s does not exist for update", t)
  12. 	}
  13. 	return
  14. }

huangapple
  • 本文由 发表于 2016年12月27日 11:39:27
  • 转载请务必保留本文链接：https://go.coder-hub.com/41338715.html
  • database
  • error-handling
  • go
  • postgresql
匿名

发表评论

匿名网友

:?: :razz: :sad: :evil: :!: :smile: :oops: :grin: :eek: :shock: :???: :cool: :lol: :mad: :twisted: :roll: :wink: :idea: :arrow: :neutral: :cry: :mrgreen:

确定