正确使用类？

Question

我是一名大学生（计算机科学），刚刚开始 C# 编程课程。 对于我们的作业，我一直使用一个名为“Display”的类，其中放置了可以在整个项目中多次使用的任何控制台输出。例如，继续或退出程序的请求。我不需要在 Main() 中多次键入它，而是从 Display 类中调用该方法。

另一位高年级的学生告诉我，我不应该这样做。这是糟糕的编码实践，我应该只在主类（包含 Main()）中包含所有方法，并且仅在绝对需要时才使用其他类。

我只是在寻找一些意见和建议。

我被要求包含代码。我原本打算写这篇文章，但不想让这篇文章太长。我选择了一项相当短的作业。我想澄清一下，我只是在学习，所以代码并不像你们许多人写的那么优雅。非常欢迎建设性的批评。

最终我只是在玩类的使用。我知道 Display 类中的一些方法也可以轻松地位于 Main() 中。

这是包含 Main() 的 Program 类

namespace Chapter_6_10
{
class Program
{
    static void Main()
    {
        string triangle = "", result = " ";;
        char printingCharacter = ' ';
        int peakNumber = 0;
        Display.Instructions();
        Display.Continue();
        // perform a do... while loop to build triangle up to peak
        do
        {
            Console.Clear();
            Request.UserInput(out printingCharacter, out peakNumber);
            int  counter = 1, rowCounter = 0;
            do
            {
                do
                {
                    triangle += printingCharacter;
                    rowCounter++;
                }
                while (rowCounter < counter);
                counter++;
                rowCounter = 0;
                triangle += "\n";
            }
            while(counter != peakNumber);
            // perform a do... while loop to build triangle from peak to base
            do
            {
                do
                {
                    triangle += printingCharacter;
                    rowCounter++;
                }
                while (rowCounter < counter);
                counter--;
                rowCounter = 0;
                triangle += "\n";
            }
            while (counter != 0);
            Console.Clear();
            Console.WriteLine(triangle); // display triangle
            Display.DoAgain(out result); // see if user wants to do another or quit
            triangle = "";                
        }
        while (result != "q"); 
    }
}

这是 Display 类

namespace Chapter_6_10
{
// This class displays various outputs required by program
class Display
{
    // This method display the instructions for the user
    public static void Instructions()
    {
        Console.WriteLine("\nThis program will ask you to enter a character to be used "
            + " to create triangle."
            + "\nThen you will be asked to enter a number that will represent the"
            + "\ntriangles peak."
            + "\nAfter your values have been received a triangle will be drawn.");
    }
    // This method displays the choice to continue
    public static void Continue()
    {
        Console.WriteLine("\n\nPress the enter key when you are ready to continue...");
        Console.ReadLine();
    }
    // This method displays an error message
    public static void Error(string ErrorType)
    {
        Console.WriteLine("\nYou have entered \"{0}\", which is a value that is not valid!"
            + "\nThis is not rocket science."
            + "\n\nTry agian...", ErrorType);
    }
    // This method will ask user to press enter to do again or 'q' to quit
    public static void DoAgain(out string Result)
    {
        string input = " ";
        Console.WriteLine("\nPress enter to run program again"
            + "\nor type the letter 'q' to close the application.");
        input = Console.ReadLine();
        // convert input to lowercase so that only one test needed
        Result = input.ToLower();
    }        
}

这是 Request 类

namespace Chapter_6_10
{
// This class is used to get user input
class Request
{
    public static void UserInput(out char PrintingCharacter, out int PeakNumber)
    {
        string input = " ";
        char testCharacter = ' ';
        int testNumber = 0;

        // a do... while loop to get Printing Character from user
        // use TryParse() to test for correct input format
        do
        {
            Console.Write("\nPlease enter a character to be used to build triangle : ");
            input = Console.ReadLine();
            bool result = char.TryParse(input, out testCharacter);
            if (result)
            {

            }
            else
            {
                Console.Clear();
                Display.Error(input);
                input = " ";
            }
        }
        while (input == " ");
        // a do... while loop to get number from user
        // use TryParse() to test for correct input format
        do
        {
            Console.Write("\nPlease enter a number <between 1 and 10> for the peak of the triangle : ");
            input = Console.ReadLine();
            bool result = int.TryParse(input, out testNumber);
            if (result)
            {
                if ((testNumber > 0) && (testNumber < 11))
                {                        
                }
                else
                {
                    Console.Clear();
                    Display.Error(testNumber.ToString());
                    input = " ";
                }
            }
            else
            {
                Console.Clear();
                Display.Error(input);
                input = " ";
            }
        }
        while (input == " ");
        // assigned received values to 'outs' of method
        PrintingCharacter = testCharacter;
        PeakNumber = testNumber;
    }
}

就是这样。这会被认为是一种低效的编码方式吗？我该如何改进它？

感谢迄今为止的所有投入。这对我来说非常有价值。

Answer 1

彻底且设计良好的类结构对于遵守面向对象的设计 原则。

以下是考虑将相关代码分解为单独的类的几个原因：

• 它创建了劳动分工并隔离了不同的任务。这有时被解释为单一职责原则，它说每个对象（类）都应该有一个单一责任并专注于完成单一任务。 封装也很快成为这里的一个重要原则，基本上意味着数据是捆绑的使用负责操作该数据的方法。这也有助于减少错误潜入代码的机会。

• 它可以帮助促进代码重用。将现有的类与您已经编写的代码集成到另一个应用程序中通常比该代码分散在整个应用程序中要容易得多。为什么要一遍又一遍地重写相同的代码？

• 精心设计的类结构可以创建逻辑层次结构，使您的代码更易于理解和概念化，并且从长远来看使您的应用程序更易于维护。这与我们将计算机上的文件组织到文件夹中以及我们生活中的其他一切（或者至少我们尝试过）的原因相同。

Answer 2

这是一种糟糕的编码实践，我应该只在主类中包含所有方法[...]，并且仅在绝对需要时才使用其他类。

我绝对不同意。这种思路最终会导致一些夸大的、厨房水槽类的结果，它们最终会做所有。

如果我们谈论结构化编程，您的同事是对的，您可能只有子例程而没有类 - 组织功能的主要方法是将其划分为子例程。但这里我们谈论的是面向对象编程，它也为您提供了将功能划分为不同类的方法。毕竟，类是用来使用的，否则我们就不会称之为 OOP！

如果我是您，我会更喜欢在需要时定义新类的相当自由的方法。

---

PS： 不用说，确实有一些最佳实践可以帮助您决定是否需要为某些东西创建一个新类，以及应该如何设计它。 （参见 Cody Gray 的回答。）我只想在这里指出，在 OOP 中主动避免类肯定没有意义。

Answer 3

永远不要听那些无法向您解释原因的人说“这是好的做法”或“这是不好的做法” > 这是好是坏。就像在其他领域一样，编程中也存在许多刻板印象，请尽量避免它们。

对于你的特定示例，这有点模糊，因为我实际上不知道你的类是做什么的，我不认为将 IO 功能分离在一个单独的类中是一个非常糟糕的主意。然而，这实际上取决于它的作用，是否独立等等。也取决于个人品味:)

Answer 4

一般来说，如果您多次输入相同的内容，我认为就会出现问题。像这样封装显示例程实际上是一种很好的形式。它允许您轻松更改多个实例的输出，特别是对于错误消息之类的内容。

他可能有更多的 C 风格背景，而 C# 则更像 Java。允许甚至鼓励自由使用类。