Skip to content

Change Execute Result #9

@fatihtatoglu

Description

@fatihtatoglu

In the first version, the implementation was completed by returning the sql.Result pointer in the Execute method. But it creates some problems with getting RowsAffected results.

To make the usage more simple, it should be changed.

cmd,_ := CreateNewDBCommand(conn)

res,err := cmd.Execute("UPDATE `users` SET `isEnabled`=1 WHERE `userid`=?", 123)
if err != nil{
  fmt.Println(err)
  return
}

// Current Usage
affectedRow,err := (*res).RowsAffected()
if err != nil{
  fmt.Println(err)
} else {
  fmt.Println(affectedRow)
}

lastInsertedId,err := (*res).LastInsertId()
if err != nil{
  fmt.Println(err)
} else {
  fmt.Println(lastInsertedId)
}

// Suggesting Usage
fmt.Println(res.RowsAffected())
fmt.Println(res.LastInsertId())
```

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions