Skip to main content
adicionou 57 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, por isto não postei, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é em um TryParse(). Também pode tirar o pattern matching ee tornar o código mais verboso, só não acho que faça sentido evitar essas coisas mais.

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, por isto não postei, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse(). Também pode tirar o pattern matching e tornar o código mais verboso.

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, por isto não postei, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é em um TryParse(). Também pode tirar o pattern matching e tornar o código mais verboso, só não acho que faça sentido evitar essas coisas mais.

adicionou 93 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, por isto não postei, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse(). Também pode tirar o pattern matching e tornar o código mais verboso.

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse().

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, por isto não postei, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse(). Também pode tirar o pattern matching e tornar o código mais verboso.

adicionou 246 caracteres ao conteúdo
Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse().

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura.

A primeira grande mudança é tornar o método genérico e não vazar abstração.

Outra mudança é que eu juntaria os dois métodos em um só usando tuplas evitando exceções, mudanças entre a verificação e o uso, fora a duplicidade de código. Só pode usar o nome de o booleano for verdadeiro, caso contrário ele é inválido. Fiz isto porque o uso indicava que esta era a intenção.

Não entendi algumas coisas deste código, inclusive fazer um loop dentro de outro loop para varrer os mesmos dados. Então simplifiquei isto.

Fiz uma validação porque o que for passado pode não ser uma enumeração. Pena que o C# não deixa restringir isto no código em tempo de compilação. Aí faz sentido ser exceção porque é erro de programação. Nunca capture ela.

Fui direto nos membros do tipo e não na enumeração para simplificar.

Melhorei os nomes das variáveis e do próprio método.

Também usei pattern matching.

Simplifiquei outras coisas.

Eu faria assim:

using System;
using static System.Console;
using System.Xml.Serialization;

public class Program {
    public static void Main() {
        (var ok, var nome) = EnumFromXmlAttribute<Velocidade>("02");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
        (ok, nome) = EnumFromXmlAttribute<Velocidade>("05");
        if (ok) WriteLine($"O resultado é: \"{nome}\"");
    }
    private static (bool, T) EnumFromXmlAttribute<T>(string texto) {
        var type = typeof(T);
        if (!type.IsEnum) throw new ArgumentException("O tipo precisa ser uma enumeração");
        foreach (var item in type.GetFields())  {
            if (Attribute.GetCustomAttribute(item, typeof(XmlEnumAttribute)) is XmlEnumAttribute attribute && texto == attribute.Name)
                return (true, (T)item.GetValue(null));
        }
        return (false, default(T));
    }
}

[System.SerializableAttribute()]
public enum Velocidade {
    [XmlEnumAttribute("01")] Baixa,
    [XmlEnumAttribute("02")] Normal,
    [XmlEnumAttribute("03")] Rapida,
}

Coloquei no GitHub para referência futura. Como usa recursos mais novos, esses IDEs online não compilam este código, mas testei no VS e está ok.

Se não quiser usar tupla pode usar um parâmetro out, assim como é um TryParse().

Fonte Link
Maniero
  • 484,4mil
  • 94
  • 1,3mil
  • 2,2mil
Carregando