algo.monads

Add conditionals stratements to domonad forms

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
    None
  • Environment:
    The extension was developed in Mac OS X Lion
  • Patch:
    Code

Description

I implemented a patch that would allow us to do conditionals on domonad macros

Example:

(domonad monad-m
[a (action 1)
:if (= a 3)
:then [
result (action c)
]
:else [
result (action d)
]]
result)

Conditional blocks can be nested.

One gotcha one has to be aware of is: you may use bindings from a conditional branch in the m-result sexp as long as that branch gets called, given that one doesn't know for sure which branch is going to be called, you should set the same bindings on both branches.

The patch code is on: github.com/roman/algo.monads

I didn't do a pull request given that you need to this first.

NOTE: I've added this feature in some other projects I've been working on (parser combinators) and is working perfectly so far, there is tests in the patch but I wanted to give you that safe net as well.

Activity

Hide
Roman Gonzalez added a comment -

Adding support for monadic conditionals on the domonad macro

Show
Roman Gonzalez added a comment - Adding support for monadic conditionals on the domonad macro
Roman Gonzalez made changes -
Field Original Value New Value
Attachment monadic-if-else.diff [ 10763 ]
Show
Konrad Hinsen added a comment - This has been accepted long ago: https://github.com/clojure/algo.monads/commit/d9b13dcc57be0d11934441c2dc7a9b287385f435
Konrad Hinsen made changes -
Resolution Completed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: