我将如何清理这个？

Question

我正在使用一个利用谷歌天气 API 的应用程序，现在我得到了一些真正丑陋的代码来填充 3 天的预测，看起来像这样（记住不要笑）

groupBox3.Text = set.Forecast[1].DayOfTheWeek;
label4.Text = string.Format("High {0}", set.Forecast[1].High);
label3.Text = string.Format("Low: {0}", set.Forecast[1].Low);
label11.Text = string.Format("Condition: {0}", set.Forecast[1].Condition);
pictureBox1.Load(set.Forecast[1].Icon);

groupBox4.Text = set.Forecast[2].DayOfTheWeek;
label14.Text = string.Format("High {0}", set.Forecast[2].High);
label13.Text = string.Format("Low: {0}", set.Forecast[2].Low);
label20.Text = string.Format("Condition: {0}", set.Forecast[2].Condition);
pictureBox2.Load(set.Forecast[2].Icon);

groupBox5.Text = set.Forecast[3].DayOfTheWeek;
label7.Text = string.Format("High {0}", set.Forecast[3].High);
label6.Text = string.Format("Low: {0}", set.Forecast[3].Low);
label11.Text = string.Format("Condition: {0}", set.Forecast[3].Condition);
pictureBox3.Load(set.Forecast[3].Icon);

我告诉过你这很丑。 我想做的是有一个循环（和可能的泛型）来完成与这个丑陋代码相同的任务。 我尝试了几种不同的方法，但它们总是失败。 天气是这样生成的

WeatherSet set = WeatherService.Response(GoogleWeatherRequest.RequestData(new WeatherService("99109")));

在变量集中有当前信息和 3 天预测的信息，但是让它像这样运行让我发疯，因为它太笨拙而且效率不高。 所以有人有任何酷而有效的方法来完成这项任务吗？

Answer 1

我认为您应该使用 groupbox、三个标签和图片框为其创建自己的自定义控件 ForecastView。 让它有一个名为Forecast的属性，并将填充字段的逻辑放在它的 setter 中。 这样表单的代码就会非常干净，比如

forecastView1.ForeCast = set.Forecast[1];
forecastView2.ForeCast = set.Forecast[2];
// etc

鉴于您已经将字段放在组框内，放置应该不是问题。

此外，如果有一天您需要在另一个表单或什至在另一个应用程序中显示这些数据，那么将它们全部置于单独的控制中将为您提供大量代码重用选项。 更不用说如果您决定将图片框放在左边而不是右边等，您将能够一次更改所有预测的外观。

Answer 2

您的代码没有什么低效的地方。 重复自己有点难过，但这并没有什么错。

如果您想避免重复，只需将您的控件放在 arrays 中，这样您就可以执行以下操作：

for (index in number of forecasts) {
   groupBox[index].Text = set.Forecast[index]....;
   hiLabel[index].Text  = ...:
   lowLabel[index].Text = ...:
   ...
}

Answer 3

您是否尝试过对控件进行分组或动态分类控件？ 这样您就不必索引到预测列表中，您只需枚举集合，而不是为每次迭代使用不同的控件，您只需填充单个集合。

foreach (var item in set.Forecast.Take(3))
{
    var groupBox = new GroupBox(); // it don't if the class is actually called GroupBox...
    groupBox.Text = item.DayOfTheWeek;
    labelHigh.Text = string.Format("High {0}", item.High);
    labelLow.Text = string.Format("Low: {0}", item.Low);
    labelCondition.Text = string.Format("Condition: {0}", item.Condition);
    pictureBox.Load(item.Icon);
    groupBox.Controls.Add(labelHigh)
    ...
    this.Controls.Add(groupBox);
}

上面的代码只是给你一个想法。

实际上，go 的代码并没有太多错误，我唯一真正缺少的是断言您的索引没有超出范围。

Answer 4

这是我结束的解决方案（感谢@dyppl）。 为用户控制理念

public partial class forecastView : UserControl
{    
    public forecastView()
    {
        InitializeComponent();
    }

    public forecastView(int x, int y, int index,WeatherSet set)
    {
        InitializeComponent();

        label7.Text = string.Format("High:{0}", set.Forecast[index].High);
        label8.Text = string.Format("Low: {0}", set.Forecast[index].Low);
        pictureBox3.Load(string.Format("http://www.google.com/{0}", set.Forecast[index].Icon));
        groupBox1.Text = set.Forecast[index].DayOfTheWeek;

        this.Location = new System.Drawing.Point(x, y);
    }
}

我以这种方式加载它们

private void LoadControls(WeatherSet set)
{
    RemoveConrols();
    //form2.Dispose();

    form = new forecastView(12, 136, 1, set);
    form1 = new forecastView(155, 136, 2, set);
    form2 = new forecastView(12, 218, 3, set);

    this.Controls.Add(form);
    this.Controls.Add(form1);
    this.Controls.Add(form2);
}

所以感谢所有帮助我解决这个问题的人；）