Форум программистов, компьютерный форум, киберфорум
Haskell
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
82 / 75 / 10
Регистрация: 12.08.2019
Сообщений: 81

Как лучше (пере)оформить этот код ?

28.05.2024, 17:23. Показов 802. Ответов 0

Студворк — интернет-сервис помощи студентам
Я решил написать простую программу для игры "виселица". Она работает, но мне очень не нравится как выглядит код: он выглядит очень рваным и неструктурированным, хотя я старался разделять на отдельные функции.

Модули
Кликните здесь для просмотра всего текста
Main.hs
Кликните здесь для просмотра всего текста
Haskell
1
2
3
4
5
6
module Main (main) where
import HangmanOld
 
main :: IO ()
main = do
    runGame

HangmanOld.hs:
Кликните здесь для просмотра всего текста
Haskell
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
{-# LANGUAGE TupleSections #-}
module HangmanOld where
import Control.Monad.Trans.State.Lazy
import System.Random
import Control.Monad.Trans.Class (MonadTrans(..))
import Data.Char (toLower)
import Consts
import Util
import Data.Maybe (fromMaybe, listToMaybe, isJust)
 
data GameState = GS {lives :: Int, word :: [(Char, Bool)], guesses :: String} -- тип данных "состояние игры"
 
instance Show GameState where
    show (GS l word _) = map (\(ch, b) -> if b then ch else '_') word ++ "\n"
                            ++ "Жизней: (" ++ take maxLives (replicate l '#' ++ repeat ' ') ++ ")\n" --toString()
 
initGameState :: String -> GameState
initGameState word = GS maxLives (map (, False) word) "" --по загаданному слову лепим исходное состояние⌈
 
runGame :: IO ()
runGame = do
    strings <- words <$> readFile fileName                                   -- считываем файл
    word <- (strings !!) . fst . randomR (0, length strings-1) <$> newStdGen -- парсим его на слова, выбирая среди них рандомное
    evalStateT hangMan (initGameState word)                                  -- начинаем игру на выбранном слове
 
hangMan :: StateT GameState IO () -- сама игра
hangMan = do
    lift . print =<< get --печатаем состояние игры
    test <- testEndGame <$> get --смотрим не гейм овер ли у нас
    maybe (untilM id tryChar *> hangMan) (lift . putStrLn) test --если геймовер то печатаем, а если нет, то идём считывать букву
 
 
testEndGame :: GameState -> Maybe String  --тестируем состояние игры на предмет геймовера
testEndGame (GS lives word _)   | lives <= 0   = Just loseMsg -- всосали, если жизней - 0
                                | all snd word = Just winMsg  -- выиграли, если все буквы открыты
                                | otherwise    = Nothing      -- если нет, то продолжаем играть
 
tryChar :: StateT GameState IO Bool -- пробуем считать букву с строки stdin
tryChar = do
    maybeChar <- listToMaybe <$> lift getLine -- считываем строку, с которой мы будем брать первую букву
    maybe (return ()) acceptChar maybeChar --смотрим на то, что считали
    return $ isJust maybeChar -- возвращаем булеан наличия буквы
 
acceptChar :: Char -> StateT GameState IO () -- меняем состояние по букве
acceptChar ch = do
    (msg, gs') <- applyChar ch <$> get -- смотрим на изменённое состояние
    put gs' -- обновляем состояние
    lift $ putStrLn msg -- сообщаем об изменениях
 
 
applyChar :: Char -> GameState -> (String, GameState) --обновление состояния игры по букве
applyChar chr gs@(GS lives word guesses) | elemGuesses  = (sameCharMsg,     gs)                                  --уже загадывали такую букву
                                         | correctGuess = (correctGuessMsg, gs{word = word', guesses = guesses'})--отгадали правильно
                                         | otherwise    = (wrongGuessMsg,   gs{lives = pred lives, guesses = guesses'}) where--назвали неверную букву
    ch = toLower chr  --регистр не учитываем
    elemGuesses = ch `elem` guesses --была ли эта буква у нас раньше?
    correctGuess = ch `elem` map (toLower . fst) word --еть ли данная буква в слове?
    word' = map (\(c, b) -> (c, b || toLower c == ch)) word -- обновили отгаданные буквы
    guesses' = ch : guesses  --добавляем букву к списку использованных

Util.hs
Кликните здесь для просмотра всего текста
Haskell
1
2
3
4
5
6
module Util where
 
untilM :: Monad m => (a -> Bool) -> m a -> m a --выполняем эффект вычисления до тех пор, пока предикат не выдаст на его значении True
untilM f ma = do
    a <- ma
    if f a then return a else untilM f ma

Consts.hs
Кликните здесь для просмотра всего текста
Haskell
1
2
3
4
5
6
7
8
9
10
11
12
13
14
module Consts where
 
maxLives :: Int
maxLives = 5
 
fileName, loseMsg, winMsg, sameCharMsg, wrongGuessMsg, correctGuessMsg :: String
fileName = "words.txt"
 
loseMsg = "Вы проиграли"
winMsg = "Вы выиграли"
 
sameCharMsg = "Буква уже была загадана"
wrongGuessMsg = "Правильно"
correctGuessMsg = "Неправильно"

зависимости в package.yaml
Кликните здесь для просмотра всего текста
Haskell
1
2
3
4
5
dependencies:
- arrows #нужны для аналогичной проги на стрелках
- transformers
- random
- base >= 4.7 && < 5


Мне не нравится как код скомпонован: всё как-то скомкано и неструктурировано. Прошу обозреть мой код и сказать что и как здесь можно исправить в сторону увеличения порядка. Прошу не кидаться тапками слишком сильно - я самоучка.
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
28.05.2024, 17:23
Ответы с готовыми решениями:

Подскажите как лучше оформить код с указателями
Подскажите пожалуйста, какой вариант лучше предпочесть и почему? int main(int argc, char** argv) { QApplication app(argc,...

Как упростить, сделать лучше этот код
import random L1 = random.randint(0,24) R1 = random.randint(0,24) L2 = random.randint(25,49) R2 = random.randint(25,49) ...

Оформить лучше программный код
У меня была такая задача создать мультик человека поднимающего штангу,я нарыл в интернете пару функций, ну и создал вот ЭТО!!! Помогите...

0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
28.05.2024, 17:23
Помогаю со студенческими работами здесь

Как оформить в программе этот цикл?
как оформить в программе этот цикл Вот начало программы program lab2; uses crt; var...

Как оформить этот массив через функциию?
Что бы значения в функцию передавались в качестве параметров #include &lt;iostream&gt; #include &lt;cmath&gt; using namespace std; ...

Как лучше оформить event?
Добрый день, имеется такая конструкция вызова эвента: namespace name_Space_Main_Form { public partial class Form1 : Form ...

Как лучше оформить таблицу?
Добрый день. на сайте хочу создать таблицу с переводом популярных слов на один иностранный язык с 3 колонками: 1. слово 2....

Подскажите, как лучше оформить на Bootstrap
Здравствуйте Использую Bootstrap 3 Подскажите, как лучше разместить изображения, чтобы не было проблем с адаптивностью ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
1
Ответ Создать тему
Новые блоги и статьи
Автоматическое создание документа при проведении другого документа
Maks 29.03.2026
Реализация из решения ниже выполнена на нетиповых документах, разработанных в конфигурации КА2. Есть нетиповой документ "ЗаявкаНаРемонтСпецтехники" и нетиповой документ "ПланированиеСпецтехники". В. . .
Настройка движения справочника по регистру сведений
Maks 29.03.2026
Решение ниже реализовано на примере нетипового справочника "ТарифыМобильнойСвязи" разработанного в конфигурации КА2, с целью учета корпоративной мобильной связи в коммерческом предприятии. . . .
Автозаполнение реквизита при выборе элемента справочника
Maks 27.03.2026
Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .
Сумматор с применением элементов трёх состояний.
Hrethgir 26.03.2026
Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .
Автозаполнение реквизитов при создании документа
Maks 26.03.2026
Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .
Команды формы и диалоговое окно
Maks 26.03.2026
1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .
Кому нужен AOT?
DevAlt 26.03.2026
Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .
Отправка уведомления на почту при создании или изменении элементов справочника
Maks 24.03.2026
Программная отправка письма электронной почты на примере типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной записи электронной. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru