Início / coding / Perguntas / 77706671
Accepted
José Ramírez
Asked: 2023-12-23 13:01:01 +0800 CST

Por que a exceção não é lançada a menos que o enumerável retornado seja enumerado e como alterar esse comportamento? [duplicado]

  • 772

Ok, preciso de algum mestre .Net para me dizer por que a exceção no código a seguir só é lançada quando o valor de retorno é enumerado.

       public IEnumerable<string> BuildFor(Environment environment)
        {
            if (environment == Environment.None)
            {
                throw new Exception($"Impossible to build for environment '{environment}'.");
            }
            string clientId = ClientIds.Api[environment];
            foreach (string scope in scopes)
            {
                yield return $"api://{clientId}/{scope}";
            }
        }

Ok, correção: preciso de confirmação do que está acontecendo. A evidência está me dizendo que toda a chamada do método está sendo adiada, e isso é, provavelmente, uma otimização do compilador.

A pergunta seguinte: Como posso dizer ao compilador para parar de fazer isso?

c#

2 respostas

  1. Se você não quiser uma avaliação preguiçosa, basta escrever o método da maneira "tradicional", ou seja

    public List<string> BuildFor(Environment environment)
{
 List<string> l = new List<string>();

 if (environment == Environment.None)
 {
     throw new Exception($"Impossible to build for environment '{environment}'.");
 }
 string clientId = ClientIds.Api[environment];
 foreach (string scope in scopes)
 {
    l.Add($"api://{clientId}/{scope}");
 }

 return l;
}

    Ou você pode garantir que a avaliação ocorra no local assim:

    List<string> l = BuildFor(Environment.Env1).ToList();

    O compilador não tem nada a ver com isso.

    • 1
  2. Best Answer

    Uma opção seria tornar o iterador real um método privado chamado de dentro do método que lança a exceção. O código a seguir é muito parecido com o seu e não lançará uma exceção até entrar no segundo foreachloop do Mainmétodo:

    public static void Main()
{
    var items = GetItems(new object());

    foreach (var item in items)
    {
        Console.WriteLine(item);
    }

    items = GetItems(null);

    foreach (var item in items)
    {
        Console.WriteLine(item);
    }
}

public static IEnumerable<string> GetItems(object obj)
{
    var items = new[] { "First", "Second", "Third" };

    if (obj == null)
    {
        throw new ArgumentNullException(nameof(obj), "Argument cannot be null.");
    }

    foreach ( var item in items )
    {
        yield return item;
    }
}

    Esta variação funciona da mesma maneira no que diz respeito à iteração, mas será lançada na segunda GetItemschamada sem esperar pela enumeração:

    public static void Main()
{
    var items = GetItems(new object());

    foreach (var item in items)
    {
        Console.WriteLine(item);
    }

    items = GetItems(null);

    foreach (var item in items)
    {
        Console.WriteLine(item);
    }
}

public static IEnumerable<string> GetItems(object obj)
{
    if (obj == null)
    {
        throw new ArgumentNullException(nameof(obj), "Argument cannot be null.");
    }

    return GetItemCore(obj);
}

private static IEnumerable<string> GetItemsCore(object obj)
{
    var items = new[] { "First", "Second", "Third" };

    foreach (var item in items)
    {
        yield return item;
    }
}

    Conforme sugerido nos comentários, se a sua versão do C# suportar, você poderá usar um método local em vez de um método de membro privado. Basicamente, é apenas mover o método privado para dentro do método público e remover o private static. Isso fornece a mesma funcionalidade, mas evita que esse método extra seja acessado em qualquer outro lugar, de modo que o método público ainda seja independente, como era originalmente.

    public static IEnumerable<string> GetItems(object obj)
{
    if (obj == null)
    {
        throw new ArgumentNullException(nameof(obj), "Argument cannot be null.");
    }

    return GetItemsCore(obj);

    IEnumerable<string> GetItemsCore(object obj)
    {
        var items = new[] { "First", "Second", "Third" };

        foreach (var item in items)
        {
            yield return item;
        }
    }
}
    • 1

relate perguntas