gen-class without a -main doesn't cause that error

that is the kind of error you get when you requiring the namespace foo.bar but the file that lives at foo/bar.clj has the namespace foo.baz

ah, no, not that either, it really looks like a syntax error in your file

Clojure 1.11.1
user=> instant.scripts.analytics
Syntax error (ClassNotFoundException) compiling at (REPL:0:0).
instant.scripts.analytics
user=> 

so I would check to make sure the source code you are editing is actually what is being loaded (you aren't working on a different second checkout, don't have another file with the same name on the classpath, etc)

Full disclosure: I've done programming in other functional languages such as Elm and OCaml.

Nice and simple, I like it. I've been learning Clojure slowly for a couple of months now, so I can't provide much feedback, sorry. I remember that I overcomplicated my solution by using 'split-lines,' which gave me empty strings. Then, I had to use 'partition-by blank?' to group the values. So... yeah, I prefer your approach.

day 1 is often a straightforward enough problem that there isn't a ton of nuance to critique - I happened to be focusing on generalizing in the 2022 AOC, so not necessarily an improvement, but a thing that might be interesting to play with, is to solve both with one function that takes a number of elves

i guess it would help to have the question and expected answer (test) in the example code. i have no idea what is going on

(defn parse-input [s]
  (->>
    (str/split s #"\n\n")
    (map #(->> (str/split % #"\n") (map parse-long) (reduce +)))))

if my map function is complicated, i like to give it a name (defn). it's easier to tag with a debugger, and it's easier to quickly right some tests against it (when a map -> map fails then you have to write more code to debug/test)

(defn parse-input [s]
  (->>
    (str/split s #"\n\n")
    (map #(->> (str/split % #"\n")
               (map parse-long)
               (reduce +)))))

if parse-input always returns a single number (wrapped in a vector for some reason), why is the code for part-1 and part-2 so complicated?

when i run your sample data on part-1 and part-2 i get the same answer, is that supposed to happen?

(defn parse-input [s]
  (->>
    (str/split s #"\n+")
    (map parse-long)
    (reduce +)))

#_(parse-input sample-input)

(defn part-1 [input]
  (parse-input input))

(defn part-2 [input]
  (parse-input input))

(part-1 sample-input)
(part-2 sample-input)

@U0LAJQLQ1 your implementations of part1 and part2 above is not correct. The first part should return the biggest number. The second part should return the sum of three biggest numbers. Not sure how you end up with the same number. I’ve tested my code multiple times.

I do appreciate the feedback regarding breaking out into a separate function to ease debugging. I can also see how reducing within parse-input is weird. I did that to save repeating myself in part-1 and part-2, but it should most likely just be repeated.

checkout #adventofcode for comparative solutions. Each day has a thread pinned, here’s https://clojurians.slack.com/archives/C0GLTDB2T/p1669870809772339

Your code is nice! Have a look at http://clojuredocs.org for more regex based functions. They typically start with re- I'm half sure you can do more in fewer steps (computations). It's sometimes interesting to do the same thing in different ways.

@U05JP9PDU75 how am i supposed to know what these function are supposed to do? when posting code for other people to look at it really matters that you do some work to help them understand the goal. i ran these 2 functions with your sample-data and they produced the same output, so i changed my code as if those were the tests that needed to be passed.

@U0LAJQLQ1 they don’t produce the same output for the sample data either, so I’m not sure what you did wrong. :man-shrugging:

But I’ll make sure to make the expectations of program more clear the next time around.

@U01EFUL1A8M thanks! Will do. 🙂

@U05JP9PDU75 you are running different code than the one you posted in the gist, if that is the case

ok, seems like there is some problem copying and pasting code form the gist

newlines just disappear

that's pretty annoying

Yeah, I was about to say, the new lines are quite important. 😅