[英]find the highest number in an array made by the method
有一个 class 供乘客使用,其中包含string name
、 int age
、 string job
等。我制作了这个 class 的数组,假设它有 10 个位置。 我想找到最年长的乘客。
我的代码不起作用,因为无法将passenger[i]
与 integer 进行比较。 我的意思是我只需要passenger[]
的age
[]
如何在passenger[]
中找到最旧的?
编辑:返回值应该是他的name
和major
的passenger
,而不仅仅是它的age
。
public Passenger Oldest()
{
int oldest = 0;
for (int i = 0; i < passengers.Length; i++)
{
if (passengers[i] > oldest)
{
oldest = passengers[i];
}
}
return oldest;
}
class Passenger
{
int age;
string name;
string major;
public Passenger(int _age, string _name, string _major)
{
age = _age;
name = _name;
major = _major;
}
}
首先,正如@Cid 在评论中提到的,质疑Passenger
的所有字段都是private
的(默认情况下,当未指定修饰符时)。 您应该将 then 标记为public
(以在声明类之外访问它们)或更好地创建公共属性:
class Passenger
{
public int Age { get; set; } // public auto property
public string Name { get; set; } // public auto property
public string Major { get; set; } // public auto property
public Passenger(int age, string name, string major)
{
Age = age;
Name = name;
Major = major;
}
}
其次,您需要比较乘客的Age
(财产,而不是整个对象):
if (passengers[i].Age > oldest)
{
oldest = passengers[i].Age;
}
此外,您可以使用LINQ
找到最年长的乘客:
var oldest = passengers.Max(item => item.Age);
最后,返回最年长的乘客:
public Passenger Oldest()
{
// if no passengers -> return null
if (!passengers?.Any() ?? true)
{
return null;
}
var maxAge = passengers.Max(item => item.Age);
return passengers.First(item => item.Age == maxAge);
}
此外,正如@DmitryBychenko 所述,该方法可以缩短为:
public Passenger Oldest()
{
// if no passengers -> return null
if (!passengers?.Any() ?? true)
{
return null;
}
return passengers.Aggregate((s, a) => s.Age > a.Age ? s : a);
}
或没有LINQ
:
public Passenger Oldest()
{
// if no passengers -> return null
if (passengers == null || passengers.Length == 0)
{
return null;
}
var maxAge = passengers[0].Age;
var oldestPassengerIndex = 0;
for (var i = 1; i < passengers.Length; i++)
{
if (passengers[i].Age > maxAge)
{
oldest = passengers[i].Age;
oldestPassengerIndex = i;
}
}
return passengers[oldestPassengerIndex];
}
使用 Linq 很容易
var oldest=passengers.Max(x=>x.Age):
否则
Passenger oldest=new Passenger(0,"","");
foreach (Passenger p in Passengers){
if (p.age>oldest.age) oldest=p;
}
罗曼答案的稍微更有效的版本,特别是如果最年长的乘客出现在列表的最后:
public Passenger Oldest()
{
if (passengers.Length == 1) return passengers[0];
var oldest = passengers[0];
for (int i = 1; i < passengers.Length; i++)
{
if (passengers[i].Age > oldest.Age) oldest = passengers[i];
}
return oldest;
}
这只需要一次迭代。
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.