Consigli su un programmino.

[ziofu]

Ciao a tutti! Sono nuovo di qui.

Ho iniziato da poco a programmare in C++. Vengo da Java, quindi il passaggio non è rose e fiori. Non sono ancora arrivato alle classi, qunidi in sostanza (da quanto ho capito) è come se fino adesso avessi programmato in C.

Seguendo un libro, ho fatto un programma che dovrebbe essere un primordiale programma di disegno (turtle graphics) e volevo sapere:

- se il codice è abbastanza pulito: non sono molto contento dei 2 switch (sono convinto ci sia un metodo più elegante).

- come prendere in input 2 numeri consecutivi, ma lasciando il secondo "facoltativo". Cioè con:
cin >> a >> b;

salvare in a il primo ed eventualmente in b se viene passato.

Questo è il mio codice (siate pure cattivi che devo imparare ^^):

#include <iostream>

using namespace std;

void printMap(void);
void printCommands(void);
void move(int);

const int END = 9;
const int commands[7] = { 0, 1, 2, 3, 4, 5, 6 };
int map[20][20] = { 0 };
bool penDown = false;
int x = 0;
int y = 0;
int dir = 0; // 0 = N, 1 = E, 2 = S, 3 = O

int main() {

    cout << "Welcome to the Turtle Graphics! Here's the list of commmands:" << endl;
    printCommands();
    
    int command;
    int args;
    
    cout << "Enter a command: ";
    cin >> command;
    if (command == 5) cin >> args;
    
    while (command != END) {
        switch (command) {
            // print commands
            case 0:
            printCommands();
            break;
            
            // set the pen down
            case 1:
            if (penDown) penDown = false;
            else cout << "The pen is already up." << endl;
            break;
            
            // pen up
            case 2:
            if (!penDown) penDown = true;
            else cout << "The pen is already down." << endl;
            break;
            
            // turn right
            case 4:
            dir = (dir + 1) % 4;
            cout << "The new direction is " << dir << endl;
            break;
            
            // turn left
            case 3:
            dir -= 1;
            if (dir < 0) dir = 3;
            cout << "The new direction is " << dir << endl;
            break;
            
            // move forward
            case 5:
            cout << "Moving forward by " << args << endl;
            move(args);
            break;
            
            // show the map
            case 6:
            printMap();
            break;
        
        }
        
        cout << "Enter a command (0 to show commands): ";
        cin >> command;
        if (command == 5) cin >> args;
        
    }
    
    cout << "Bye bye!" << endl;
    return 0;
}

void move(int steps) {
    while (steps-- > 0) {
        switch (dir) {
            // N
            case 0:
            if (y > 0) y--;
            break;
            
            // E
            case 1:
            if (x < 20) x++;
            break;
        
            // S
            case 2:
            if (y < 19) y++;
            break;
            
            // O
            case 3:
            if (x > 0) x--;
            break;
        }
        if (penDown) map[y][x] = 1;
    }
}

void printMap() {
    for (int i = 0; i < 20; i++) {
        for (int j = 0; j < 20; j++) {
            if (x == j && y == i) cout << "X";
            else if (map[i][j] == 0) cout << " ";
            else cout << "*";
        }
        cout << endl;
    }
}

void printCommands() {
    cout << "0    - Show the commands available\n"
         << "1    - Turn up the pen\n"
         << "2    - Turn down the pen\n"
         << "3    - Turn left\n"
         << "4    - Turn right\n"
         << "5, n - Move therd of n tiles\n"
         << "6    - Show the drawing\n"
         << "9    - Exit program" << endl; 
}