1.5 KiB
#interview #logos #pair-programming
Ricardo sent him the Minesweeper problem.
Wasn't early to the meeting. Showed up at 13:02
Console application in C#, limited GUI
Seems familiar with Visual Studio. Able to explain his basic plan to get going: two models (Cell and Board), one service to calculated neighboring cells.
Interesting choice: adding "NeighborCount" on the cell model--should the cell model know about its neighbors?
IN progress: ahs not initialized the Board property yet--how will he discover this?
Using Random to fill 10 mines into the board, knows basics of Random API.
Why does each Cell need to know its neighbor count? They need to show a number, when revealed, of how many neighbors are mines. He renamed the property accordingly.
Speaks English very well.
Found a bug he wrote in column and row check before executing program by thinking about it logically and visualizing the board.
Has a bug using .Length of a multi-dimensional array! Will be x * y and not x as he's hoping. He figured it out without any prompting from me and used GridSize in the check instead.
Questions:
- Can you GridSize everywhere to avoid errors?
- Does each Cell really need to know it's neighbor mine count? Or is that something the board knows about?
- Instead of a
(Row, Column)comment, is there another way we could label the values? - Is there a way to calculate the coordinates of the surrounding cells without statically constructing a list of offset pairs?
Nits:
- Property, field and variable naming not consistent