thảo luận Best practice cho việc throw exception?

GeniVN đã nói:

ý em là

Java:

class UserService {
    public User getUserInfo(int id) {
        // logic
        if (...) {
             throw UserNotFoundException("user not found");
        }

    }
}

Java:

class UserHttpController {
    private UserService userService;

    @GetMapping("/users/{id}")
    public Response<User> getUser(@RequestParam int id) {
 try {
    User user = userService.getUser(id);
    return Response(Http.Ok, ...);
 } catch (UserNotFound e) {
    return Response(Http.NotFound, ...);

 } catch (Exception e) {
    return Response(Http.Internal, ...);
   }
}

}

hay là các bác dùng kiểu gì khác kiểu này để tối ưu code hơn vậy

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

exception phải là cái cực kì bất ngờ, như connection vào db đang chạy ngon lành bỗng dưng bị đứt bóng vì server db sập thì cái đó mới là exception, còn id ko tồn tại trong bảng mà quăng exception thì backend dev gì ngu vậy

ngôn ngữ có exception mà xài vô tội vạ thì là dev ocschos hết, tránh xa cái fw đó ra

có lẽ nó muốn code nó đẹp, function signature đẹp, ví dụ User getUser(id) chứ đéo muốn xấu xí như expected<User> getUser(id) hay tuple<User, ErrorCode> getUser(id) nên xài exception vô tội vạ, thì về lớp 1 dạy vở sạch chữ đẹp đi chứ làm công việc logic làm gì mà thích vở sạch chữ đẹp hơn logic

Nhấp để mở rộng...

DancingBoy đã nói:

tại sao thrown ex nó lại tốn tới 1000 cycle vậy thím, cái này hay nè, code thì đẹp hơn if/else mà performance thì tệ thế nhỉ ?

Có lẽ giờ CPU quá mạnh rồi, nên dev ko quan tâm vụ này lắm, trừ khi nó quá chậm.

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

exception phải là cái cực kì bất ngờ, như connection vào db đang chạy ngon lành bỗng dưng bị đứt bóng vì server db sập thì cái đó mới là exception, còn id ko tồn tại trong bảng mà quăng exception thì backend dev gì ngu vậy

ngôn ngữ có exception mà xài vô tội vạ thì là dev ocschos hết, tránh xa cái fw đó ra

có lẽ nó muốn code nó đẹp, function signature đẹp, ví dụ User getUser(id) chứ đéo muốn xấu xí như expected<User> getUser(id) hay tuple<User, ErrorCode> getUser(id) nên xài exception vô tội vạ, thì về lớp 1 dạy vở sạch chữ đẹp đi chứ làm công việc logic làm gì mà thích vở sạch chữ đẹp hơn logic

Nhấp để mở rộng...

The_Hermit đã nói:

Không nên lạm dụng exception thì đúng chứ bảo chỉ xài cho thứ cực bất ngờ thôi thì nghe hài hài.
Đầy FW nổi tiếng nó vẫn throw exception như là một constraint ràng buột các layer bên ngòai phải handle. Anh return error thì nó có thể có hoặc có thể không handle.
Ví dụ đọc file mà file ko tồn tại thì có được gọi là cực bất ngờ không ?

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

đương nhiên là ko bất ngờ. Nó là lỗi liên quan trực tiếp tới việc open file thì sao gọi là bất ngờ được??
thằng Java nó throw thì nó là ngôn ngữ ngu học cho mấy thằng ngu kiếm tiền chứ có phải ngôn ngữ được ca tụng là logic đâu
thằng java thì chúa hack rồi. Generic đéo có nên nó hack treo đầu dê generic thực chất chỉ là thịt chó đa hình nên chậm vl, vd chỉ có thằng ngu mới xài ArrayList<Int>. Đéo có lambda expression nên cũng phải hack làm cái Runnable interface. Đéo có keyword nodiscard nên chắc cũng phải hack kiểu ép xử lý lỗi bằng ngoại lệ. Thay vì update language thêm feature thì sợ fuk tạp ko dám add, nên toàn xài hack ko chứ logic mẹ gì. Sau này cũng phải ăn lại cứt mình ỉa ra thêm mấy feature mới vào rồi.

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

