Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestion of a different solution to Matrix #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Suggestion of a different solution to Matrix #75

wants to merge 1 commit into from

Conversation

ramiloif
Copy link

Maybe my solution will be a little easier to understand .
My solution uses the fact that we moving on the same pattern :
right -> down -> left -> up -> (->right ->....)
I created a location object in order to simplify the movement action ;
the object have row and col properties .
we are starting with location = {row:0, col:0}
Each direction can be easily translated to a similar object that represents a step:

const right = {row: 0, col:1};
const down = {row:1, col:0};
const left = {row :0, col:-1};
const up  = {row : -1 , col: 0};

The last part is to create an empty matrix fill it with 0 ( or null or whatever not in our numbers range )

now all left is to use counter until it gets to n*n
fill the location and move to the next legal location .
if the next location is not legal or we already been there ? we just change the direction to the next one in a cyclic way :)

Maybe my solution will be a little easier to understand .
@ramiloif ramiloif changed the title I'm suggesting a little different solution Suggestion for different solution to Matrix Nov 30, 2019
@ramiloif ramiloif changed the title Suggestion for different solution to Matrix Suggestion of a different solution to Matrix Nov 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant