Метод:
private string GetValuesFromJObjectsArray<T>(IEnumerable<T> objects)
{
var type = typeof(T);
var list = new string[] { };
if (type == typeof(JProperty))
{
list = (objects as IEnumerable<JProperty>).Select(f => f.Name).ToArray();
}
else if (type == typeof(JToken))
{
list = (objects as IEnumerable<JToken>).Select(f => (f.Parent as JProperty).Name).ToArray();
}
return $"Correct values: { String.Join(", ", list) }. [ Total elements: {objects.Count()}]";
}
Метод работает корректно - он принимает коллекцию JProperty или JToken. Далее вытягивается Name элементов и преобразуется полученный масив в строку. К сожалению у этих двух объектов нету общих предков, поэтому мне пришлось напистаь else if.
Если бы вы проводили код ревью, на что Вы бы обратили внимание / что Вам не понравилось здесь?
Спрашиваю в целях самообразования
P.S. Я надеюсь я правильный тег прицепил ("инспекция кода")
Спасибо
jObjects
, потому чтоobjects
сильно коррелирует с мнемоническим названием типаobject