đương nhiên là ko bất ngờ. Nó là lỗi liên quan trực tiếp tới việc open file thì sao gọi là bất ngờ được??
thằng Java nó throw thì nó là ngôn ngữ ngu học cho mấy thằng ngu kiếm tiền chứ có phải ngôn ngữ được ca tụng là logic đâu
thằng java thì chúa hack rồi. Generic đéo có nên nó hack treo đầu dê generic thực chất chỉ là thịt chó đa hình nên chậm vl, vd chỉ có thằng ngu mới xài ArrayList<Int>. Đéo có lambda expression nên cũng phải hack làm cái Runnable interface. Đéo có keyword nodiscard nên chắc cũng phải hack kiểu ép xử lý lỗi bằng ngoại lệ. Thay vì update language thêm feature thì sợ fuk tạp ko dám add, nên toàn xài hack ko chứ logic mẹ gì. Sau này cũng phải ăn lại cứt mình ỉa ra thêm mấy feature mới vào rồi.

Nhấp để mở rộng...

bibooboo đã nói:

Vậy thím xử lý exception sao mà ko if/else khi có nhiều loại exception
If/else nhiều là chuyện quá bình thường, chỉ cần tập trung và return sớm để tránh code branching là đc

via theNEXTvoz for iPhone

Nhấp để mở rộng...

GeniVN đã nói:

ý em là

Java:

class UserService {
    public User getUserInfo(int id) {
        // logic
        if (...) {
             throw UserNotFoundException("user not found");
        }

    }
}

Java:

class UserHttpController {
    private UserService userService;

    @GetMapping("/users/{id}")
    public Response<User> getUser(@RequestParam int id) {
 try {
    User user = userService.getUser(id);
    return Response(Http.Ok, ...);
 } catch (UserNotFound e) {
    return Response(Http.NotFound, ...);

 } catch (Exception e) {
    return Response(Http.Internal, ...);
   }
}

}

hay là các bác dùng kiểu gì khác kiểu này để tối ưu code hơn vậy

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

đương nhiên là ko bất ngờ. Nó là lỗi liên quan trực tiếp tới việc open file thì sao gọi là bất ngờ được??
thằng Java nó throw thì nó là ngôn ngữ ngu học cho mấy thằng ngu kiếm tiền chứ có phải ngôn ngữ được ca tụng là logic đâu
thằng java thì chúa hack rồi. Generic đéo có nên nó hack treo đầu dê generic thực chất chỉ là thịt chó đa hình nên chậm vl, vd chỉ có thằng ngu mới xài ArrayList<Int>. Đéo có lambda expression nên cũng phải hack làm cái Runnable interface. Đéo có keyword nodiscard nên chắc cũng phải hack kiểu ép xử lý lỗi bằng ngoại lệ. Thay vì update language thêm feature thì sợ fuk tạp ko dám add, nên toàn xài hack ko chứ logic mẹ gì. Sau này cũng phải ăn lại cứt mình ỉa ra thêm mấy feature mới vào rồi.

Nhấp để mở rộng...

perepermin đã nói:

spring có cái exception handling mà?

Nhấp để mở rộng...

Bleach & Sarcasm đã nói:

đương nhiên là ko bất ngờ. Nó là lỗi liên quan trực tiếp tới việc open file thì sao gọi là bất ngờ được??
thằng Java nó throw thì nó là ngôn ngữ ngu học cho mấy thằng ngu kiếm tiền chứ có phải ngôn ngữ được ca tụng là logic đâu
thằng java thì chúa hack rồi. Generic đéo có nên nó hack treo đầu dê generic thực chất chỉ là thịt chó đa hình nên chậm vl, vd chỉ có thằng ngu mới xài ArrayList<Int>. Đéo có lambda expression nên cũng phải hack làm cái Runnable interface. Đéo có keyword nodiscard nên chắc cũng phải hack kiểu ép xử lý lỗi bằng ngoại lệ. Thay vì update language thêm feature thì sợ fuk tạp ko dám add, nên toàn xài hack ko chứ logic mẹ gì. Sau này cũng phải ăn lại cứt mình ỉa ra thêm mấy feature mới vào rồi.

Nhấp để mở rộng...

perepermin đã nói:

nói nghe ngu bỏ mẹ, ignore thôi.
á đù âm react

Nhấp để mở rộng...

GeniVN đã nói:

đại ý là dùng exception để xử lý business error thôi ạ

Nhấp để mở rộng...