CS/INFO 4152: Advanced Topics in Computer Game Development

Code Walkthroughs

Code walkthroughs are an important part of the software engineering process. As your group gets more an more used to your current software design decisions, everyone has a harder time seeing the problems. A code walkthrough allows a reviewer with fresh eyes to look at what you have done. The reviewer can tell you whether the code is understandable, and point out possible design problems.

Another advantage of the code walkthough is that it gives you a chance to compare your code to your architecture specification. You might think that the specification was an accurate representation of the code that you have written so far, but an outside observer might have other ideas.

We will structure this class like a discussion day, except that we will give it a lot more structure. We will pair groups with each other (and tripling up in one case). You will each present your code to each other. In addition, we will make sure that every pair has a dedicated programming TA with it at all times, in order to facilitate the discussion.


The Code Walkthrough

This class is closer to a presentation than a discussion section because you must come prepared a head of time. In particularly, you need to bring two things.

  • Several print-outs of your dependency diagram (nothing else) from your architecture specification.
  • A snapshot of your code on at least two laptops so that everyone can look at it.

Neither of these items requires that you create anything new for the lab. Your snapshot can either be the current state of your repository or your submission for alpha release. Your dependency diagram can be the one you submitted for document revisions. However, if you made more recent changes, you should bring the new version.

Please bring several copies of your dependency diagram, so that you can pass them about the other group. Four or so to share should be sufficient.


Walkthrough Format

Each group should take turns doing a walkthrough. In the time alloted, that means 20-25 minutes per group. The group making the presentation should have a single presenter by the computer with the code, while the reviewing team gathers around. If you want to swap out presenters during the walkthrough, that is okay. However, it might get crowded around the computer, so make sure that the reviewers can always see the code.

At the start of the presentation, hand out your dependency diagram to the other team so that they can look over it while you walkthrough the code. Then your presenter should show off the major classes in your code (e.g. the ones that correspond to those in the dependency diagram, not the minor classes like the North-East-South-West enumeration that you did not bother to write down). We recommend that you show off the classes in the following order.

  • The root controller class
  • Any controller subsystems (e.g. specialized controller classes)
  • The primary model class(es)
  • The primary view class(es)

Your presentation should show how they all fit together.


Reviewing the Code

While one group is presenting their code, the other group -- the reviewers -- should be frequently making comments and asking questions. In particular, the reviewers should ask themselves the following:

  • Is the code organization understandable?
  • How easy is it to break up the work among several people?
  • How extendable is the software architecture?
  • Do the class relationships match the dependency diagram?

We do not necessarily want the reviewers to ask the questions above to the presenter. Instead, reviewers should ask questions that help them form their own answers to the questions above. And if there answers indicate problems with the architecture design, the reviewers should point that out to the other team.


Submission

There is nothing to turn in for this assignment. You should simply take our comments on your architecture into consideration as you move forward with your design.