Question

我正在尝试为CoordinateValidation创建一些单元测试，但是需要将一些控件传递给它。我已经尝试过进行MVC（尚不成熟），但我认为我做得还不错，就好像我不需要传递控件一样。

是我没有正确完成MVC还是我只是缺少某些东西？

这里是我尝试测试的方法和实际的单元测试：

private static bool ValidRange(TextBox tb)
        {
            float x = float.Parse(tb.Text.Split(',')[0]);
            float y = float.Parse(tb.Text.Split(',')[1]);

            if ((x < 10 || x > 1000) || (y < 10 || y > 730))
            {
                return false;
            }
            return true;
        }

public static bool CoordinateValidation(Panel pointsPanel, ErrorProvider errorProvider)
        {
            bool status = true;
            Regex regex = new Regex(@"^((\d+\.?\d*),{1}(\d+\.?\d*))$");

            foreach (TextBox tb in pointsPanel.Controls)
            {
                Match match = regex.Match(tb.Text);

                if (!match.Success)
                {
                    errorProvider.SetError(tb, "Invalid coordinates!");
                    status = false;
                }

                else if (!ValidRange(tb))
                {
                    errorProvider.SetError(tb, "Invalid range!");
                    status = false;
                }

                else
                {
                    errorProvider.SetError(tb, "");
                }
            }
            return status;
        }

namespace BezierLibrary.UnitTests
{
    [TestClass]
    public class BezierHelperTests
    {
        [TestMethod]
        public void CoordinateValidation_InvalidCoordinate_ReturnFalse()
        {
            //Arrange
            var panel = new Panel();
            var textbox = new TextBox();
            var errorProvider = new ErrorProvider();
            panel.Controls.Add(textbox);
            textbox.Text = "5";

            //Act
            var result = BezierHelper.CoordinateValidation(panel, errorProvider);

            //Assert
            Assert.IsFalse(result);

        }
    }
}

Answer 1

正如莱斯泽克·雷皮（Leszek Repie）在评论中所说，您不应该通过控制。切勿将表示层（显示/控件）与业务层（计算）混合使用。因此，当用户单击“确定”按钮时，您可以在OkButtonClick处理程序中从文本框中提取文本。然后，您可以将此文本作为参数传递给业务功能。这还具有使您的单元测试更加简单的好处-您只需对测试字符串进行硬编码即可。

考虑使用这种形式的功能：

bool CoordinateValidation(String coordinates, ref String errorMessage    )

bool ValidRange( String coordinates )

完成该工作后，将注意力转移到CoordinateValidation（）。它自己进行一些验证（验证坐标），然后调用ValidateRange（）。我建议提取坐标的验证并将该代码放在新函数中。然后，您的CoordinateValidation函数仅调用新函数和ValidateRange（）。

希望有帮助。

编辑-这是一个小例子：

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

namespace WindowsFormsApp1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click( object sender, EventArgs e )
        {
            List<DataTransferObject> dtos = new List<DataTransferObject>();
            dtos.Add( new DataTransferObject() { Field = FieldId.FieldA, Text = textBox1.Text } );
            dtos.Add( new DataTransferObject() { Field = FieldId.FieldB, Text = textBox2.Text } );

            BusinessThing businessThing = new BusinessThing();
            businessThing.Validate( dtos );

            foreach( var dto in dtos )
            {
                if( dto.Error != null )
                {
                    if( dto.Field == FieldId.FieldA )
                    {
                        label1.Text = dto.Error;
                    }
                    // etc
                }
            }
        }
    }

    public enum FieldId
    {
        FieldA,
        FieldB
    }

    public class DataTransferObject
    {
        public FieldId Field { get; set; }
        public String Text { get; set; }
        public String Error { get; set; } = null;
    }

    public class BusinessThing
    {
        private Regex regex = new Regex(@"^((\d+\.?\d*),{1}(\d+\.?\d*))$");

        public void Validate( List<DataTransferObject> dtos )
        {
            foreach( var dto in dtos )
            {
                Validate( dto );
            }
        }

        public void Validate( DataTransferObject dto )
        {
            if( ValidateCoordonate( dto ) )
            {
                ValidateRange( dto );
            }
        }

        public Boolean ValidateCoordonate( DataTransferObject dto )
        {
            Match match = regex.Match(dto.Text);

            if( match.Success )
            {
                return true;
            }
            else
            {
                dto.Error = "Invalid coordinates!";
                return false;
            }
        }


        public Boolean ValidateRange( DataTransferObject dto )
        {
            float x = float.Parse(dto.Text.Split(',')[0]);
            float y = float.Parse(dto.Text.Split(',')[1]);

            if( (x < 10 || x > 1000) || (y < 10 || y > 730) )
            {
                dto.Error = "Invalid range!";
                return false;
            }
            return true;
        }
    }
}

单元测试获胜表格验证方法

1 个答案: