Citat:
Ursprungligen postat av
Binary
Du missar poängen. Jag använder funktionell stil så ofta som möjligt. Det ger mig mer lättläst, och mindre kod med färre buggar. T.ex. använder jag mycket Linq, det är en form av funktionell programmering.
En challenge: skriv om följande kod till klassisk, imperativ stil så jämför vi sedan?
Kod:
List<Tuple<string, string>> tuples = //
var grouped = tuples.GroupBy(t => t.First)
.OrderBy(grp => grp.Key)
.Select(grp => new { Key = grp.Key, Items = grp.OrderBy(t => t.Second) });
Den stora charmen med detta är att jag inte skapar en massa egna tillstånd i programmet med olika variabler. I stället säger jag vad jag vill ha och då får jag det. Det finns färre möjligheter att göra fel och jag ser mycket snabbare vad koden gör.
Nu kanske just detta är ett exempel där linq passar väldigt bra, men i många fall så inser man att man kan optimera och ta bort minst en loop om man skriver koden utan linq.
Men vi kan testa. Jag skrev om ditt program så att det fungerar:
Kod:
List<Tuple<string, string>> tuples = new List<Tuple<string, string>>();
tuples.Add(Tuple.Create("1", "pelefant"));
tuples.Add(Tuple.Create("1", "apa"));
tuples.Add(Tuple.Create("2", "jagular"));
tuples.Add(Tuple.Create("2", "platypus"));
tuples.Add(Tuple.Create("3", "unicorn"));
var result = tuples.GroupBy(x => x.Item1)
.OrderBy(grp => grp.Key)
.Select(grp => new { Key = grp.Key, Items = grp.OrderBy(t => t.Item2) });
foreach(var r in result)
{
foreach (var i in r.Items)
{
Console.WriteLine($"{i}");
}
}
Och här är samma kod i imperativt format villket blev 26 rader i stället för 19 rader.
Kod:
List<Tuple<string, string>> tuples = new List<Tuple<string, string>>();
tuples.Add(Tuple.Create("1", "pelefant"));
tuples.Add(Tuple.Create("1", "apa"));
tuples.Add(Tuple.Create("2", "jagular"));
tuples.Add(Tuple.Create("2", "platypus"));
tuples.Add(Tuple.Create("3", "unicorn"));
Dictionary<string, List<Tuple<string, string>>> groupedTuples = new Dictionary<string, List<Tuple<string, string>>>();
foreach (var tuple in tuples)
{
if (!groupedTuples.ContainsKey(tuple.Item1))
{
groupedTuples[tuple.Item1] = new List<Tuple<string, string>>();
}
groupedTuples[tuple.Item1].Add(tuple);
}
foreach (var key in groupedTuples.Keys.OrderBy(k => k))
{
var sortedTuples = groupedTuples[key].OrderBy(t => t.Item2);
foreach (var tuple in sortedTuples)
{
Console.WriteLine($"{tuple}");
}
}
Nu tycker jag egentligen inte att linq är det värsta som C# har att erbjuda.
De kunde ha kallat select för map i stället och haft reduce, dvs samma som javascript.
Men så bra skulle det ju inte vara.
Det jag stör mig på mest är folk som sitter och hittar på egena funktionella syntaxer
t.ex. genom att skapa en massa olika statiska metoder. Jo det är ju fint, men jag kan ju inte ditt
funtionella språk du har hittat på. Linq är ju i alla fall en del av språket och något man kan förvänta
sig att folk som sätter sig in i ett nytt projekt kan grunderna i.
