Пишем рогалик: пояснения по коду и его критика

#include <iostream>
#include <fstream>
#include <stdlib.h>
#include "hero.h"
#include "control.h"
 
 
using namespace std;
 
int main()
{
    srand (time(NULL));
    hero me(1);
    
    
    const int size_a = 5;
    const int size_b = 5;
    
    char fi[size_a][size_b][2]; // тут два уровня:
                                // в первом находится все поле,
                                // по второму ходит герой (изначально
                                // весь уровень закрыт)    
                                
    //generate(size_a, size_b, *fi[size_a][size_b]); // хотел вынести в функцию, 
    for(int i =0; i<size_a; i++)                   // не получилось
    {
        for(int j=0; j<size_b; j++)
        {
            int a=random()%25;
            if(a>20)
                fi[i][j][0] = '$';//это враг
            else
                fi[i][j][0] = ' ';
            fi[i][j][1] = '#';  
        }
    }
    fi[random()%size_a][random()%size_b][0] = '@';//это портал 
    
    for(;;)// может лучше while(true)? И если да то чем?
    {
        fi[me.x][me.y][1] = 'i';//это наш герой
        
        for(int i =0; i<size_a; i++)
        {
            for(int j=0; j<size_b; j++)
                cout<<fi[i][j][1];
            cout<<endl; 
        }
        
        if (fi[me.x][me.y][0]=='$')             //тут выводится
            cout<<"Ther is an enemy"<<endl; //содержимое клетки
        else if (fi[me.x][me.y][0]=='@')                //на которой мы стоим
            cout<<"There is a portall"<<endl;
        fi[me.x][me.y][1] = fi[me.x][me.y][0];
        
        
        
        
        control(&me.x, &me.y);
        back(&me.x, &me.y, size_a, size_b);
        
        
        cout<<"\033[2J";        //это очистка 
        cout<<"\033[0;0f";      //экрана
    }
    
    cout<<endl;
    
    
    return 0;
}

#include <stdlib.h>
#include <fstream>
using namespace std;
class hero
{
    public:                 //я слышал что переменные должны в private,
        int strength;       //но не кто не обьяснил почему 
        int health;
        int luck;
        int exp;
        int x;
        int y;
        
        void print();
        void hit(hero *en);
        hero(bool a = false, int X = 0, int Y = 0)                    // если а - правда,
        {                                            //то вы - герой
            strength = random()%10 + 6;
            health = random()%20 + 15;
            luck = random()%9 + 2;
            exp=0;
            
            if(a)
            {
                strength = 5;
                health = 6;
                luck = 3;
            }
            
            x = X;
            y = Y;
             
        }
};
 
void hero::print()
{
    cout<<" health = "<<health<<endl;
    cout<<" strength = "<<strength<<endl;
    cout<<" luck = "<<luck<<endl;
}
 
void hero::hit(hero *en)
{
    if(luck>random()%10)                       //для критических ударов
    {
        en->health -= strength*1.5;
    }
    else
        en->health -= strength;
}

void back(int *x, int *y, int size_a, int size_b)
{
    if(*x>size_a)
        *x=0;
    if(*y>size_b)                    // это чтобы герой не убегал 
        *y=0;                         // за приделы поля
    if(*y<0)
        *y=size_b;
    if(x<0)
        *x=size_a;
}
 
void control(int *x, int *y)
{
    char dir;
    dir = getchar();                   // в винде была ф-я getch()
                            // хотелось бы что-то такое же
        
 
    if(dir=='d')
        *y+=1;
        
    if(dir=='a')                        // по совершенно не понятным
        *y-=1;          // причинам, у++ и х++ 
                        // не работали =(
    if(dir=='s')        
        *x+=1;          
            
    if(dir=='w')
        *x-=1;
}