Tôi có một số phương thức thực hiện một số thay đổi dữ liệu trong cơ sở dữ liệu (chèn, cập nhật và xóa). Các ORM Tôi đang sử dụng giá trị int trở lại hàng bị ảnh hưởng đối với những loại phương pháp. Tôi nên trả lại cái gì cho "phương pháp của tôi", để chỉ ra trạng thái thành công / thất bại của hoạt động?
Hãy xem xét mã đang trả về một int:
A.1
public int myLowerLevelMethod(int id) {
...
int affectedRows = myOrm.deleteById(id)
...
return affectedRows;
}
Sau đó sử dụng:
A.2
public void myOtherMethod() {
...
int affectedRows = myLowerLevelMethod(id)
if(affectedRows > 0) {
// Success
} else {
// Fail
}
}
So sánh với việc sử dụng boolean:
B.1
public boolean myLowerLevelMethod(int id) {
...
int affectedRows = myOrm.deleteById(id)
...
return affectedRows > 0;
}
Sau đó sử dụng:
B.2
public void myOtherMethod() {
...
boolean isSuccess = myLowerLevelMethod(id)
if(isSuccess) {
// Success
} else {
// Fail
}
}
Cái nào (A hoặc B) là tốt hơn? Hoặc ưu / nhược điểm của từng?
Trong "A.2" của bạn. Nếu hàng 0 bị ảnh hưởng, tại sao đó là thất bại nếu hàng 0 cần phải bị ảnh hưởng? Nói cách khác, nếu không có lỗi cơ sở dữ liệu, tại sao nó lại thất bại?
—
Jaydee
Có sự khác biệt về ngữ nghĩa giữa "không thành công" và "hàng không bị ảnh hưởng" không? Ví dụ: khi xóa tất cả các đơn đặt hàng của khách hàng, có một sự khác biệt giữa "khách hàng không tồn tại" và "khách hàng không có đơn hàng".
—
Cephalepad
Bạn có xem xét xóa với hàng không bất ngờ? Trong trường hợp đó ném.
—
usr
@Arian Tôi nghĩ đó là câu hỏi thực sự cho tôi. Tôi nghĩ rằng tôi đã chọn B, vì với A, mã của tôi hiện chứa kiểm tra 0 ở một số nơi và cho -1 ở những nơi khác
—
Hoàng Trần