Co jest źle?

Question

Gdzie jest błąd w zapisie tego if-a?

If ((lpc(int n)) and (x%n==0))

Answer 1

A dlaczego uważasz, że jest błąd? Nie kompiluje Ci się, czy program działa w sposób nieoczekiwany? Na przyszłość daj więcej informacji, dla ludzi chcących Ci pomóc każdy szczegół jest ważny. :)

Jeśli język to C++ (taki jest w tagach), "If" powinien być rozpoczynany małą literą, a spójniki logiczne - chociaż wersje słowne również są dopuszczalne - zwykło się zapisywać symbolicznie, && dla and oraz || dla or.

Wywołanie funkcji raczej nie wiąże się z definicją nowej zmiennej dla jej parametru, więc do lpc(int n) powinieneś wysłać już konkretną wartość n, zdefiniowaną wcześniej, więc jej wywołanie powinno wyglądać tak: lpc(n).

Nawias, otaczający w warunku wywołanie lpc() jest w tym wypadku niepotrzebny i tylko pogarsza czytelność, drugi nawias może zostać. :)

Poprawiona wersja wygląda tak:

if(lpc(n) && (x%n == 0))

Answer 2

zamiast "and" użyj && :)

if ((lpc(int n)&&(x%n==0))

 

Comments

Niestety nie tu tkwi problem

---

W takim razie to nie wina ifa - rozumiem, że lpc(int n) to funkcja? Tutaj byłby kolejny problem(podajesz argument formalny, a tego nie powinno się robić)

Answer 3

wydaje mi się, że operatory logiczne w warunkach powinny wyglądać tak:

http://guidecpp.cal.pl/cplus,operators-logic