Sonar-Verletzung

Question

public Integer getMaxCount(String id,String type) { 
    String maxCount ="SELECT MAX(ID) AS COUNT FROM STUDENT"; 
    try { 
     return queryForObject(
      maxCount, 
      new Object[]{id,type}, 
      (rs, i) -> {return rs.getInt(COUNT);}, 
      "maxCount" 
     ); 
    } catch (EmptyResultDataAccessException ex) { 
     LOG.error(ex); 
     return 9999; 
    } 
} 

Unter Sonarverletzungen für oben genannten Block des Codes, wie zu beheben.

Lambdas sollte mit Verfahren ersetzt werden, verweist

Lambdas nur eine Anweisung sollte nicht verschachtelt diese Aussage in einem Block

Answer 1

Sie enthalten nicht wirklich hier brauchen blockieren. Sie können den Wert mit dieser zurück:

(rs, i) -> rs.getInt(COUNT) 

Answer 2

Das sind:

• S1612 "Lambdas should be replaced with method references"
• S1602 "Lamdbas containing only one statement should not nest this statement in a block"

Beide Seiten umfassen beispielsweise, wie Sie Ihren Code zu beheben.

x -> System.out.println(x+1) 
(a, b) -> a+b //For return statement, the return keyword should also be dropped