Início / coding / Perguntas / 79438517
Accepted
beyarkay
Asked: 2025-02-14 14:17:19 +0800 CST

Avisos do compilador C# sobre conversões implícitas no loop foreach (IDE0220)

  • 772

Como um exemplo simplificado do meu problema, tenho um array de ISomeType, e quero fazer um loop sobre todos os elementos naquele array que são realmente MyType, mas estou recebendo avisos IDE0220 "Add explicit cast in foreach loop" que não acho que se apliquem. Aqui está um exemplo de código:

public interface ISomeType {
    void DoThing();
}

public class MyType: ISomeType {
    void DoThing() { /* do something */ }
    void DoMyTypeThing() { /* do something specific to MyType */ }
}

public class YourType: ISomeType {
    void DoThing() { /* do something */ }
    void DoMyTypeThing() { /* do something specific to MyType */ }
}

ISomeType[] maybeMyTypes = [new MyType()];


// I get the error on this line because I cast the element into `MyType`
foreach (MyType foobar in maybeMyTypes.Where(i => i.GetType() == typeof(MyType))) {
    // use the MyType methods availbale on foobar
}

O compilador reclama que ele converte implicitamente os elementos de maybeFooBarsem MyType, e que isso pode falhar em tempo de execução, então devo ser explícito sobre a conversão:

// Code with violations.
var list = new List<object>();
foreach (string item in list) { }

// Fixed code.
var list = new List<object>();
foreach (string item in list.Cast<string>())

Meu código poderia realmente falhar em tempo de execução, já que estou verificando o tipo e apenas fazendo cast implicitamente se o tipo estiver correto? Ou o compilador C# não é inteligente o suficiente para ver que eu me protegi contra os tipos estarem incorretos?

c#

2 respostas

  1. Best Answer

    .Where(i => i.GetType() == typeof(MyType))não dirá ao compilador que você está lidando apenas com MyTypemembros. Para isso, você vai querer OfTypeem vez disso. Então você vai usá-lo como

    foreach (MyType foobar in maybeMyTypes.OfType<MyType>()) {
    // use the MyType methods availbale on foobar
}
    • 3

  2. GetTypeé uma operação de tempo de execução , não ajudará o compilador a inferir o tipo correto. Então, para o compilador , sua Whereinstrução retorna apenas uma lista de ISomeType. Então você recebe o aviso de que precisa converter o tipo para o derivado.

    Além disso, sua instrução - selecionaráWhere apenas os itens que correspondem à condição. Ela não transformará seus itens para o tipo desejado. apenas retorna exatamente o mesmo tipo de coleção que recebeu como parâmetro. Então, se a coleção de entrada para for do tipo , isso se aplica à saída também.WhereWhereISomeType

    Então, afinal, o que você quer é selecionar esses objetos cujo tipo é o desejado e convertê-los apropriadamente. Então, use OfTypeem vez disso, que é semelhante a executar um as-cast e retornar aqueles cujo valor de retorno não é null. Alternativamente, você pode usar Cast<MyObject>, no entanto, isso lhe dará um InvalidCastExceptionse houver algum elemento que não seja desse tipo.

    • 2

relate perguntas