exercism configure 2>&1 | grep -- --api
returns blank
@sjwarner-bp https://gist.github.com/zantetsuken88/645a2f3ac2a4f12a732f4bce9947dd89
I've included the test file if you need it, and also the feedback conversation is included at the bottom as a comment.
So it is a good solution - most of the logic looks quite similar to our reference answer, so I'm happy that you're answering this quite well :slight_smile:
What is being recommended is as follows (please ask more questions if I don't explain well):
So, where the mentor says:
what if you store
int[][] matrix
instead ofString matrixAsString
?
At the top of your Matrix.java
, on line 3 you create a variable to store your matrixAsString
, which is passed in to the constructor, right? Then, every time that your getRow
method gets called you _re-_convert the matrix (line 10, for example) into the form of int[][]
in order to return your int[]
.
While this works, you shouldn't have to re-convert the matrix each time you want to get a row or column, after all it shouldn't change! So really, we could just do this once and store the output in a variable!