[MroczekBall3D]

witam koderzy, pytanie kieruję do tych w #csharp
Mianowicie, mam taki kod:

 int numOfRows = 0;
 foreach (var day in t.days)
 if (day.lessonsNum > numOfRows)
 numOfRows = day.lessonsNum;

i resharper pokazuje, że mogę go zmienić na

int numOfRows = t.days.Select(day => day.lessonsNum).Concat(new[] {0}).Max();

Pytanie brzmi, jakim cudem ma zwrócić mi wartość największą w liście t.days?
Oraz dlaczego w Concat podane jest new[] {0}
Z tego co wyczytałem, to List Concat. Two Lists can be combined. With the Concat extension method, we do this without a loop.
No to porównaj mi każdą wartość int, z ... 0 ? oraz zwróc wartość maxymalną..
Zastanawiam się, czy przypadkiem nie trzeba by było nawet
t.days.Select(day => day.lessonsNum).Max() skoro Select zwraca IEnumerable


#programowanie

[s.....a]

@MroczekBall3D: dodaje 0, bo najpierw porównujesz do numOfRows, któremu przypisałeś 0

[Yahoo_]

@MroczekBall3D: olej tą podpowiedź. Dużo mniej czytelne niż oryginalny zapis.

[s.....a]

@MroczekBall3D: var numOfRows = t.days.Max(day => day.lessonsNum);

[MroczekBall3D]

@slucham-psa-jak-gra: Teach me master ( ͡° ͜ʖ ͡°)
@Yahoo_: Myślałem, że resharper się nie myli :D

[Yahoo_]

@MroczekBall3D: To nie jest kwestia czy się myli czy nie :). Czytelność to kwestia mocno względna :). Można wyłączyć takie sugestie w opcjach R#.

[Yahoo_]

@slucham-psa-jak-gra: co zwraca Max dla pustej listy? default(T)?

[s.....a]

@Yahoo_: z tego co widzę to rzuca wyjątek
@MroczekBall3D: bo resharper się raczej nie myli ( ͡° ͜ʖ ͡°) zamieił kod tak, by odpowiadał twojemu :)

[Yahoo_]

@slucham-psa-jak-gra: to trzeba obsłużyć :P
@MroczekBall3D: var numOfRows = t.days.Count() > 0 ? t.days.Max(day => day.lessonsNum) : 0;

[s.....a]

@Yahoo_: var numOfRows = t.days.Max(day => day.lessonsNum) ?? 0; ( ͡° ͜ʖ ͡°)

[s.....a]

@Yahoo_: aaaa p------o mi sie

[s.....a]

@Yahoo_: dobrze napisałeś ;)

[Yahoo_]

@slucham-psa-jak-gra to gdzie mój plus ( ͡° ͜ʖ ͡°)

[KontoUsuniete]

@Yahoo_: imho czytelniej:

int numOfRows = t.days.Select(day => day.lessonsNum).DefaultIfEmpty(0).Max();

[Yahoo_]

@KontoUsuniete kwestia gustu :)
Twoja wersja ma plus, że jeśli t.Days byloby IEnumerable to nie ma ryzyka, że ewaluacja odbędzie się dwa razy. Ale nie chciało mi się wchodzić w takie szczegóły :)