Question

我正在制作一款名为＆＃34; Lingo＆＃34;对于我的编程课程。我已经完成了大部分工作。但是我们应该在表单类中尽可能少地使用代码。所以我必须把它的大部分都移到另一个班级。关于我应该如何做的任何想法？

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Text.RegularExpressions;
using System.IO;

namespace Lingo
{
    public partial class Form1 : Form
    {
        Random r = new Random();
        AboutBox1 about = new AboutBox1();
        int gissningar, x, score, highcsore;
        Label[] labels = new Label[25];
        string ord;
        public Form1()
        {
            InitializeComponent();
            gissningar = 0;
            x = 0;
            score = 1000;
            highcsore = 0;
            labels[0] = label1;
            labels[1] = label2;
            labels[2] = label3;
            labels[3] = label4;
            labels[4] = label5;
            labels[5] = label6;
            labels[6] = label7;
            labels[7] = label8;
            labels[8] = label9;
            labels[9] = label10;
            labels[10] = label11;
            labels[11] = label12;
            labels[12] = label13;
            labels[13] = label14;
            labels[14] = label15;
            labels[15] = label16;
            labels[16] = label17;
            labels[17] = label18;
            labels[18] = label19;
            labels[19] = label20;
            labels[20] = label21;
            labels[21] = label22;
            labels[22] = label23;
            labels[23] = label24;
            labels[24] = label25;
            String[] ordLista = File.ReadAllLines(("words.txt").ToString());
            ord = ordLista[r.Next(r.Next(ordLista.Length))];
            ord = ord.ToUpper();
            labels[0].Text = Convert.ToString(ord[0]);
        }

        private void quitToolStripMenuItem_Click(object sender, EventArgs e)
        {
            this.Close();
        }

        private void aboutToolStripMenuItem_Click(object sender, EventArgs e)
        {
            about.Show();
        }

        private void addWordsToolStripMenuItem_Click(object sender, EventArgs e)
        {
            new Form2().Show();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            string ordet = ord;
            string svar = textBox1.Text.ToUpper();
            if (Regex.IsMatch(svar, "^[a-zA-Z]{5}$"))
            {
                if (x <= 20)
                {

                    if (svar == ord)
                        for (int i = 0; i <= 4; i++)
                        {

                            labels[i + x].Text = Convert.ToString(svar[i]);
                            labels[i + x].BackColor = Color.Green;
                            highcsore = score;
                        }
                    else
                    {
                        gissningar++;
                        score = score / gissningar;
                        for (int i = 0; i <= 4; i++)
                        {
                            labels[i + x].BackColor = Color.Red;
                        }
                        for (int i = 0; i <= 4; i++)
                        {
                            labels[i + x].Text = Convert.ToString(svar[i]);
                            if (svar[i] == ord[i])
                            {

                                labels[i + x].BackColor = Color.Green;
                                ordet = ordet.Insert(i, "?");
                                ordet = ordet.Remove(i + 1, 1);
                                if (x < 20)
                                {
                                    labels[x + 5].Text = Convert.ToString(ord[0]);
                                    labels[i + x + 5].Text = Convert.ToString(svar[i]);
                                }
                            }
                        }
                        for (int i = 0; i <= 4; i++)
                        {
                            for (int k = 0; k <= 4; k++)
                            {
                                if (svar[i] == ordet[k])
                                {

                                    labels[i + x].BackColor = Color.Orange;
                                    svar = svar.Insert(i, "#");
                                    svar = svar.Remove(i + 1, 1);
                                }



                            }
                        }

                    }
                    //kod för att jämföra chars


                    x = x + 5;
                }
                else
                {
                    MessageBox.Show("Du förlorade! \r\n Ordet var: " + ord);
                }

            }
            else
            {
                MessageBox.Show("Ordet måste ha 5 bokstäver");
            }

        }


    }
}

Answer 1

我不确定复制/粘贴其他类/方法中的所有内容就是答案。

你最好在“图层”中重构你的代码：游戏逻辑，渲染/图形逻辑，与用户一起行动......

Answer 2

选择要移至课堂的代码块，右键单击 - ＆gt;重构 - ＆gt;生成方法..

这将生成一个方法，将所有依赖项作为输入，然后您可以将函数移动到另一个类或类库！

<强>更新
在重构之前，将Daniel的逻辑代码与控件分开。这将帮助您将逻辑保存在类文件中并控制表单类！您可以将控件的输入作为参数发送到类函数/方法！

Answer 3

我建议做一个“游戏”课程。该类将包含所有游戏初始化和设置逻辑。所以在它里面你可能有一个init()来执行你在构造函数中拥有的大部分内容。您在课程开始时拥有的大多数属性也应该移动到这个新类中。

我不确定button1_Click中的代码是做什么的，但也会有doGameLogic方法。这个类应该创建一个新的游戏实例，并通过游戏类访问游戏特定的成员和逻辑。您的表单类应该使用游戏所需的控件。

如何将方法移动到另一个类并调用它？

3 个答案: