How should I handle collision detection for a Zelda clone?

Alright, so I'm working on Phantasy Blade again, and I need a new way to do collision detection because the way I'm doing it isn't working too well.

This is how I'm doing it:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17

public boolean mapCheck(Map m, float x, float y, float xmod, float ymod)
   {
      if( (int) ((x/m.tileWidth) + xmod) >= 0 && 
         (int) ((y/m.tileHeight) + ymod) >= 0 &&
         (int) ((x/m.tileWidth) + xmod) < m.width && 
         (int) ((y/m.tileHeight) + ymod) < m.height)
      {
         if(m.tileInfo[m.tiles[(int) ((x/m.tileWidth) + xmod)][(int) ((y/m.tileHeight) + ymod)]] == 0){
            return false;
         }
         
         else
            return true;
      }

      return false;
   }

And this is how I move:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63

public float moveUp(Map m, float worldY){
      if(!mapCheck(m, (float) x, (float) y, (float) 0, (float) -0.1)){
         y -= speed;
         worldY -= speed;
      }
      
      else{
         dashing = false;
         dashDelay = 30;
      }
      
      state = 0;
      walking = true;
      return worldY;
   }
   
   public float moveDown(Map m,float worldY){
      if(!mapCheck(m, (float) x, (float) y, (float) 0, (float) 0.5)){
         y += speed;
         worldY += speed;
      }
      
      else{
         dashing = false;
         dashDelay = 30;
      }
      
      state = 2;
      walking = true;
      return worldY;
   }
   
   public float moveLeft(Map m,float worldX){
      if(!mapCheck(m, (float) x, (float) y, (float) -0.1, (float) 0)){
         x -= speed;
         worldX -= speed;
      }
      
      else{
         dashing = false;
         dashDelay = 30;
      }
      
      state = 3;
      walking = true;
      return worldX;
   }
   
   public float moveRight(Map m,float worldX){
      if(!mapCheck(m, (float) x, (float) y, (float) 0.5, (float) 0)){
         x += speed;
         worldX += speed;
      }
      
      else{
         dashing = false;
         dashDelay = 30;
      }
      
      state = 1;
      walking = true;
      return worldX;
   }

So what are some better ways I could handle collision detection?