14
\$\begingroup\$

Design questions are ones that deal with architecture, package layout, object hierarchies, etc., but do not necessarily include code.

We already deal with this sort of question on a daily basis at programmers. While a certain amount of overlap is inevitable (and desirable, see this Stack Exchange blog post), this could be a big subject.

Should on-topic be limited to actual code, or can design be reviewed here as well?

\$\endgroup\$

3 Answers 3

15
\$\begingroup\$

In my opinion, Code Review should strive to create cleaner, more readable, "better" code and architecture / design / design patterns are a part of that.

switch (object.type): case "Foo": doX(); break case "BAR" doY(); break; .....

is something that could be solved using "design" (inheritance).

Also I would like to be able to post question like "I've build this object hierarchy and now it bites me because X, Y and Z. And by god I have no idea how to refactor that into something that changes me problems".

The same thing that I would to in a real life code review when going over some new module.

\$\endgroup\$
1
  • 3
    \$\begingroup\$ +1. I think Design Review is a part of Code Review - just a "big picture view", that is all.And I have a question along these lines that I was thinking of asking! \$\endgroup\$
    – winwaed
    Jan 21, 2011 at 3:25
10
\$\begingroup\$

The Code Review Stack Exchange is for sharing code from projects you're working on to get peer feedback and review. There may be rare exceptions, but the bulk of questions will likely contain actual code you are working on.

The Software Engineering Stack Exchange is for "white board" programming questions. Higher-level design and architecture questions should be posted to Software Engineering SE.

Incidently, questions about the discipline and practice of code reviews should be posted to Software Engineering SE. If you see any questions about conducting code reviews (best practices, techniques, etc), those questions should be closed and the author kindly directed to the Software Engineering SE site.

\$\endgroup\$
3
  • 1
    \$\begingroup\$ This should be in the FAQ. \$\endgroup\$ Jan 21, 2011 at 17:22
  • \$\begingroup\$ @Mark Loeser: Done. codereview.stackexchange.com/faq \$\endgroup\$ Jan 22, 2011 at 2:02
  • \$\begingroup\$ @200_success, along with the site name there has been a major scope redefinition. Aren't you concerned that, by editing just the site name part of the answers here, you may actually make them mean something that they actually don't? \$\endgroup\$
    – Marc.2377
    Oct 27, 2019 at 0:50
7
\$\begingroup\$

I personally think that code should be involved. Anything that is purely a design question should go to Software Engineering.

\$\endgroup\$

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .