fr3nchk1ss / snaked Goto Github PK

CR by Ali

In general, adding two Coordinates is suspicious to me. However, subtracting two Coordinates makes sense and should return a Distance object.

CR by Ali

Accounting for the walls with +1, -1, etc. looks scary to me.
I wonder whether the playground could be detached from the walls and you could use the familiar 0 indexed access.

Perhaps by slicing inside a bigger area or by a special type that hides that logic from us.

interface pattern for console
version (win)
version(linux)

The POST-MORTEM will describe how the Dlang code matched the requirements.

CR by Ali

The invariant line 26 in ground.d might be a 'static assert':

(invariant has runtime cost.)

CR by Ali

A random thought about 'in' and 'out' contract blocks: Instead of adding statements inside these blocks, one can use functions to give a name to the check and also use the new contract syntax. Instead of

    in {
        // The snake shall not backtrack on itself
        assert(body[0] + unitMotion[direction] != body[1]);
    }

Something like this:

  bool updateBody(Direction direction)
  in (!doesBackTrack(direction), "Snake backtracks")
  {
    // The body of the function ...
  }

Describe in more details the purpose of the project