System.InvalidCastException：无法将“System.Int32”类型的对象转换为“System.String”类型[关闭]

Question

任务是：给定一个正整数值序列integerList。 获取仅奇数 integerList 值的字符串表示序列并按升序排序 命令。 这是我的代码：

public static IEnumerable<string> Task5(IEnumerable<int> integerList)
    {
        var result = from item in integerList
                     where item % 2 != 0
                     orderby item descending
                     select item;
        return result.Cast<string>();
    }

但我每次尝试开始测试时都会看到此消息：System.InvalidCastException : Unable to cast object of type 'System.Int32' to type 'System.String'。

Answer 1

不要强制转换，使用 .ToString() 代替

  public static IEnumerable<string> Task5(IEnumerable<int> integerList)
    {
        var result = from item in integerList
                     where item % 2 != 0
                     orderby item descending
                     select item.ToString();
        return result;
    }

Answer 2

result 是 整数 的集合，而不是 字符串。而且您不能直接将整数转换为字符串。但是你可以打电话给他们.ToString()。因此，您可以使用 .Select() 之类的东西使用该操作将它们投影到新集合中。例如：

return result.Select(r => r.ToString());

尽管当源数据为整数时，此方法是否应该返回字符串集合可能值得研究。为什么不直接返回整数集合呢？看起来更干净：

public static IEnumerable<int> Task5(IEnumerable<int> integerList)
{
    return from item in integerList
           where item % 2 != 0
           orderby item descending
           select item;
}

很难确定，因为“Task5”并没有详细描述正在执行的操作。但是该方法是接收整数，看起来意图只是修改该集合，所以它会返回整数是有道理的。

如果出于此方法范围之外的某些目的需要字符串，则由消费代码将它们转换为字符串。

Answer 3

  public static List<string> Task5(IEnumerable<int> integerList)
    {
        var result = from item in integerList
                     where item % 2 != 0
                     orderby item ascending
                     select item;
        List<int> mylist = result.ToList();
        return mylist.ConvertAll(x => x.ToString());
    }


    public static IEnumerable<string> Task5(IEnumerable<int> integerList)
    {
        var result = from item in integerList
                     where item % 2 != 0
                     orderby item ascending
                     select item.ToString();
        return result;
    }

Answer 4

这听起来对我来说也绝对像是一个家庭作业……如果是这样，也许 Task5 这个名字是作业的要求。如果不是，那么正如@David 建议的那样，方法名称应该反映该方法执行的操作（例如FindOddNumbersAscending()）

以下是我认为基于作业描述的正确实施方式，也反映了上述反馈：

public static IEnumerable<string> FindOddNumbersAscending(IEnumerable<int> integerList)
{
    return from item in integerList
           where item % 2 != 0
           orderby item
           select item.ToString();
}

这可以稍微进一步防止空输入以及删除任何重复项（如果需要的话；注意切换到流畅的语法而不是查询语法来删除重复项）：

public static IEnumerable<string> FindOddNumbersAscending(IEnumerable<int> integerList)
{
    return integerList is null
        ? Enumerable.Empty<string>()
        : integerList
            .Where(item => item % 2 != 0)
            .Distinct()
            .OrderBy(item => item)
            .Select(item => item.ToString());
}