Следующий фрагмент кода неправильно проверяет странность:
public static boolean isOdd(int i) {
return i % 2 == 1;
}
Я прочитал в Интернете, что я должен сделать это следующим образом:
public static boolean isOdd(int i) {
return i % 2 != 0;
}
Почему это?
Возможно, потому что (i % 2) != 0 работает как с положительными, так и с отрицательными числами.
() избыточен, как говорит приоритет оператора Я имею в виду, что вы можете игнорировать его
- person Grijesh Chauhan; 17.05.2013
() предназначен только для приоритета операций перезаписи и зачем это помнить :)
- person Grijesh Chauhan; 17.05.2013
Потому что когда i отрицательное --> (-1) % 2 == -1
Вы должны использовать:
(i & 1) != 0
чтобы избежать проблем со знаком.
Также обратите внимание, что использование & гарантирует, что каким бы глупым ни был компилятор, он никогда не попытается использовать деление для выполнения операции %.
i % 1 всегда 0
- person Stochastically; 17.05.2013
i & 1.
- person OldCurmudgeon; 17.05.2013
(i & 1) != 0 и (i % 2) != 0. #include <stdio.h> int main(void) { volatile int i = 1; volatile int j = (i & 1) != 0; printf("%d", j); return j; } (и то же самое с i%2 вместо i&1). cc -S и diff -u говорят, что единственное различие заключается в директиве .file (потому что я создал по одному файлу для каждого).
- person user; 18.05.2013
-O0, так и с -O3 с одинаковым результатом.
- person user; 20.05.2013
Первый фрагмент будет работать правильно, если оператор по модулю будет заменен оператором побитового и:
public static boolean isOdd(int i) {
return (i & 1) == 1;
}
Это связано с тем, как модуль работает в Java. Если я отрицателен, ответ также будет отрицательным. Каждый отрицательный ввод вернет false.
Поскольку приведенный ниже метод ведет себя правильно при передаче отрицательных, нулевых и положительных значений для каждого числового параметра.
public static boolean isOdd(int i) {
return i % 2 != 0;
}
Если мы воспользуемся описанным выше методом isOdd(), производительность будет не очень хорошей, лучше использовать побитовый AND operator(&) вместо оператора remainder(%).
public static boolean isOdd(int i) {
return (i & 1) != 0;
}
Этот код работает быстрее, чем первый, в зависимости от того, какую платформу и виртуальную машину мы используем.
Примечание. Как правило, операции деления и остатка выполняются медленнее, чем другие арифметические и логические операции.
%— оператор остатка. docs.oracle.com/ javase/specs/jls/se7/html/ - person djeikyb schedule 18.05.2013