所以，这个程序可以正常运行，但我不认为代码非常干净，而且＃34;所以我正在寻找建议。我遇到的两个大问题是：
对于方法public double temperatureInFahrenheit，我想将参数celsiusTemperature传递给函数而不必重新声明变量，将其从文本转换为double等等。每当我尝试尝试我得到的MessageBox.Show中的一个错误，当我尝试调用函数myAirport.temperatureInFahrenheit时（我没有特别记住错误是什么，因此如果需要，我将不得不重新编码）。关于我能做些什么才能使这项工作的任何想法？
公共分部类Form1中的MessageBox.Show对我来说似乎是凌乱的代码。我想要做的是在内部类机器中编写一个方法，在那里将必要的参数传递给方法，然后执行类似MessageBox.Show（myAirport.message（））的操作，但是我可以使用它。我假设如果我尝试了，我会得到同样的错误，因为我有任何想法？
同样，代码功能齐全，可以工作并符合规定的要求，但我不喜欢只使用功能代码，我喜欢功能代码非常好。＆＃34;＃34 ;注意：我没有对任何方法，变量等进行评论。我现在正在对它们进行评论，但我认为我首先尝试获得一些反馈。
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
private void button1_Click(object sender, EventArgs e)
{
string airportName;
double celsiusTemperature, elevation;
if (String.IsNullOrEmpty(txtAirport.Text)) {
MessageBox.Show("You did not enter a name for the airport. Please enter a name for the airport.");
return;
} else
{
airportName = Convert.ToString(txtAirport.Text);
}
if (Double.TryParse(txtTemperature.Text, out celsiusTemperature))
{
if (celsiusTemperature > 50 || celsiusTemperature < -50)
{
MessageBox.Show("The value you entered for temperature is outside the acceptable range. Please reenter the information");
Application.Restart();
}
}
else
{
MessageBox.Show("You did not enter a numeric value for the temperature. Please enter a valid, i.e. numeric, value for the temperature.");
return;
}
if (Double.TryParse(txtElevation.Text, out elevation))
{
if (elevation > 12000 || elevation < -300)
{
MessageBox.Show("The value you entered for elevation is outside the acceptable range. Please reenter the information.");
Application.Restart();
}
}
else
{
MessageBox.Show("You did not enter a numeric value for the elevation. Please enter a valid, i.e. numeric, value for the elevation.");
return;
}
Airport myAirport = new Airport(airportName, celsiusTemperature, elevation);
MessageBox.Show("The airport name is: " + myAirport.airportName(airportName) + Environment.NewLine + "The Celsius temperature is: " + myAirport.celsiusTemperature(celsiusTemperature)
+ Environment.NewLine + "The Fahrenheit temperature is: " + myAirport.temperatureInFahrenheit(celsiusTemperature) + Environment.NewLine + "The elevation is: " + myAirport.elevation(elevation));
}
private void button2_Click(object sender, EventArgs e)
{
Close();
}
}
internal class Airport
{
private string airportName1;
private double celsiusTemperature1;
private double elevation1;
public Airport(string airportName1, double celsiusTemperature1, double elevation1)
{
this.airportName1 = airportName1;
this.celsiusTemperature1 = celsiusTemperature1;
this.elevation1 = elevation1;
}
public string airportName(string airportName1)
{
return airportName1;
}
public double celsiusTemperature(double celsiusTemperature1)
{
return celsiusTemperature1;
}
public double elevation(double elevation1)
{
return elevation1;
}
public double temperatureInFahrenheit(double celsiusTemperature1)
{
double fahrenheitTemperature = 0;
fahrenheitTemperature = celsiusTemperature1 * (1.8) + 32;
return fahrenheitTemperature;
}
}
}
答案 0 :(得分：1)
以下是如何简化Airport类：
public class Program
{
static void Main(string[] args)
{
var airport = new Airport { AirportName = "JFK", Temperature = 28.5 };
Console.WriteLine(airport.ToString());
}
}
public class Airport
{
private string _airportName;
private double _temperatureInCelsius;
private double _temperatureInFahrenheit;
public string AirportName
{
get
{
return _airportName;
}
set
{
if (string.IsNullOrWhiteSpace(value))
{
throw new Exception("You did not enter a name for the airport. Please reenter the information.");
}
_airportName = value;
}
}
public double Temperature
{
get
{
return _temperatureInCelsius;
}
set
{
if (value > 50 || value < -50)
{
throw new Exception("The value you entered for temperature is outside the acceptable range. Please reenter the information");
}
_temperatureInCelsius = value;
_temperatureInFahrenheit = _temperatureInCelsius *(1.8) + 32;
}
}
public override string ToString()
{
return string.Format(
"The airport name is: {0}\r\nThe Celsius temperature is: {1}\r\nThe Fahrenheit temperature is: {2}", _airportName, _temperatureInCelsius, _temperatureInFahrenheit);
}
}
请注意，AirportName setter会验证传递的机场名称（通过value），如果它无效，则会抛出异常。
我将离开另一个属性 - 海拔，作为你完成的练习。
关注Message.Show你可以这样做：
Message.Show(airport.ToString());
ToString()方法返回描述机场的string。
我的代码是一个建议，您不必完全使用它（即您可能不喜欢从setter中抛出异常。相反，您可以在创建Airport实例之前验证这些值。） ，但希望它会指导你。
答案 1 :(得分：0)
我认为，您应该通过以下方式重构代码：
机场似乎是一个数据类。所以它不应该知道Form1的TextBox。所以，试着将它们分开。
为了便于访问，您应该考虑机场类中字段的属性（或getter / setter方法）。
当您从机场课程中读取数据时，您正在检查数据。如果您以前从未阅读或使用过它，该怎么办？所以，你应该把它分开。
如果您估计错误，则表示您正在重新启动整个应用程序。这通常对用户来说不是一件好事。因此，请尝试显示错误并让用户进行一些更正。并再次检查，依此类推......
如果Airport是数据类，那么您可以拥有Celsius和Fahrenheit的属性。无论你有什么样的代表性，你都有。
我想在没有代码的情况下首先提供这些提示，因此在查看完整的解决方案之前，您可以自己思考并尝试。然后，如果你被困在某个地方，我（以及其他人）会给出更具体的提示。
所以，祝你好运......
答案 2 :(得分：0)
有很多方法可以分开不同的要求，所以我试着提出这个想法。
首先，一个简单的机场数据类，包含属性和内部验证（这意味着无效实例）：
internal class Airport
{
private string _airportName;
private double _celsiusTemperature;
private double _elevation;
public Airport(string airportName, double celsiusTemperature, double elevation)
{
this._airportName = airportName;
this._celsiusTemperature = celsiusTemperature;
this._elevation = elevation;
}
public string AirportName
{
get
{
return _airportName;
}
set
{
_airportName = value;
}
}
public double CelsiusTemperature
{
get
{
return _celsiusTemperature;
}
set
{
_celsiusTemperature = value;
}
}
public double Elevation
{
get
{
return _elevation;
}
set
{
_elevation = value;
}
}
public double TemperatureInFahrenheit
{
get
{
return _celsiusTemperature * (1.8) + 32.0;
}
set
{
if (value != 32.0)
{
_celsiusTemperature = (value - 32.0) / (1.8);
}
else
{
_celsiusTemperature = 0.0;
}
}
}
public bool IsValid(out string errorMessage)
{
bool result = false;
bool ok = true;
errorMessage = "";
if (String.IsNullOrEmpty(_airportName))
{
ok = false;
errorMessage = "You did not enter a name for the airport.";
}
if (_celsiusTemperature > 50 || _celsiusTemperature < -50)
{
ok = false;
errorMessage = "The value you entered for temperature is outside the acceptable range.";
}
if (_elevation > 12000 || _elevation < -300)
{
ok = false;
errorMessage = "The value you entered for elevation is outside the acceptable range.";
}
result = ok;
return result;
}
}
请注意，华氏度是基于摄氏度的计算值。反之亦然。
另请注意验证。机场类接受所有值，仅定义字段的类型。它进行语义检查（业务逻辑）。
现在，如何使用它：
private void button1_Click(object sender, EventArgs e)
{
string airportName;
double celsiusTemperature;
double elevation;
// Get data from controls and do syntactic checks
bool ok = true;
airportName = txtAirport.Text;
ok = Double.TryParse(txtTemperature.Text, out celsiusTemperature);
if (!ok)
{
// Error
MessageBox.Show("The value you entered for temperature is not a number!", "Error");
}
ok = Double.TryParse(txtElevation.Text, out elevation);
if (!ok)
{
// Error
MessageBox.Show("The value you entered for elevation is not a number!", "Error");
}
if (ok)
{
// Create the instance of the data class and do semantic checks
Airport myAirport = new Airport(airportName, celsiusTemperature, elevation);
string errorMessage;
if (!myAirport.IsValid(out errorMessage))
{
// Error
MessageBox.Show(errorMessage + " Please reenter the information", "Error");
}
else
{
// Ok, data is valid. Continue normal work...
MessageBox.Show("The airport name is: " + myAirport.AirportName + Environment.NewLine +
"The Celsius temperature is: " + myAirport.CelsiusTemperature + Environment.NewLine +
"The Fahrenheit temperature is: " + myAirport.TemperatureInFahrenheit + Environment.NewLine +
"The elevation is: " + myAirport.Elevation);
}
}
您从控件获取数据并进行语法检查。如果everthing是好的，那么让这个类自己进行语义检查。这里它将给出一个字符串作为消息详细信息，但许多方法是可行的。
因此，最后，如果发生合成错误，用户会收到消息并可以继续。如果发生语义错误，则用户会收到消息并可以继续。如果一切正常，您可以操作有效数据。
希望这会有所帮助......