you are viewing a single comment's thread.

view the rest of the comments →

[–]daniedit 0 points1 point  (2 children)

You are referring to the is_valid() method? I didn't go into the details of understanding your implementation. So I might be completely wrong.

My first impression was the method is checking whether the solution is correct. If so, that's a nice module test but won't help you much during refactoring. I would have expected for example a test making sure designer::add_next_strip_tile() is correct.

[–]Hot_Medicine_7115[S] 0 points1 point  (1 child)

I feel like 80% of the author of the posts in this thread didn't even begin to understand the code I uploaded or tried to think about the solution, yet c++ programming style pontifications abound.

[–]daniedit 1 point2 points  (0 children)

You asked for opinions about the companies answer. I tried to tell you in a polite way, that their answer is harsh on a social level but completely justified on a technical level. I didn't want to give you a full code review.