4 ms·

I tend to run in to more critical issues than performance, like shipping the product/feature at all, meeting deadlines, writing maintainable code, and writing d

I tend to run in to more critical issues than performance, like shipping the product/feature at all, meeting deadlines, writing maintainable code, and writing documentation. In your highlighting example, instead of working with someone who will optimize, I'd much rather work with the guy who will realize we can just debounce the dialog, write a one line comment about the performance issue, and then move on to the next thing. Maybe I'm biased though. I've worked with the performance guy; he was brilliant and the code was clever and highly performant, but he was also slow and didn't write clear, maintainable code. We didn't ship in time and I had to find a new job when the project got scrapped. Damned if it wasn't fast though.

stcredzero 10y ago
We didn't ship in time and I had to find a new job when the project got scrapped. Damned if it wasn't fast though. Sounds more like a project management issue to me, than a problem with that coder.
- nulagrithom 10y ago
  Three man team and he was the lead. Yes, it was a project management issue. I'd rather see basic project management skills than basic knowledge of algorithms. Unfortunately, the former is much more rare.
attractivechaos 10y ago
> In your highlighting example, instead of working with someone who will optimize, I'd much rather work with the guy who will realize we can just debounce the dialog, write a one line comment about the performance issue, and then move on to the next thing. As long as you are aware of the issue with naive string concatenation, you can use a string buffer or a mutable string – it is trivial to solve in a few more lines of code. What's more difficult involves deletion in the previous text. If this happens often, you will need rope or skip list, which is challenging if you implement from scratch. In the latter case, I would leave a comment without implementing the optimal solution and let the profiler decide later.

stcredzero 10y ago

We didn't ship in time and I had to find a new job when the project got scrapped. Damned if it wasn't fast though. Sounds more like a project management issue to me, than a problem with that coder.

nulagrithom 10y ago
Three man team and he was the lead. Yes, it was a project management issue. I'd rather see basic project management skills than basic knowledge of algorithms. Unfortunately, the former is much more rare.

attractivechaos 10y ago

> In your highlighting example, instead of working with someone who will optimize, I'd much rather work with the guy who will realize we can just debounce the dialog, write a one line comment about the performance issue, and then move on to the next thing. As long as you are aware of the issue with naive string concatenation, you can use a string buffer or a mutable string – it is trivial to solve in a few more lines of code. What's more difficult involves deletion in the previous text. If this happens often, you will need rope or skip list, which is challenging if you implement from scratch. In the latter case, I would leave a comment without implementing the optimal solution and let the profiler decide later.