I wonder why the definline didn't act like a function?
=> (defn pred-transform [obj pred tf]
     (if (pred obj) obj
       (tf obj)))
#'cgws.notcore/pred-transform
=> (pred-transform (println 1) #(not (nil? %)) #(println % "."))
1
nil .
nil
=> (pred-transform (println 1) nil? #(println % "."))
1
nil

=> (fn? pred-transform)
true
=> (definline pred-transform [obj pred tf]
   `(let [o# ~obj]
      (if (~pred o#) o#
       (~tf o#))))
#'cgws.notcore/pred-transform
=> (fn? pred-transform)
true



On Sat, May 25, 2013 at 3:08 PM, atkaaz <atk...@gmail.com> wrote:

> like:
> => (definline pred-transform [obj pred tf]
>    `(let [o# ~obj]
>       (if (~pred o#) o#
>        (~tf o#))))
> #'cgws.notcore/pred-transform
>
> => (pred-transform (println 1) nil? #(println % "."))
> 1
> nil
> => (pred-transform (println 1) #(not (nil? %)) #(println % "."))
> 1
> nil .
> nil
>
>
> On Sat, May 25, 2013 at 3:07 PM, atkaaz <atk...@gmail.com> wrote:
>
>> in which case it does get evaluated twice if form:
>> => (pred-transform (println 1) #(not (nil? %)) #(println % "."))
>> 1
>> 1
>> nil .
>> nil
>>
>> => (pred-transform (println 1) nil? #(println % "."))
>> 1
>> 1
>> nil
>>
>> so maybe a let + gensym would be in order?
>>
>>
>>
>> On Sat, May 25, 2013 at 3:04 PM, atkaaz <atk...@gmail.com> wrote:
>>
>>> Shouldn't it be like:
>>>
>>> (definline pred-transform [obj pred tf]
>>>    `(if (~pred ~obj) ~obj
>>>        (~tf ~obj)))
>>> => (pred-transform 1 #(not (nil? %)) println)
>>> 1
>>> => (pred-transform 1 nil? println)
>>> 1
>>> nil
>>>
>>>
>>>
>>>
>>> On Sat, May 25, 2013 at 2:55 PM, atkaaz <atk...@gmail.com> wrote:
>>>
>>>> just wondering if obj is a form does it get evaluated twice?
>>>>
>>>>
>>>>  On Sat, May 25, 2013 at 2:51 PM, Jim - FooBar(); <jimpil1...@gmail.com
>>>> > wrote:
>>>>
>>>>>  no need for macros... :)
>>>>>
>>>>> (definline safe-assoc [m k v]
>>>>> `(if (contains? ~m ~k) ~m
>>>>>   (assoc ~m ~k ~v)))
>>>>>
>>>>> (definline pred-transform [obj pred tf]
>>>>> `(if ~(pred obj) ~obj
>>>>>     ~(tf obj)))
>>>>>
>>>>> Jim
>>>>>
>>>>>
>>>>>
>>>>> On 25/05/13 12:44, atkaaz wrote:
>>>>>
>>>>> may I see the macro for the latter, if you decide to go that way ? thx
>>>>>
>>>>>
>>>>> On Sat, May 25, 2013 at 2:24 PM, Steven Degutis 
>>>>> <sbdegu...@gmail.com>wrote:
>>>>>
>>>>>> There are two patterns I find in my code that I'm still unhappy with
>>>>>> but I don't know how to clean up.
>>>>>>
>>>>>>  The first is: (if (:attr obj) obj (assoc obj :attr something))
>>>>>>
>>>>>>  I'm basically saying, give this hash-map an attribute if it doesn't
>>>>>> already have it. And just return the thing with an attribute, regardless 
>>>>>> if
>>>>>> I had to add it or not.
>>>>>>
>>>>>>  This version is ugly because it repeats obj three times. I could
>>>>>> write my own macro to de-duplicate it, but I avoid doing that when I can
>>>>>> because there's usually a better built-in solution that I just don't know
>>>>>> about yet.
>>>>>>
>>>>>>  The second is like it: (if (some-test obj) obj (some-transformation
>>>>>> obj))
>>>>>>
>>>>>>  In this one, I just want to return the object, but maybe transform
>>>>>> it first. But the reference to obj happens three times! Still feels like 
>>>>>> it
>>>>>> could be cleaned up.
>>>>>>
>>>>>>  Any thoughts on how to clean these up?
>>>>>>  --
>>>>>> --
>>>>>> 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
>>>>>> ---
>>>>>> You received this message because you are subscribed to the Google
>>>>>> Groups "Clojure" group.
>>>>>> To unsubscribe from this group and stop receiving emails from it,
>>>>>> send an email to clojure+unsubscr...@googlegroups.com.
>>>>>> For more options, visit https://groups.google.com/groups/opt_out.
>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>  --
>>>>> --
>>>>> 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
>>>>> ---
>>>>> You received this message because you are subscribed to the Google
>>>>> Groups "Clojure" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>>> an email to clojure+unsubscr...@googlegroups.com.
>>>>> For more options, visit https://groups.google.com/groups/opt_out.
>>>>>
>>>>>
>>>>>
>>>>>
>>>>>  --
>>>>> --
>>>>> 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
>>>>> ---
>>>>> You received this message because you are subscribed to the Google
>>>>> Groups "Clojure" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>>> an email to clojure+unsubscr...@googlegroups.com.
>>>>> For more options, visit https://groups.google.com/groups/opt_out.
>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>

-- 
-- 
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
--- 
You received this message because you are subscribed to the Google Groups 
"Clojure" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to clojure+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to