Ném và bắt ngoại lệ trong cùng một chức năng / phương pháp

Tôi đã viết một hàm yêu cầu người dùng nhập liệu cho đến khi người dùng nhập số nguyên dương (số tự nhiên). Ai đó nói rằng tôi không nên ném và bắt ngoại lệ trong chức năng của mình và nên để người gọi chức năng của tôi xử lý chúng.

Tôi tự hỏi những nhà phát triển khác nghĩ gì về điều này. Tôi cũng có thể lạm dụng các ngoại lệ trong chức năng. Đây là mã trong Java:

private static int sideInput()
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                // probably a misuse of exceptions
                throw new NumberFormatException();
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

Tôi quan tâm đến hai điều:

1. Tôi có nên để người gọi lo lắng về ngoại lệ? Điểm của chức năng là nó cằn nhằn người dùng cho đến khi người dùng nhập số tự nhiên. Là điểm của chức năng xấu? Tôi không nói về UI (người dùng không thể thoát khỏi vòng lặp mà không có đầu vào thích hợp), nhưng về đầu vào bị lặp với các ngoại lệ được xử lý.
2. Bạn có thể nói tuyên bố ném (trong trường hợp này) là lạm dụng các ngoại lệ? Tôi có thể dễ dàng tạo một cờ để kiểm tra tính hợp lệ của số và xuất thông báo cảnh báo dựa trên cờ đó. Nhưng điều đó sẽ thêm nhiều dòng vào mã và tôi nghĩ nó hoàn toàn có thể đọc được.

Có điều là tôi thường viết một hàm đầu vào riêng biệt. Nếu người dùng phải nhập một số nhiều lần, tôi tạo một hàm riêng cho đầu vào xử lý tất cả các ngoại lệ và giới hạn định dạng.

Điểm của một trường hợp ngoại lệ là nó cho phép một phương thức cho biết người gọi đó đã vào trạng thái không thể tiếp tục bình thường mà không buộc bạn phải nhúng mã lỗi vào giá trị trả về.

Trong trường hợp của bạn, phương thức của bạn biết chính xác phải làm gì khi đầu vào không lớn hơn 0. Lý do duy nhất bạn lưu dòng ở đây là do bạn tình cờ ném ngoại lệ giống như bạn sẽ nhận được nếu đầu vào không phải là số. Tuy nhiên, ngoại lệ bạn đang ném không thể hiện đúng lý do tại sao mã của bạn không thích đầu vào. Nếu có người khác đi cùng và xem mã này, họ sẽ phải dành thêm thời gian để cố gắng xem chính xác mọi thứ đang hoạt động như thế nào.

Đây là một sử dụng xấu của ngoại lệ. Để bắt đầu, một số không tích cực không phải là một ngoại lệ định dạng.

Tại sao sử dụng ngoại lệ ở tất cả? Nếu bạn biết đầu vào nào không được phép, chỉ cần không thoát ra khỏi vòng lặp cho đến khi bạn nhận được đầu vào hợp lệ từ người dùng, đại loại như sau:

while (true)
{
   // Get user input.
   String input = scanner.nextLine();

   try
   {
      side = Integer.parseInt(input);

      break;
   }
   catch (NumberFormatException ex)
   {
      // Inform user of invalid input.
      System.out.println("Invalid input. Enter a natural number.");
   }
}

Chỉ bắt ngoại lệ nếu bạn định làm điều gì đó có liên quan đến lệnh gọi phương thức hiện tại; tức là dọn dẹp, logic thất bại, v.v.

Người ta có thể thoát khỏi thử / bắt ở đây và chỉ cần ghi lại cuộc gọi phương thức:

//Throws NumberFormatException if read input is less than 0
private static int sideInput()

Người ta vẫn cần xử lý ngoại lệ đó hơn nữa trong chuỗi cuộc gọi / ngăn xếp!

Bạn không nên ném và bắt cùng một ngoại lệ trong một phương thức, tôi thậm chí nghĩ rằng khối bắt sẽ bắt được ngoại lệ giống như bạn đang ném, vì vậy bạn không thực sự ném nó.

Nếu parseIntthành công, thì đó không phải là a NumberFormatException.

nếu bên nhỏ hơn 0, bạn nên ném a NegativeSideLengthException;

Tạo một ngoại lệ tùy chỉnh / kinh doanh có tên NegativeSideLengthException

public class NegativeSideLengthException extends Exception
{


    public NegativeSideLengthException(Integer i)
    {
        super("Invalid negative side length "+i);        
    }

}

Sau đó sideInputném NegativeSideL wavelException

private static int sideInput() throws NegativeSideLengthException
{
    int side = 0;
    String input;
    Scanner scanner = new Scanner(System.in);

    do {
        System.out.print("Side length: ");
        input = scanner.nextLine();
        try {
            side = Integer.parseInt(input);
            if (side <= 0) {
                throw new NegativeSideLengthException(side);
            }
        }
        catch (NumberFormatException numFormExc) {
            System.out.println("Invalid input. Enter a natural number.");
        }
    } while (side <= 0);

    return side;
}

Bạn thậm chí có thể (nếu bạn muốn) thêm một khối bắt khác để bắt NegativeSideLengthExceptionvà không có phương thức ném nó.

do {
    System.out.print("Side length: ");
    input = scanner.nextLine();
    try {
        side = Integer.parseInt(input);
        if (side <= 0) {
            throw new NegativeSideLengthException(side);
        }
    }
    catch (NumberFormatException numFormExc) {
        System.out.println("Invalid input. Enter a natural number.");
    } catch (NegativeSideLengthException e){
        System.out.println("Invalid input. Enter a non-negative number.");
    }
} while (side <= 0);

Cờ không phải là một cách tốt để xử lý các trường hợp ngoại lệ.

Ngoại lệ là những thứ khá ác mộng, chúng mang lại nhiều phức tạp hơn những gì chúng giải quyết.

Thứ nhất, nếu bạn không nắm bắt được ngoại lệ của mình, người gọi chỉ có thể thực hiện on error resume next, nghĩa là sau một tuần, thậm chí bạn sẽ không biết chức năng của mình có thể ném gì và phải làm gì với nó:

{
    ...
}
catch(OutOfMemory, CorruptedMemory, BadData, DanglingPointers, UnfinishedCommit)
{
    Console.WriteLine("Nothing to see here, move on.");
    Console.WriteLine("The app is very stable, see, no crashing!");
}

Về cơ bản, nếu bạn bắt được chúng, bạn phải hiểu rất rõ về hợp đồng và đảm bảo ngoại lệ. Điều đó hiếm khi xảy ra trong một thế giới thực. Và mã của bạn cũng sẽ khó đọc.

Ngoài ra, điều buồn cười là nếu bạn thực sự có bất kỳ cơ hội xử lý ngoại lệ nào, bạn cần ngôn ngữ RAII thực sự, đó là loại hài hước, vì Java và .NET là tất cả về ngoại lệ ...

Lặp lại một lần nữa, nhưng ...:

http://bloss.msdn.com/b/oldnewthing/archive/2004/04/22/118161.aspx

http://bloss.msdn.com/b/oldnewthing/archive/2005/01/14/352949.aspx

http://www.joelonsoftware.com/items/2003/10/13.html