Hacker News new | past | comments | ask | show | jobs | submit login

If you have a 95% match on something nontrivial (and it likely won't diverge significantly), I'd go for merging even with 2 cases. At least merge most of the common parts.

Reading a couple of ifs, and some not-quite duplicate procedures seems much better than having a complete 2-set in cross-refenenced files.




Why are you reading a couple of ifs instead of having the two similar things represented by separate classes with shared functionality in a common class? Or even if you prefer composition to inheritance you could still make it work cleaner without a bunch of if statements.


With a 95% match, you really only have one use case, with some minor variation.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: