Czy ta klasa jest odpowiednia?

Question

Witam! Ostatnio sobie piszę projekt konsolowy (w języku C#), który polega na tym, że użytkownikowi wyświetla się menu z opcjami. Każda opcja to jakaś mini-gra, bądź opcja wyjścia z programu.

Poniżej jest kod klasy do którego mam kilka pytań (Jest to gra wisielec):

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace MultiGry
{
    class HangmanGame : IMenuOption
    {
        public string NameOption => "Wisielec";
        private string[] HangmanDrawing;
        private string[] AllWords;
        private int NumberOfUserErrors;
        private string RandomWord;
        private string GuessedLetters;
        private char PlayerLetter;
        private List<char> User_SelectedLetters;

        public HangmanGame()
        {
            SetHangmanGrawing();
            SetAllWords();
            User_SelectedLetters = new List<char>();
        }

        private void SetHangmanGrawing()
        {
            HangmanDrawing = new string[]
            {
                @"  ___________",
                @"  |    |    |",
                @"  |    |    |",
                @"  |    |    |",
                @"  |    O    |",
                @"  |   /|\   |",
                @"  |  / | \  |",
                @"  |    |    |",
                @"  |    |    |",
                @"  |   / \   |",
                @"  |  /   \  |",
                @"  |         |",
                @"  |         |",
                @" /|\       /|\",
                @"/ | \     / | \"
            };
        }

        private void SetAllWords()
        {
            AllWords = new string[]
            {
                "telefon",
                "komputer",
                "rewolwer",
                "autostrada",
                "huragan",
                "kompresja",
                "kasztan",
                "helikopter",
                "kamper",
                "butelka",
                "kaskader",
                "laptop",
                "komputer",
                "myszka",
                "telefon",
                "pilot",
                "koniunkcja",
                "operator",
                "stolik",
                "pastuch",
                "owca",
                "paluszki",
                "krakersy",
                "serwis",
                "mieszkanie",
                "balon",
                "obiad",
                "karygodny",
                "krokodyl",
                "autostrada",
                "policja",
                "konfident",
                "bachor",
                "kataklizm",
                "wariatka",
                "kontakt"
            };
        }


        public OptionsCategory OptionExecuting()
        {
            SetDefaults();
            DisplayGuessedLetters();

            PlayingGames();

            GameSummary();
            Console.ReadKey();

            var ProgramExecution = new DecisionOnFurtherCourseOfProgram(this);
            return ProgramExecution.UserDecidesWhatToDoNext();
        }


        private void SetDefaults()
        {
            NumberOfUserErrors = 0;
            DrawWordToDuess();
            SetDefaultValueForGuessedLetters();
            User_SelectedLetters.Clear();
        }

        private void DrawWordToDuess()
        {
            var Random = new Random();
            int RandomNumberWord = Random.Next(0, AllWords.Length);
            RandomWord = AllWords[RandomNumberWord];
        }

        // if the letter is not guessed, underscore is inserted
        private void SetDefaultValueForGuessedLetters()
        {
            var tmp = new StringBuilder();
            for (int i = 0; i < RandomWord.Length; ++i)
                tmp.Append("_");

            GuessedLetters = tmp.ToString();
        }

        private void DisplayGuessedLetters()
        {
            for (int i = 0; i < GuessedLetters.Length; ++i)
                Console.Write(GuessedLetters[i] + " ");

            Console.WriteLine();
        }

        private void PlayingGames()
        {
            while (!IsGameOver())
            {
                UserGuessingLetter();
                DisplayGuessedLetters();
                DisplayLettersSelectedByUser();
                Console.WriteLine();
                DisplayHangmanItems();
            }
        }

        private bool IsGameOver()
        {
            for (int i = 0; i < GuessedLetters.Length; ++i)
                if (GuessedLetters[i] == '_')
                    return NumberOfUserErrors == HangmanDrawing.Length;

            return true;
        }

        private void UserGuessingLetter()
        {
            UserGivesLetter();

            if (char.IsLetter(PlayerLetter) == false)
            {
                ErrorMessage("To nie jest litera!");
                return;
            }

            if (WasLetterEntered())
            {
                ErrorMessage("Znak był już wprowadzany!");
                return;
            }

            else if (DidUserGuessedLetter())
                DisclosureOfGuessedLetters();

            else
                ++NumberOfUserErrors;

            Console.Clear();
            User_SelectedLetters.Add(PlayerLetter);
        }

        private void UserGivesLetter() =>
            PlayerLetter = Console.ReadKey(true).KeyChar;

        private void ErrorMessage(string Message)
        {
            Console.WriteLine(Message);
            System.Threading.Thread.Sleep(1000);
            Console.Clear();
        }

        private bool WasLetterEntered()
        {
            foreach (var item in User_SelectedLetters)
                if (item == PlayerLetter)
                    return true;

            return false;
        }

        private bool DidUserGuessedLetter()
        {
            foreach (var item in RandomWord)
                if (PlayerLetter == item)
                    return true;

            return false;
        }

        private void DisclosureOfGuessedLetters()
        {
            for (int i = 0; i < RandomWord.Length; ++i)
                if (PlayerLetter == RandomWord[i])
                {
                    var tmp = new StringBuilder(GuessedLetters);
                    tmp[i] = RandomWord[i];
                    GuessedLetters = tmp.ToString();
                }
        }

        private void DisplayLettersSelectedByUser()
        {
            foreach (var item in User_SelectedLetters)
                Console.Write(item + " ");
        }

        private void DisplayHangmanItems()
        {
            for (int i = 0; i < NumberOfUserErrors; ++i)
                Console.WriteLine(HangmanDrawing[i]);
        }

        private void GameSummary()
        {
            Console.Clear();

            if (DidUserWin())
                Console.WriteLine("Odgadłeś wyraz " + RandomWord + "!");

            else
                Console.WriteLine("Nie udało ci się odgadnąć wyrazu " + RandomWord);

            Console.WriteLine("Ilość błędów: " + NumberOfUserErrors);  
        }

        private bool DidUserWin()
        {
            for (int i = 0; i < GuessedLetters.Length; ++i)
                if (GuessedLetters[i] == '_')
                    return false;

            return true;
        }
    }
}

Czy ta klasa jest za duża (ma około 255 linijek, 1 metodę publiczną i 18 prywatnych metod), jeśli tak, to czy da się ją podzielić na mniejsze części?

Czy ta klasa wykonuje zbyt dużo operacji?

Czy ta klasa narusza zasadę pojedynczej odpowiedzialności, bądź inne zasady SOLID?

Jestem osobą, która od niedawna zaczęła przejmować się czystością kodu i nie wiem jak pisać dobry kod, dobre klasy itd. Zatem prosiłbym o nakierowanie mnie jak mógłbym osiągnąć czystość kodu, co trzeba wiedzieć aby pisać dobrze.

PS.

Jeśli mój kod łamie jakieś inne zasady, koncepcje, praktyki programistyczne to byłbym wdzięczny za pokazanie co jest nie tak. 

Z góry dziękuje za pomoc.

Comments

Wszystko zależy od konkretnego przypadku. 255 linijek to nie tak dużo, czasami widuję takie co mają grubo ponad 2tyś.

Co do zasad SOLID to myślę, że zostały złamane zasady, ale czy jest sens się tym przejmować? To tylko wytyczne dobrych praktyk, które często dają więcej problemów niż korzyści.

---

Czemu uważasz, że zasady SOLID są złe? Jeśli mogę spytać.

---

Np. tu:

private void ErrorMessage(string Message)
        {
            Console.WriteLine(Message);
            System.Threading.Thread.Sleep(1000);
            Console.Clear();
        }

chcąc zmienić sposób wyświetlania błędu albo dodać nową funkcjonalność do tej metody trzeba będzie zmodyfikować tą klasę

Tu masz fajny opis i nawet jest przykład z SendMessage: http://itcraftsman.pl/solidny-jak-solid-pisanie-solidnych-aplikacji-w-jezyku-c/

Czemu uważasz, że zasady SOLID są złe? 

 Każdy program jest inny i każdy kod służy czemuś innemu. Te zasady to tylko zbiór dobrych praktyk. Skoro wiadomo, że dana klasa w programie będzie działać tak i tak to często nie ma seansu jej zmieniać tylko po to by spełniała jakieś zasady. SOLID według mnie najlepiej sprawdza się przy pisaniu bibliotek i bardzo dużych programów. Przy aplikacjach webowych nie ma seansu się tym zbytnio przejmować.

---

@JakSky Dzięki za odpowiedź!

---

linijki od 53 do 88 to słowa, które lepiej byłoby wrzucić do pliku i z niego je w jakiś sposób odczytywać. Bo teraz to ładujesz do ramu z 30 słów które tam raczej na nic się nie przydadzą. Lepiej byłoby wylosować liczbę, od 1 do *ilość linijek w pliku wyrazy.txt* i wziąć linie o takim numerze.

---

MsMaciek123 Faktycznie, to byłoby lepsze rozwiązanie. Dzięki!