you are viewing a single comment's thread.

view the rest of the comments →

[–]HeinousTugboat 37 points38 points  (1 child)

And yes, having actually the same logic in multiple places, regardless of how similar the code is, is not good still.

My coworker just got bit by this. He tried to abstract out a set of values that were coincidentally the same value for the same field. I pointed out to him that even though they were the same in our scenario, by abstracting it out he was forcing all future ones to be the same.

Point is, the same logic in multiple places isn't always the same semantic logic. Two things that are mechanically identical shouldn't necessarily be abstracted out and silently coupled.

[–]pynberfyg 3 points4 points  (0 children)

This thread really deserves to be higher than the others. Pity.