создать программу более эффективной

Question

проблемы,

• возврата «а», если число х можно разделить на 3
• возврата «б», если число х, можно разделить на 5
• возврата «AB» если число х можно разделить на 3 и 5

мое решение

if(x mod 3 ==0) return "a" 
else if (x mod 3 == 0) return "b" 
else if (x mod 3 == 0 && x mod 5) return "ab" 

есть ли другое эффективное решение, чем это?

Answer 1

Ифф ((х мод 3 == 0) и (х мод 5 == 0)) < => (х мод 15 == 0)

if (x mod 15 == 0) return "ab"  
else if(x mod 3 == 0) return "a" 
else if (x mod 5 == 0) return "b" 

Answer 2

Это должно быть

if (x mod 3 == 0 && x mod 5 == 0) return "ab"  
else if(x mod 3 ==0) return "a" 
else if (x mod 5 == 0) return "b" 

иначе "аб" не будут возвращены, даже если число делится на оба 3 и 5

Answer 3

String result=""; 
    result=(x%3 == 0 && x%5 == 0 ? "ab" : x%5 == 0 ? "a" : x%3 == 0 ? "b" : "not divisible to any number"); 
    System.out.println(result); 

Answer 4

В качестве альтернативы:

String result = ""; 
if (x mod 3 == 0) result += "a"; 
if (x mod 5 == 0) result += "b"; 

return result; 

Но он имеет верхние строчки конкатенации строк

Answer 5

string result = "" 
if(x mod 3 == 0) 
{ 
    result += "a" 
} 
if(x mod 5 == 0) 
{ 
    result += "b" 
} 

return result; 

Этот подход проверяет по модулю 3 и 5 только один раз.

Answer 6

сэкономить несколько вычислений:

boolean mod3 = x mod 3 == 0 
boolean mod5 = x mod 5 == 0 
if (mod3 && mod5) return "ab" 
if (mod3) return "a" 
if (mod5) return "b" 

Answer 7

незначительно более эффективной ... и, безусловно, более ПРАВИЛЬНО ... решение:

if (x mod 3 == 0) { 
    if (x mod 5 == 0) { 
     return "ab" 
    } else { 
     return "a" 
    } 
} else if (x mod 5 == 0) { 
    return "b"; 
} else { 
    // probably ... 
    return ""; 
} 

Обратите внимание, что разница в производительности, скорее всего ИНСИ gonificant, и вполне возможно, что он равен нулю, если JIT-компилятор достаточно умен. Это относится к всем решениям.

Answer 8

Это классическая FizzBuzz проблема, которую вы всегда должны рассматривать первый случай является общим делителем, а затем продолжить в порядке убывания так:

if (x mod 3 == 0 && x mod 5) return "ab"  
else if(x mod 5 ==0) return "b" 
else if (x mod 3 == 0) return "a"