Code example for leap-year incorrect about curriculum HOT 6 CLOSED

The code as written in flow_control.md works for me:

(defn leap-year?
  "Every four years, except years divisible by 100, but yes for years divisible by 400."
  [year]
  (and (zero? (mod year 4))
       (or (zero? (mod year 400))
           (not (zero? (mod year 100))))))

(leap-year? 1900)  ; false
(leap-year? 2000)  ; true

What were you seeing?

from curriculum.

I switched lines 5 and 6 and then it worked, before it was returning true,
true

On Mon, Dec 22, 2014 at 1:55 PM, David Chambers [email protected]
wrote:

The code as written in flow_control.md
https://github.com/ClojureBridge/curriculum/blob/master/outline/flow_control.md
works for me:

(defn leap-year?
"Every four years, except years divisible by 100, but yes for years divisible by 400."
[year](and %28zero? %28mod year 4%29%29
%28or %28zero? %28mod year 400%29%29
%28not %28zero? %28mod year 100%29%29%29%29))

(leap-year? 1900) ; false
(leap-year? 2000) ; true

What were you seeing?

—
Reply to this email directly or view it on GitHub
#122 (comment)
.

from curriculum.

Could you try copying the snippet from my comment above and pasting it into the instarepl? Perhaps one of your parens was in the wrong place.

from curriculum.

that is weird it does work!

On Mon, Dec 22, 2014 at 2:10 PM, David Chambers [email protected]
wrote:

Could you try copying the snippet from my comment above and pasting it
into the instarepl? Perhaps one of your parens was in the wrong place.

—
Reply to this email directly or view it on GitHub
#122 (comment)
.

from curriculum.

maybe it was light table, sorry

On Mon, Dec 22, 2014 at 2:19 PM, Shakrah Yves [email protected] wrote:

that is weird it does work!

On Mon, Dec 22, 2014 at 2:10 PM, David Chambers [email protected]
wrote:

Could you try copying the snippet from my comment above and pasting it
into the instarepl? Perhaps one of your parens was in the wrong place.

—
Reply to this email directly or view it on GitHub
#122 (comment)
.

from curriculum.

No worries. :)

from curriculum.