Discussion :

[Antoine_935]

Salut, c'est encore moi

Je n'aime pas beaucoup les blocs do, et m'efforce donc de les supprimer.
Sur le coup, j'ai un peu de mal.

Le code avec le do:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
import qualified Data.ByteString.Lazy as L
 
hasElfMagic :: L.ByteString -> Bool
hasElfMagic content = L.take 4 content == elfMagic
    where elfMagic = L.pack [0x7f, 0x45, 0x4c, 0x46]
 
isElfFile :: FilePath -> IO Bool
isElfFile path = do
  content <- L.readFile path
  return (hasElfMagic content)

Je souhaite donc supprimer le do dans l'action isElfFile.

Ma première idée a été d'utiliser l'opérateur >>=, puis j'ai essayé de chainer les fonctions avec des . (points)
Rien n'y fait, Haskell ne me laissera pas mélanger des actions et des fonctions :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
Prelude> :load isElf
[1 of 1] Compiling Main             ( isElf.hs, interpreted )
 
isElf.hs:8:37:
    Couldn't match expected type `IO Bool' against inferred type `Bool'
    In the second argument of `(>>=)', namely `hasElfMagic'
    In the expression: L.readFile path >>= hasElfMagic
    In the definition of `isElfFile':
        isElfFile path = L.readFile path >>= hasElfMagic
Failed, modules loaded: none.
Prelude> :load isElf
[1 of 1] Compiling Main             ( isElf.hs, interpreted )
 
isElf.hs:8:26:
    Couldn't match expected type `L.ByteString'
           against inferred type `IO L.ByteString'
    In the second argument of `(.)', namely `L.readFile'
    In the expression: hasElfMagic . L.readFile
    In the definition of `isElfFile':
        isElfFile = hasElfMagic . L.readFile
Failed, modules loaded: none.

J'ai fini par trouver une solution avec une fonction lambda, mais je trouve que c'est beaucoup de bazar pour une petite chose

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

isElfFile path = L.readFile path >>= (\contents -> return $ hasElfMagic contents)

Existe-t-il une solution plus concise ?

[dividee]

Existe-t-il une solution plus concise ?

Ceci est plus concis, mais pas forcément plus lisible:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

isElfFile path = L.readFile path >>= return . hasElfMagic

La transformation que tu fais est souvent indentée comme ceci, en écho au bloc do (et ça reste lisible si le bloc est plus long):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
isElfFile path =
        L.readFile path >>= \contents ->
        return $ hasElfMagic contents

[Antoine_935]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

isElfFile path = L.readFile path >>= return . hasElfMagic

Gnarf, j'y étais presque

Je n'avais pas percuté que return était une fonction.
En inversant l'opérateur >>=, on peut arriver à une notation presque point-free.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
isElfFile :: FilePath -> IO Bool
isElfFile path = return . hasElfMagic =<< L.readFile path

Seulement, dès que je retire le path, c'est la catastrophe

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
*Main System.FilePath> :load isElf
[1 of 1] Compiling Main             ( isElf.hs, interpreted )
 
isElf.hs:8:12:
    Couldn't match expected type `IO Bool' against inferred type `Bool'
      Expected type: FilePath -> IO Bool
      Inferred type: FilePath -> Bool
    In the first argument of `(=<<)', namely `return . hasElfMagic'
    In the expression: return . hasElfMagic =<< L.readFile
Failed, modules loaded: none.

Une idée ?


Merci en tout cas pour cette belle réponse


Autre question en passant :
L'opérateur >> existe pour ordonner deux actions.
Existe-t-il l'opérateur inverse << ?*(celui là ne semble pas exister du moins...)

[dividee]

Une idée ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
isElfFile = (=<<) (return . hasElfMagic) . L.readFile
-- ou bien
isElfFile = (return . hasElfMagic =<<) . L.readFile
-- ou encore
isElfFile = (>>= return . hasElfMagic) . L.readFile

Mais si tu préfères ça au bloc do, il faut te faire soigner

[EDIT:] pour ton autre question, tu peux toujours écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
(<<) :: Monad m => m b -> m a -> m b
(<<) = flip (>>)

[Antoine_935]

Mais si tu préfères ça au bloc do, il faut te faire soigner

Haha, probablement

Merci pour ces réponses bien claires

[viro]

Haha, probablement

Merci pour ces réponses bien claires

ce qui est de la forme

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
do variable <- action
   return ( fonction variable )

peut généralement se transformer en

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

fonction `fmap` action

Donc là on a :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

is2 p = hasElfMagic `fmap` L.readFile p

Et en pointfree:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

is3 = (hasElfMagic `fmap`) . L.readFile