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

Brak definicji kierunków ruchu #97

Open
BartoszChrostowski opened this issue Mar 13, 2020 · 5 comments
Open

Brak definicji kierunków ruchu #97

BartoszChrostowski opened this issue Mar 13, 2020 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@BartoszChrostowski
Copy link
Collaborator

Kierunki ruchu nie są zdefiniowane, powinno być

N - ruch w stronę pozytywnych Y
S - ruch w stronę negatywnych Y
E - ruch w stronę pozytywnych X
W - ruch w stronę negatywnych X

@BartoszChrostowski BartoszChrostowski added bug Something isn't working help wanted Extra attention is needed labels Mar 13, 2020
@Rashair
Copy link
Contributor

Rashair commented Mar 15, 2020

To raczej dosyć intuicyjne

@BartoszChrostowski
Copy link
Collaborator Author

Patrząc od strony indeksów macierzy, ktoś mógłby zamienić N z S

@rottenmonday
Copy link

Czy zgodnie z #99 nie intuicyjniej by było to odwrócić? Na północ idziemy do góry, po wierszach (czyli N - ruch w stronę negatywnych X). I tak dalej.

@Rashair
Copy link
Contributor

Rashair commented Mar 18, 2020

To już raczej odwrócić tam,
y = wiersz = pierwsza współrzędna
x = kolumna = druga współrzędna

@BartoszChrostowski
Copy link
Collaborator Author

Czy zgodnie z #99 nie intuicyjniej by było to odwrócić? Na północ idziemy do góry, po wierszach (czyli N - ruch w stronę negatywnych X). I tak dalej.

Przez X Y miałem na myśli te z układu współrzędnych.
Po prostu chciałem zaznaczyć, żeby N inkrementowało współrzędną Y, a E inkrementowało współrzędną X.

Zmienię nazwy enumów na

incrementX
decrementX
incrementY
decrementY

Przy najbliższej okazji, bo to już będzie jednoznaczne, niezależnie czy ktoś robi (y,x) czy (x,y) czy w ogóle jeszcze inaczej

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants