osipukr / net-training-tasks Goto Github PK

1. В этом методе происходит boxing/unboxing (https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/types/boxing-and-unboxing), что влияет на производительность. Одно из преимуществ generics как раз в том, чтобы избегать boxing/unboxing, и оно тут не используется.
2. Для вызова функции можно опускать Invoke
3. ascending.GetHashCode() - это неверный код, тесты не должны были пустить такое. В С# этот метод возвращает 0 для false и 1 для true, и это значит, что лямбда выражение для false всегда будет возвращать 0, и массив просто не отсортируется. (https://docs.microsoft.com/en-us/dotnet/api/system.boolean.gethashcode?view=netframework-4.7.2)

Выражения вида data.Count().Equals(0) для сравнения чисел выглядят вычурно. data.Count()==0 намного читабельней.

RTFM http://csharpindepth.com/Articles/General/Singleton.aspx

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L195

Тут функция возаращает последних 3 элемента, а по условию надо 3 наибольших.

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L459

А зачем тут Where(x => data.All(y => y.Contains(x))) и OrderBy(x => x) ?

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L120

Это же можно проще написать: (val, idx) => idx % 2 == 1

Тут тоже можно убрать Invoke:

https://github.com/fredstone624/Net-Training-Tasks/blob/5b9322843d6004ac00351a3a38867c4625cdde39/02-Generics/Generics/Generics.cs#L174

https://github.com/fredstone624/Net-Training-Tasks/blob/5b9322843d6004ac00351a3a38867c4625cdde39/02-Generics/Generics/Generics.cs#L211

https://github.com/fredstone624/Net-Training-Tasks/blob/3c01cbc2697b5898e135517a9b0ecb13e0a3940b/05-LinqToXml/LinqToXml/LinqToXml.cs#L86-L92

Тут должно быть еще проще: return XElement.Parse(xmlRepresentation).Value;

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L401

Опять немного вычурно, можно проще, учитавая, что string - это IEnumerable :
var digits = "0123456789";

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L105

При сравнении строк использование ToLower\ToUpper не очень хороший подход. Каждый метод ToLower/ToUpper создает новую копию строки с измененнным регистром, и потом эти созданные строки надо будет удалять сборщику мусора.
Оптимальным подходом является сравнение (поиск) с помощью методов, которые позволяют указать значение типа StringComparision, в этом случае не происходит создания новых строк. См https://docs.microsoft.com/en-us/dotnet/csharp/how-to/compare-strings#case-insensitive-ordinal-comparisons

https://github.com/fredstone624/Net-Training-Tasks/blob/5f7b4eadbdc4fff8a77bd738e7555679ce5575a1/04-Enumerable/Enumerable/Task.cs#L383
Тут ToList() лишнее. ToList\ToArray загружает все данные из IEnumerable и такого поведения надо стараться избегать.