哪种方法风格更好？ 在方法中修改变量通常是不好的做法吗？
public class Person
{
public string Name { get; set;}
}
//Style 1
public void App()
{
Person p = new Person();
p.Name = GetName();
}
public string GetName()
{
return "daniel";
}
//Style 2
public void App()
{
Person p = new Person();
LoadName(p)
}
public void LoadName(Person p)
{
p.Name = "daniel";
}
【问题讨论】：
标签： c# methods parameter-passing
有时两种风格都可能有意义。例如，如果您只是设置名称，那么您可能会选择第一种样式。不要将对象传递给方法来改变一件事，只需检索一件事。这种方法现在作为一个附带好处更可重用。把它想象成得墨忒耳法则或最少知识原则。
在其他情况下，您可能需要根据用户输入进行批量更新。如果您要显示一个人的属性并允许用户进行修改，那么您可能会将对象传递给一个方法，以便可以在一个位置应用所有更新。
任何一种方法都可以在不同的时间得到保证。
【讨论】：
我认为当方法不改变传递的对象时，代码更清晰易读。尤其是传递对象的内部字段。
有时可能需要这样做。但总的来说我会避免它。
根据评论更新（好点）
【讨论】：
我同意安东尼的回答。 There are times when both styles may make sense.
此外，为了提高可读性，您可以在 person 类中添加 LoadName 函数。
public void App()
{
Person p = new Person();
p.LoadName(); //If you need additional data to set the Name. You can pass that as Parameter
}
【讨论】：
您正在使用技术上通过方法的属性访问数据。您担心的是访问 iVar 或内部变量的属性。允许访问 iVar 通常不好的原因是，任何人都可以在您不知情或未经您许可的情况下修改变量，如果它通过方法（属性），您可以在获取或设置消息时拦截它，或阻止它被读取或写入，因此它通常被认为是最佳实践。
【讨论】：
我同意罗恩的观点。尽管出于发布原因，您的特定示例可能会略微做作，但我将有一个用于 Name 的公共 getter 和一个私人 setter。将名称传递给构造函数，Name 属性将在那里设置，但之后无法再修改。
例如：
public class Person
{
public string Name { get; private set; }
public Person( string name)
{
Name = name;
}
}
【讨论】：
FirstName 怎么样？ :)