Rysowanie mapy w konsoli, co robię nie tak?

Question

Witam, próbuję narysowac w konsoli mapę 15x15 wypełnioną znakami "I", "o" oraz "_" która będzie się losowo generować za każdym razem. Prosiłbym o jakieś wskazówki które doprowadzą mnie do rozwiązania tego zadania :) 

Oto dotychczasowy kod: 

#include <iostream>
#include <cstdlib>
#include <time.h>
using namespace std;

int mapa_wys = 15;
int mapa_szr = 15;

int mapa[mapa_wys][mapa_szr];

int main()

{
    for(int i = 0, i < mapa_wys; i++)
    {

        for(int j = 0; j < mapa_szr; j++)
        {
            srand(time(NULL));
            mapa[i][j] = rand()%3+1;
        }
    }
    cout<<mapa;
    return 0;
}

 

Answer 1

UPDATE

Program odpala się juz bez błędów! :) Poprawiłem wszystko tak jak zostało mi doradzone. Teraz pozostał tylko ostatni krok czyli narysowanie mapy przy użyciu znaków.

Poprawiony kod:

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <windows.h>


using namespace std;
HANDLE g_hConsol = GetStdHandle(STD_OUTPUT_HANDLE);
void gotoxy (const int x, const int y)
{
    COORD coord = {x, y};
    SetConsoleCursorPosition(g_hConsol, coord);
}
const int mapa_wys = 15;
const int mapa_szr = 15;

int mapa[mapa_wys][mapa_szr];

int main(){
    srand(time(NULL));
    for(int y = 0; y < mapa_wys; y++)
    {

        for(int x = 0; x < mapa_szr; x++)
        {
            mapa[x][y] = rand()%3+1;
        }
    }
    return 0;
}

 

Comments

A przy pomocy mojej funckcji powinno to wyglądać tak:
 

#include <iostream>
#include <cstdlib>
#include <time.h>
#include <windows.h>
 
 
using namespace std;
HANDLE g_hConsol = GetStdHandle(STD_OUTPUT_HANDLE);
void gotoxy (const int x, const int y)
{
    COORD coord = {x, y};
    SetConsoleCursorPosition(g_hConsol, coord);
}
const int mapa_wys = 15;
const int mapa_szr = 15;
 
int mapa[mapa_wys][mapa_szr];
 
int main(){
    srand(time(NULL));
    for(int y = 0; y < mapa_wys; y++)
    {
 
        for(int x = 0; x < mapa_szr; x++)
        {
            gotoxy(x,y);
            cout<<rand()%3+1;
        }
    }
    return 0;
}

Myślę że to powinno rozwiązać problem ;)

---

Nie podoba mi się włączanie do tego zadania jakichś dodatkowych funkcji, dlatego chciałbym zaprezentować mój sposób rozwiązania problemu:

#include <iostream>
#include <cstdlib>
#include <time.h>

using namespace std;

string mapa[15];

void wypelnij(string* mapa, unsigned szerokosc, unsigned wysokosc)
{
    for(unsigned y=0; y<wysokosc; ++y)
    {
        for(unsigned x=0; x<szerokosc; ++x)
        {
            // ten fragment mozna rozbudowac, czy tez zmienic
            // aby uzyskac inne losowe elementy na mapie
            char znak = rand() % 3 + 33;
            // to juz lepiej zostawic w spokoju
            mapa[y] += znak;
        }
    }
}

void narysuj(string* mapa, unsigned wysokosc)
{
    for(unsigned i=0; i<wysokosc; ++i)
        cout << mapa[i] << endl;
}

int main()
{
    srand(time(NULL));
    wypelnij(mapa, 15, 15);
    narysuj(mapa, 15);
    return 0;
}

Zauważ, że narysowanie mapy jest teraz bardzo proste oraz, że możesz dowolnie zmieniać szerokość danej linijki (do granic szerokości konsoli znakowej).

Gdybyś czegoś nie rozumiał, pisz śmiało.

---

Dziękuję bardzo za świetną odpowiedź! Skróciłeś oraz sprawiłeś, że kod jest przejrzysty :) Oto jest końcowa odpowiedź!

Answer 2

Zgaduje że masz błąd w 

int mapa_wys = 15;
int mapa_szr = 15;

Ponieważ tablice można definiować jedynie stałymi, więc by działała musisz zrobić z tego const inty

Comments

Dobrze, że szerzysz dobre nawyki  W niektórych kompilatorach sztuczka ze zmienną zadziała, ale należy używać stałych.

Answer 3

Witaj! :)

Jest to moja pierwsza odpowiedź na owym forum więc mam nadzieję że będzie pomocna :P Ok, do rzeczy.

Więc na początek programu dopisz taką funkcję:
 

#include <windows.h>

HANDLE g_hConsol = GetStdHandle(STD_OUTPUT_HANDLE);

void gotoxy(const int x, const int y)
{
	COORD coord = {x, y};
	SetConsoleCursorPosition(g_hConsol, coord);
}

I spróbuj ją do tego wykorzystać. W razie czego pisz :) 

Comments

Jesteśmy coraz bliżej rozwiązania :D Tak jak powiedział  Adrian1999 zamieniłem inty na const inty i zniknął jeden z błędów. Teraz tylko jak wykorzystać tą funkcję którą ty napisałeś? 

Answer 4

Srand przenieś przed pętla, najlepiej na samym początku main'a

Answer 5

srand trzeba wywołać tylko raz, na początku programu.

"cout << mapa" nie zadziała, musisz ręcznie iterować po mapie i drukować kolejne elementy.

Mała rada: jeśli to ma być mapa 2D, często bardziej intuicyjne jest używanie indeksów y, x zamiast i, j.