Ein Interviewthema: Was kann mit diesem Code schief gehen?

Question

Eine einfache Calculator-Klasse, die in einer Web-Anwendung als Spring-Bean verdrahtet ist:

public class Calculator { 
    int result; 

    public int addTwoNumbers(int first, int second) { 
     result = first + second; 
     return result; 
    } 
} 

Was möglicherweise kann mit diesem schief gehen?

Answer 1

Multi Threading Probleme. Standardmäßig sind Spring Beans Singletons.

Answer 2

Integer Überlauf ist ein weiteres Problem, wenn dies zählt.

es zu lösen, Sie long stattdessen zurückgeben kann:

long result; 
public long addTwoNumbers(int first, int second) 
{ 
    result = (long)first + second; 
    return result; 
} 

Seien Sie vorsichtig mit nur result = first + second; sagen, wie die beiden int ‚s hinzugefügt werden, so dass ein int (was überlaufen kann), würde das nur dann in eine long umgewandelt werden.

result = (long)first + second; entspricht result = ((long)first) + second; (nicht result = (long)(first + second);), wir eines der int s zu einem long ersten konvertieren Sinn, so wäre es long + int sein, die ein long gibt.

Answer 3

Diese Klasse ist nicht Thread sicher, wenn man davon ausgeht, dass es sich um eine Singleton-Springbohne handelt. Wenn zwei Threads addTwoNumbers aufrufen, besteht die Möglichkeit, dass der erste Thread das Ergebnis der Berechnung des zweiten Threads erhält, sodass wir 2 + 2 = 5 erhalten können. Wenn das Ergebnisfeld nirgendwo anders verwendet werden soll, sollte es entfernt werden

public class Calculator { 
     public int addTwoNumbers(int first, int second) { 
     return first + second; 
    } 
}