Welcome to the BBS.
A nice start.

About collision :
In your code, player.x and player.y are the pixel coordinates of the top left of your player sprite.
You are testing in what tile that precise point of the player is and that is the tile considered in collision with the player. This is why the player in the screenshot isn't colliding with the spikes.