Temat rozmowy kwalifikacyjnej: Co może pójść nie tak z tym kodem?

Question

Klasa prosty kalkulator, który jest podłączony do góry jak sprężyna fasoli w aplikacji internetowej:

public class Calculator { 
    int result; 

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

co potencjalnie może się nie udać z tym?

Answer 1

Problemy z wieloma wątkami. Domyślnie fasola szparagowa to single.

Answer 2

Integer overflow to kolejny problem, jeśli się to liczy.

Aby go rozwiązać, można powrócić long zamiast:

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

Uważaj tylko, że result = first + second;, jak dwa int „s zostaną dodane, dając int (co może przepełnić), co tylko będzie następnie przekonwertować na long.

result = (long)first + second; jest równoważna result = ((long)first) + second; (nie result = (long)(first + second);), co oznacza, możemy przekształcić jedną z int s Do long pierwszy, więc byłoby long + int, który daje long.

Answer 3

Ta klasa nie jest bezpieczna dla wątków, zakładając, że jest to sprężynowa pojedyncza fasola. Jeśli dwa wątki dodadzą numery, istnieje szansa, że ​​pierwszy wątek otrzyma wynik obliczenia drugiego wątku, więc możemy uzyskać 2 + 2 = 5. Jeśli pole wynikowe nie powinno być używane nigdzie indziej, powinno się go pozbyć

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