2048 - broken

Rate this contribution

+23

Comments

20 Comments

over 3 years ago

Also printing actions with another symbol crashes the game i.e. try "U D" The game has crashed. Please contact the author and enclose the following error: java.lang.ArrayIndexOutOfBoundsException: Index -1 out of bounds for length 4 at engine.Board.applyMove(Board.java:77) at engine.Board.canMove(Board.java:62) at engine.Board.playTurn(Board.java:122) at com.codingame.game.Referee.gameTurn(Referee.java:61) at com.codingame.gameengine.core.GameManager.start(GameManager.java:122) at com.codingame.gameengine.core.RefereeMain.start(RefereeMain.java:67) at com.codingame.gameengine.core.RefereeMain.main(RefereeMain.java:50)

eulerscheZahl Unnamed CodinGamer

over 3 years ago

You were not supposed to put spaces in there ;) But I updated the game to ignore everything except for UDRL, thanks for reporting.

wlesavo Unnamed CodinGamer

over 3 years ago

Wow, you actually find a way to do it. But in this way with multiple actions being output in a single turn there is no way to account for new tiles which would appear in between actions. I would propose to make it appear by some rule which would be described in the beginning. i.e. xor shift seed from which you will be able to get index of empty position to put tile and its value (2 or 4). This would allow nice simulations and would make a game much more competitive from my point of view.

wlesavo Unnamed CodinGamer

over 3 years ago

oh, nwm, didn't see that you actually do give out the seed and the generator. My bad.

GenesisQ Unnamed CodinGamer

over 3 years ago

I played it through again without noticing, such a well made game

Thyl Unnamed CodinGamer

over 3 years ago

Hello, I think : It's a very good game !! Is nice for the optimization games

trictrac Unnamed CodinGamer

over 3 years ago

Hello eulerscheZahl, Very good idea this proposal. I have a very strange behaviour on the 2 Navigators I tried (Firefox & Chrome) On Firefox, after launching default code "U" on "random test", it takes a very long time allocating memory before replay : for only 4 valid moves (but 601 in the replay memory). For the same test on Chrome : freeze of the navigator. I am on Linux Fedora.

eulerscheZahl Unnamed CodinGamer

over 3 years ago

Thanks for reporting. I don't understand why it happens but I can confirm that it does (Ubuntu + Chromium). I'll investigate. I should also choose a shorter default replay as loading the puzzle takes some time.

luke-guyton Unnamed CodinGamer

over 3 years ago

Perhaps more accurate to say the game ends after 600 game turns, rather than commands (which could be confused to mean the same as actions). Would be great to see Win and Lose sections like seen in most games. Is there any limit to the number of actions possible per game turn, e.g. 1000, 10000, 100000 ?

luke-guyton Unnamed CodinGamer

over 3 years ago

Also are there any rules such as "an invalid action will cancel any remaining actions provided that game turn" ?

WildSmilodon Unnamed CodinGamer

over 3 years ago

Hey, great contribution. How about considering removing the "Invalid action" when you try to move in a direction, where no tiles are moved. It would make algorithms without simulation easier for beginners.

Illedan Unnamed CodinGamer

over 3 years ago

Agreed. And also, send the current seed each round?

chouch Unnamed CodinGamer

over 3 years ago

hey ! the github link for the pseudo-random number generator does not seem to be good... One too many Board !? Otherwise i played a little and came across 2 things : - random does not seem random. Default code produce every time the same result ! Maybe i missed something ? - an exception. Looks like it happened when i did not provide an output in due time ! Le jeu a cessé de fonctionner. Veuillez contacter l'auteur(e) et transmettre l'erreur suivante : java.lang.NullPointerException at engine.Board.cache(Board.java:129) at com.codingame.game.Referee.gameTurn(Referee.java:50) at com.codingame.gameengine.core.GameManager.start(GameManager.java:122) at com.codingame.gameengine.core.RefereeMain.start(RefereeMain.java:67) at com.codingame.gameengine.core.RefereeMain.main(RefereeMain.java:50)

eulerscheZahl Unnamed CodinGamer

over 3 years ago

Thanks. Github link: true, I messed that up (linked before creating the repo) Random testcase: I blame CodinGame, see https://github.com/... (at first there was only 1 testcase with a fixed seed, then I made the first one random) Crash: I'll look into it. (edit: fixed)

ABOUT US CAREERS PRIVACY FAQ BLOG

The #1 tech hiring platform

SCREENING INTERVIEWING