New answers tagged

2 votes
Accepted

Updating class method to use data from new weather API endpoint while keeping same return value object structure as with old endpoint

This submission is about refactoring code so it will return a same-format response. Yet it did not include an automated test suite which shows Green bar on "old" code and again Green on this ...
J_H's user avatar
  • 22.1k
2 votes
Accepted

"stray" point correction in 2D array

This submission is about performance. It does not include any profiler measurements. robust testing ...
J_H's user avatar
  • 22.1k
6 votes
Accepted

A fixed-size dynamic array

Note: although the question is tagged C++17, I'm going to use some C++20 or later features in my review, as this simplifies the code. Many of my suggestions can be written in C++17, but at the ...
Toby Speight's user avatar
  • 74.5k
2 votes
Accepted

Writing a multidimensional array view for contiguous arrays in C++20

Overall impression: well-written and well presented. It's pretty clear what's going on. I would have liked to have seen the unit tests, too - that generally helps reviewers. The constructor default-...
Toby Speight's user avatar
  • 74.5k
1 vote
Accepted

A dynamic array with an iterator compatible with std::sort

Before I get into the review, I need to offer some advice. std::vector is a type of dynamic array, but not the only type. The thing that makes ...
indi's user avatar
  • 10.9k

Top 50 recent answers are included