views:

810

answers:

7

I'd be interested in some before-and-after c# examples, some non-idiomatic vs idiomatic examples. Non-c# examples would be fine as well if they get the idea across. Thanks.

+2  A: 

Idiomatic code is code that does a common task in the common way for your language. It's similar to a design pattern, but at a much smaller scale. Idioms differ widely by language. One idiom in C# might be to use an iterator to iterate through a collection rather than looping through it. Other languages without iterators might rely on the loop idiom.

Bill the Lizard
A: 

Just to get the ball rolling, this from Wiktionary:

Idiomatic: Pertaining or conforming to the mode of expression characteristic of a language.

So from what I gather, idiomatic code is code written in a manner of which the keepers of the language would approve, but that doesn't satisfy me somehow. I need some more insight into this.

MrBoJangles
+13  A: 

Idiomatic means following the conventions of the language. You want to find the easiest and most common ways of accomplishing a task rather than porting your knowledge from a different language.

non-idiomatic python using a loop with append:

mylist = [1, 2, 3, 4]
newlist = []
for i in mylist:
    newlist.append(i * 2)

idiomatic python using a list comprehension:

mylist = [1, 2, 3, 4]
newlist = [(i * 2) for i in mylist]
Corey Goldberg
+11  A: 

Some examples:

Resource management, non idiomatic:

StreamReader sr = File.OpenText(path);
string content = sr.ReadToEnd();
sr.Close();

Idiomatic:

string content;
using (StreamReader sr = File.OpenText(path)) {
    content = sr.ReadToEnd();
}

Iteration, non idiomatic:

for (int i=0;i<list.Count; i++) {
   DoSomething(list[i]);
}

Also non-idiomatic:

IEnumerator e = list.GetEnumerator();
do {
   DoSomenthing(e.Current);
} while (e.MoveNext());

Idiomatic:

foreach (Item item in list) {
   DoSomething(item);
}

Filtering, non-idiomatic:

List<int> list2 = new List<int>();
for (int num in list1) {
  if (num>100) list2.Add(num);
}

idiomatic:

var list2 = list1.Where(num=>num>100);
JacquesB
please, please fix the typo, it should be "idiomatic" :P
Peter Thomas
+2  A: 

Practically speaking, it means writing code in a consistent way, i.e. all developers who work on your code base should follow the same conventions when writing similar code constructs.

So the idiomatic way is the way that matches the style of the other code, non-idiomatic way means you are writing the kind of function but in a different way.

e.g. if you are looping a certain number of items, you could write the loop in several ways:

for (int i = 0; i < itemCount; i++)

for (int i = 1; i <= itemCount; i++)

for (int i = 0; i < itemCount; ++i)

etc

What is most important is that the chosen style is used consistently. That way people become very familiar and confident with how to use it, and when you spy a usage which looks different it can be a sign of a mistake being introduced, perhaps an off by one error, e.g.

for (int i = 1; i < itemCount; i++)
RickL
I guess I thought idiomatic meant "the best way" as opposed to just an arbitrary standard. But I guess a best way is subject to context.
MrBoJangles
A: 

Eric Lippert blogs about 'Big Words' we sometimes encounter. He does not address this particular word but its a great place to place a suggestion on his posts..

Vitaliy
A: 

In PHP I sometimes encounter code like:

foreach ($array as $value) {
    $trimmed[] = trim($value);
}
return $trimmed;

Which idiomatically can be implemented with:

return array_map('trim', $array);
jmoz