Coding Review – How would you convert this ‘goto’ ?
This is a legacy code snippet that uses the magic goto, which I find it very difficult to understand and refactor. doit: foreach (var line in dict.Values) { var …
Category: code review
Should you use a magic number in unit tests?
Magic numbers are not good, we want to avoid them as much as we can. However, there is one exception, as described in the following C# example: public void …
Code Review – Bound Parameter
There is a piece of bad-smelling code, I happen to read today. The following aims to compute the angle between two line vectors. This code has been in the …
[Code Review] – Reinventing the Wheel
![[Code Review] – Reinventing the Wheel](https://helloacm.com/wp-content/uploads/2017/08/re-inventing-the-wheel-coding-review-linq-300x200.png "[Code Review] – Reinventing the Wheel")
This is a code change that I observe today. It has been in the codebase for a long time, and I find it interesting to talk about it. The …
Why and When should you do a Profiling on Your Application?
There was a time that a bug was assigned to me saying that my module was slow as hell. I did profile my module (Delphi Native Win32 DLL Module) …
