search

MINI-IO / IO-project-game

Alicja Moskal, Bartosz Chrostowski, Emil Dragańczuk, Jakub Drak Sbahi, Mikołaj Molenda
12 stars 3 forks source link

Brak definicji kierunków ruchu #97

Open BartoszChrostowski opened 4 years ago

BartoszChrostowski commented 4 years ago

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

Rashair commented 4 years ago

To raczej dosyć intuicyjne

BartoszChrostowski commented 4 years ago

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

rottenmonday commented 4 years ago

Czy zgodnie z https://github.com/MINI-IO/IO-project-game/issues/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 commented 4 years ago

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

BartoszChrostowski commented 4 years ago

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