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.