>you have `username` a symbol as the key in your map, but you are >looking for `:username` they keyword as a key
At some point I tried: (get new-user-entry "username") but that had the same result. Username is clearly a key, yes? What would you suggest? On Friday, October 19, 2012 12:06:10 PM UTC-4, red...@gmail.com wrote: > > you have `username` a symbol as the key in your map, but you are > looking for `:username` they keyword as a key > > On Fri, Oct 19, 2012 at 8:47 AM, larry google groups > <lawrenc...@gmail.com <javascript:>> wrote: > > > > > > Also: > > > > > > > >> I suggesting adding printlns or logging or a debugger and checking the > >> value of this-users-params, it is almost certainly not what you expect > >> it to be. > > > > I showed the output previously, but I will do it again. > > > > If I add a println like this: > > > > > > (defn add-to-logged-in-registry [this-users-params] > > (let [right-now (. (Date.) getTime) > > new-user-entry (conj this-users-params { "updated" right-now })] > > (println apply str new-user-entry) > > > > (if (:username new-user-entry) > > (swap! registry assoc (:username new-user-entry) > new-user-entry)))) > > > > This code works perfectly at the REPL so to show a problem I have to > upload > > this to the live server. So I compile this and upload it to the server > and > > start it up and immediately the Javascript in the browsers of dozens of > > users starts to bombard the app: > > > > java -jar who-is-logged-in-1.0.1-standalone.jar 40000 > > App 'Who is logged in?' is starting > > port: 40000 > > 2012-10-19 11:37:02.731:INFO:oejs.Server:jetty-7.x.y-SNAPSHOT > > 2012-10-19 11:37:02.846:INFO:oejs.AbstractConnector:Started > > SelectChannelConnector@0.0.0.0:40000 > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661023079, username rumbella, > > first_name , last_name , user_image , site www.wpquestions.com} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661023140, username karlorihoo, > > first_name Karlo, last_name Rihoo, user_image , site www.wpquestions.com} > > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661023698} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661024149, username Rainner, > > first_name Rainner, last_name Lins, user_image Rainner_phptZ2nEJ.jpg, > site > > www.javascriptquestions.com} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661024159} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661024269} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661024646} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661024833, username dbranes, > > first_name Dbranes, last_name , user_image dbranes_phpPJVl3f.jpg, site > > www.wpquestions.com} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661025140, username Christianto, > > first_name Christianto, last_name , user_image > Christianto_phpqESzHW.jpg, > > site www.wpquestions.com} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661025337, username jsoni, > > first_name Jatin, last_name Soni, user_image jsoni_phpYlWQTr.jpg, site > > www.wpquestions.com} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661025651} > > > > #<core$apply clojure.core$apply@d1fba79> #<core$str > > clojure.core$str@7dfde44e> {updated 1350661025996} > > > > So, why does this not work? And what is the right way to be sure that > this: > > > > {updated 1350661025996} > > > > Is never added to the registry (since there is no username). > > > > > > > > > > > > > > > > > > > > > > On Friday, October 19, 2012 3:53:26 AM UTC-4, red...@gmail.com wrote: > >> > >> conj can surely produce maps, and does so happily in the following > cases: > >> > >> (conj {} [:foo :bar]) > >> (conj {} {:foo :bar}) > >> > >> I suggesting adding printlns or logging or a debugger and checking the > >> value of this-users-params, it is almost certainly not what you expect > >> it to be. > >> > >> as a side note creating a date object just to call getTime is kind of > >> gross, try (System/currentTimeMillis) > >> > >> On Fri, Oct 19, 2012 at 12:42 AM, Sean Corfield <seanco...@gmail.com> > >> wrote: > >> > On Fri, Oct 19, 2012 at 12:10 AM, larry google groups > >> > <lawrenc...@gmail.com> wrote: > >> >> > >> >> (defn add-to-logged-in-registry [this-users-params] > >> >> (let [right-now (. (Date.) getTime) > >> >> new-user-entry (conj this-users-params { "updated" right-now > >> >> })] > >> >> (if (:username new-user-entry) > >> >> (swap! registry assoc (:username new-user-entry) > >> >> new-user-entry)))) > >> >> > >> >> The if statement seems to never be true. > >> > > >> > > >> > conj produces a sequence, not a map, so the lookup of :username > fails. > >> > Try > >> > new-user-entry (assoc this-users-params "updated" right-now) > >> > -- > >> > Sean A Corfield -- (904) 302-SEAN > >> > An Architect's View -- http://corfield.org/ > >> > World Singles, LLC. -- http://worldsingles.com/ > >> > > >> > "Perfection is the enemy of the good." > >> > -- Gustave Flaubert, French realist novelist (1821-1880) > >> > > >> > -- > >> > You received this message because you are subscribed to the Google > >> > Groups "Clojure" group. > >> > To post to this group, send email to clo...@googlegroups.com > >> > Note that posts from new members are moderated - please be patient > with > >> > your > >> > first post. > >> > To unsubscribe from this group, send email to > >> > clojure+u...@googlegroups.com > >> > For more options, visit this group at > >> > http://groups.google.com/group/clojure?hl=en > >> > >> > >> > >> -- > >> And what is good, Phaedrus, > >> And what is not good— > >> Need we ask anyone to tell us these things? > > > > -- > > You received this message because you are subscribed to the Google > > Groups "Clojure" group. > > To post to this group, send email to clo...@googlegroups.com<javascript:> > > Note that posts from new members are moderated - please be patient with > your > > first post. > > To unsubscribe from this group, send email to > > clojure+u...@googlegroups.com <javascript:> > > For more options, visit this group at > > http://groups.google.com/group/clojure?hl=en > > > > -- > And what is good, Phaedrus, > And what is not good— > Need we ask anyone to tell us these things? > -- You received this message because you are subscribed to the Google Groups "Clojure" group. To post to this group, send email to clojure@googlegroups.com Note that posts from new members are moderated - please be patient with your first post. To unsubscribe from this group, send email to clojure+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/clojure?hl=en