SonarQube: (Zamiast tego złap listę określonych podtypów wyjątków)

Question

Mam pytanie dotyczące ogólnych wyjątków. Skąd mamy wiedzieć, którego nieogólnego wyjątku użyć, gdy próbujesz zrobić wiele rzeczy.

Na przykład:

  @PostConstruct
    protected void init() {
        try {
            HttpSession session = request.getSession();
            String policyInfo = (String) session.getAttribute("policyInfo");
            if(session.getAttribute("faxNumber") != null) {
                faxNumber = (String) session.getAttribute("faxNumber");
            }
            policyNumber = (String) session.getAttribute("policyNumber");
            JSONObject policyInfoObj = new JSONObject(policyInfo);
            JSONArray policiesArr = policyInfoObj.getJSONArray("policies");
            if (policiesArr.length() > 0) {
                JSONObject policyObj = policiesArr.getJSONObject(0);
                JSONArray insuredVehicle = policyObj.getJSONArray("insuredVehicle");
                checkInsuredVechile(insuredVehicle);
                termStartDate = policyObj.getString("effectiveDate");
                JSONArray addressArray = policyObj.getJSONArray("address");
                policySource = policyObj.getString("policySource");
                checkAddressArry(addressArray);

            }
            
             
             
            policyNumber = policyNumber.substring(0,5)+"-"+policyNumber.substring(5,7)+"-"+policyNumber.substring(7);
            
            response.setHeader("Cache-Control", "no-store, no-cache, must-revalidate");
        }catch(Exception  e) {
            logger.error("Exception in getting policy details",e);
        }
    }

Więc za catch(Exception e) {to będzie potrzebował nierodzajową wyjątek, ale mam kłopoty, aby określić, co to może być.

Answer 1

Powinieneś złapać tylko określone wyjątki, takie jak:

catch(org.json.JsonException e)

a nie klasą bazową Exception, co oznacza wszystkie możliwe sprawdzone i niezaznaczone wyjątki