Hi,
First you should ask yourself if you need concurrent access to that state. If not then you can thread this state as argument, e.g.
```
data State = State
someAction :: State -> IO a
someOtherAction :: State -> IO b
```
This functions compose perfectly well, if you need to update the state then each action should also return it:
```
someAction :: State -> IO (State, a)
someOtherAction :: State -> IO (State, b)
```
You still can compose those. This is `StateT` monad transformer in disguise (from `transformers` / `stm` package combo), but I personally prefer to pass arguments explicitly and only use `StateT` or `RederT` monad if they bring more clarity.
If you need concurrent access from different threads, then I'd use `MVar` or `stm` for holding a mutable variable, rather than `IORef` (which are useful if the code needs to be as fast as possible, but they present less guarantees).
If you go with mutable cell to hold the state, then there's no need to create it using `unsafePerformIO`, you can directly create it in `main :: IO ()` and pass it around as an argument. `unsafePerformIO` is really rarely needed and most of the time there are well established patterns to avoid using it.
Best regards,
Marcin Szamotulski
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Saturday, November 14th, 2020 at 14:53, Dmitriy Matrosov
Hi.
I want to use [telnet library][1] to run some
commands on cisco switches and then parse output.
Thus, first i need to login (enter username and
password, etc) and then run commands and collect
output. As far as i understand that library API,
all of output parsing should be done in
EventHandler
of type
TelnetPtr -> Event -> IO ()
But, these (mine) operations require some state
(to track where am i now (entered username,
entered password, etc) and to collect output). But
the monad of this event handler is IO, so i can't
see any simple way of adding state to it apart
from ['unsafePerfromIO' trick][2], like
telnetRef :: IORef TelnetRef
{-# NOINLINE telnetRef #-}
telnetRef = unsafePerformIO $ newIORef (TelnetRef undefined Unauth M.empty)
('undefined' is 'Telnet' pointer, which is
returned by 'telnetInit' and will be initialized
later..)
Is there a better a way to do this?
Thanks.
Beginners mailing list
Beginners@haskell.org