Лабиринт на с++

[Гриб_под_зонтом]

Привет, форумчане
Вот решил написать простенький лабиринт на с++. В общем, играйте на здоровье. Может быть скоро добавлю новые уровни

(Нажмите, чтобы показать/скрыть)

Код: (cpp) [Выделить]

#include <iostream> //cout, cin
#include "main.h" //getch(), nonblock(), clear_scr()
using namespace std;
     
     int z=0;//количество собранных вишней
     int x, y; //координаты героя на данный момент
     //игровое поле
     int a[25][24] =  {{1,2,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,},
                        {1,0,1,0,0,1,0,1,0,0,0,1,1,1,1,0,0,0,0,1,},
                        {1,0,1,1,0,1,0,1,0,1,0,0,0,0,1,0,1,1,0,1,},
                        {1,0,0,0,0,0,0,1,0,1,1,1,1,0,1,0,0,1,0,1,},
                        {1,1,1,1,1,1,0,1,0,1,4,0,1,0,1,1,0,1,0,1,},
                        {1,0,0,1,0,0,0,0,0,1,1,0,0,0,0,0,0,1,0,1,},
                        {1,0,1,1,0,1,1,1,1,1,0,0,1,0,1,1,1,1,0,1,},
                        {1,0,0,0,0,0,0,0,0,1,1,1,1,0,1,0,0,0,0,1,},
                        {1,1,1,1,1,1,0,1,1,1,0,0,0,0,1,0,1,1,1,1,},
                        {1,1,0,0,0,1,0,0,1,1,0,1,1,1,1,0,0,0,0,1,},
                        {1,4,0,1,0,0,0,0,0,1,0,0,0,0,1,1,1,1,0,1,},
                        {1,1,1,1,1,1,1,1,1,1,1,1,1,0,1,0,0,0,0,1,},
                        {1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,0,1,1,1,1,},
                        {1,0,1,1,0,1,1,1,1,1,1,1,1,1,1,0,0,0,0,1,},
                        {1,0,1,0,0,0,0,0,0,0,1,4,0,0,0,0,1,1,0,1,},
                        {1,0,1,1,1,1,1,0,1,0,1,1,1,0,1,0,0,0,0,1,},
                        {1,0,1,0,0,0,1,0,1,0,0,0,1,1,1,1,1,1,1,1,},
                        {1,0,1,0,1,0,1,0,1,0,1,0,0,0,0,0,0,0,0,1,},
                        {1,0,1,0,1,0,1,0,1,0,1,1,1,1,1,1,1,1,0,1,},
                        {1,1,1,0,1,0,1,0,1,0,0,0,0,0,0,0,0,0,0,1,},
                        {1,0,1,0,1,0,0,0,1,0,1,1,1,1,1,1,0,1,0,1,},
                        {1,0,0,0,1,1,1,1,1,0,0,0,1,0,1,0,0,1,0,1,},
                        {1,5,1,0,0,0,0,0,1,1,1,0,0,0,1,0,1,1,4,1,},
                        {1,3,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,},
                        {1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,}};
   

    void print_a() // выводим лабиринт на экран
    {
for (int i=0; i<24; i++)
{
    for (int m=0; m<24; m++)
    {
    if (a[i][m]==1) cout<<"░░";
    if (a[i][m]==0 || a[i][m]==3) cout<<"  ";
    if (a[i][m]==2) cout<<"╣╠";
    if (a[i][m]==4) cout<<"••";
    if (a[i][m]==5) cout<<"––";
    }
    cout<<endl;
    }
    cout<<"Вишней: "<<z<<endl;
    if (z==4) cout<<"Дверь открыта"; // если собрали все вишнисообщаем, что дверь открыта
    }
    int win()
    {
    if (a[23][1]==2) return 1; // если нашёл выход и открыл дверь то выиграл
    else return 0;
}
void motion()
{
    char m;
    nonblock(1); // отключаем буферизованый ввод
    m = getch(); // куда хочет пойти человек?
   
    switch (m)
    {
case  65 :
    case 'w' : x--; break;
    case  66 :
    case 's' : x++; break;    // обрабатываем ход
    case  68 :
    case 'a' : y--; break;
    case  67 :
    case 'd' : y++; break;
}
if (a[x][y]==0 || a[x][y]==3 || a[x][y]==4) a[x][y] = 2;
else   // а если стенка ?
{
switch (m)
    {
case  65 :
    case 'w' : x++; break;
    case  66 :
    case 's' : x--; break;   // возвращаем всё как было
    case  68 :
    case 'a' : y++; break;
    case  67 :
    case 'd' : y--; break;
}
a[x][y] = 2;
   }
}
    int main()
    {
int k[4];
x = 0;   // начало пути
y = 1;   // начало пути
do
{
if (k[1]!=1)
   if (x==4 && y==10)  {z++; k[1]=1;}
if (k[2]!=1)                      // что бы 1 вишню не брали 2 раза
   if (x==10 && y==1)  {z++; k[2]=1;}  // подобрали вишню
if (k[3]!=1)
   if (x==14 && y==11) {z++; k[3]=1;}
if (k[4]!=1)
   if (x==22 && y==18) {z++; k[4]=1;}
   print_a();
   a[x][y] = 0;
   motion();
   clear_scr();
   if (z==4) a[22][1] = 0;
    }
   while (win()==0); // пока не выиграл
   print_a();
   cout<<endl<<"Ты выиграл!"<<endl;
   return 0;
    }


(Нажмите, чтобы показать/скрыть)

Код: (cpp) [Выделить]

     
#include <ctime>
#include <cstdlib>
#include <cstdio>
#include <unistd.h>
#include <termios.h>
#include <sys/select.h>
using namespace std;

void clear_scr()
   {
     cout<<"\033[H\033[J\033[m";
   }
void nonblock(int state)
     {
     struct termios ttystate;
     tcgetattr(STDIN_FILENO, &ttystate);
     if (state == 1)
      {
        ttystate.c_lflag &= ~ICANON;
        ttystate.c_cc[VMIN] = 1;
     }
     else if (state == 0)
      {
        ttystate.c_lflag |= ICANON;
      }
      tcsetattr(0, TCSANOW, &ttystate);
    }
   
int getch()
   {
       return fgetc(stdin);
   }

[alsoijw]

Гриб_под_зонтом, что это?

Код: (cpp) [Выделить]

if (k[1]!=1)
                   if (x==4 && y==10)  {z++; k[1]=1;}
                 if (k[2]!=1)
                   if (x==10 && y==1)  {z++; k[2]=1;}
                 if (k[3]!=1)
                   if (x==14 && y==11) {z++; k[3]=1;}
                 if (k[4]!=1)
                   if (x==22 && y==18) {z++; k[4]=1;}

Код: (cpp) [Выделить]

while (win()==0);
print_a();
cout<<endl<<"Ты выиграл!"<<endl;Так код лучше не оформлять.
Где комментарии?

[Гриб_под_зонтом]

alsoijw,
Спасибо за поправку, добавил коментарии

[alsoijw]

Гриб_под_зонтом, советую избегать магических чисел.

[Гриб_под_зонтом]

alsoijw,
Что за магические числа? Обьясните пожалуйста

[alsoijw]

Гриб_под_зонтом,
https://ru.wikipedia.org/wiki/%D0%9C%D0%B0%D0%B3%D0%B8%D1%87%D0%B5%D1%81%D0%BA%D0%BE%D0%B5_%D1%87%D0%B8%D1%81%D0%BB%D0%BE_%28%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D0%B5%29Это константы значение которых не понятно из контекста.
Пользователь решил продолжить мысль 11 Января 2015, 22:18:48:Понял?

[Гриб_под_зонтом]

alsoijw,
Постараюсь избегать) А кроме этого какие ещё замечания?

[peregrine]

Гриб_под_зонтом, это не C++, а C с cout, даже не "C с классами"... Нет ООП, совсем.

[alsoijw]

peregrine, разве (это перенаправление, да?) cout<<endl<<"Ты выиграл!"<<endl; соберётся в си?
Гриб_под_зонтом, уровень с лабиринтом хранится в файле. В программе о нём ничего не известно. Выбираешь уровень и игра идёт. Хоть в плюсах можно не ставить скобочки после if while итд но хотя бы отступ делай.
Пользователь решил продолжить мысль [time]11 Январь 2015, 22:15:11[/time]:Гриб_под_зонтом, вот что я понял - ты пишешь аналог пакмана. Есть лабиринт, включающий стены, вишни, открытые|закрытые двери, пустоту, игрока. Пока все вишни не собрал - не выиграл. Первое - если я изменю количество вишен(массив) соберу - мне может не их не хватить или собрать не все. Почему? Количество вишен у тебя - магическое число. Вывод: мы должны просканировать массив и посчитать их количество. Две переменные - сколько их всего и сколько собрали. Второе - что такое z? Имена переменных должны говорит об их смысле. Но один момент - я не могу взять одну и ту же вишню несколько раз. Ты сделал костыль - проверяется находится ли игрок в конкретной точке и если да - вишня исчезает. Опять таки - а если я поменял массив? Как нужно сделать - проверить на какой клетке стоит игрок

Код: [Выделить]

if(pole[x][y] == 4)pole[x][y] = 0;К примеру так. Только оформи это красиво. Насчёт выхода - он как и вишня может находится в любой точке, и ты не знаешь в какой. Ещё один совет - когда вишни собраны - выход должен изменить внешний вид. Не меняй массив! Сделай так, чтобы при рисовании проверялось - все ли вишни он собрал и от этого зависел внешний вид выхода.

[Гриб_под_зонтом]

Мучаюсь над уровнями. Если кто знает, как инициализировать массив из файла при помощи ifstream? Заранее спасибо

[alsoijw]

Гриб_под_зонтом, вариантов куча - читаешь байт за байтом и присваиваешь массиву. Пока сделай остальное, потом добавишь чтение из файла.

[Гриб_под_зонтом]

alsoijw,
"Остальное"  уже сделал, осталось только это

Пользователь решил продолжить мысль 12 Января 2015, 17:39:06:Если надо, вот то, что сделал:

(Нажмите, чтобы показать/скрыть)

Код: (cpp) [Выделить]

#include <iostream>
#include <fstream> 
#include <cstdio> 
#include <unistd.h>
#include <termios.h>
#include <sys/select.h>
using namespace std;
     
     const int nado_sobrat=4; // всего вишней в лабиринте
     int lvl; // номер уровня на котором сейчас играем
     int sobrano_vishney=0; // количество собраных вишней
     int vishnia_x[nado_sobrat];
     int vishnia_y[nado_sobrat];
     int sobrano[nado_sobrat];   
     int x, y; // начальная позиция в лабиринте
     int z, q; // координаты выхода
     int d, s; // координаты двери   
     
     int a[24][19]; //игровое поле
   
    void print_a() // выводим полезную информацию и лабиринт на экран
    {
cout<<"Уровень "<<lvl<<endl; // Номер уровня
for (int i=0; i<24; i++)
{
    for (int m=0; m<24; m++)
    {
    if (a[i][m]==1) cout<<"░░";
    if (a[i][m]==0 || a[i][m]==3) cout<<"  ";
    if (a[i][m]==2) cout<<"╠╣";
    if (a[i][m]==4) cout<<"••";
    if (a[i][m]==5) cout<<"––";
    }
    cout<<endl;
    }
    cout<<"Вишней: "<<sobrano_vishney<<"/"<<nado_sobrat<<endl;
    if (sobrano_vishney==nado_sobrat) cout<<"Дверь открыта";
    // если собрали все вишни сообщаем, что дверь открыта
    }
    int win()
    {
    if (a[23][1]==2) return 1; // если нашёл выход и открыл дверь то выиграл
    else return 0;
}
void nonblock(int state)
     {
     struct termios ttystate;
     tcgetattr(STDIN_FILENO, &ttystate);
     if (state == 1)
      {
        ttystate.c_lflag &= ~ICANON;
        ttystate.c_cc[VMIN] = 1;
     }
     else if (state == 0)
      {
        ttystate.c_lflag |= ICANON;
      }
      tcsetattr(0, TCSANOW, &ttystate);
    }
    int getch()
   {
   nonblock(1);
       return fgetc(stdin);
       nonblock(0);
   }
   void clear_scr()
   {
     cout<<"\033[H\033[J\033[m";
   }
void motion()
{
    char m;
    m = getch(); // куда хочет пойти человек?

    switch (m)
    {
case  65 :
    case 'w' : x--; break;
    case  66 :
    case 's' : x++; break;    // обрабатываем ход
    case  68 :
    case 'a' : y--; break;
    case  67 :
    case 'd' : y++; break;
}
if (a[x][y]==0 || a[x][y]==3 || a[x][y]==4) a[x][y] = 2;
else   // а если стенка ?
{
switch (m)
    {
case  65 :
    case 'w' : x++; break;
    case  66 :
    case 's' : x--; break;   // возвращаем всё как было
    case  68 :
    case 'a' : y++; break;
    case  67 :
    case 'd' : y--; break;
}
a[x][y] = 2;
   }
}
    int main()
    {
char file[9]; // имя файла в котором храниться уровень
        char file2[9]; // имя файла в котором храняться переменные для уровня
       
for (lvl = 1; lvl<6; lvl++) // цикл уровней
{ 

  switch (lvl)
      {
   case 1: file = "lvl1.txt"; file2 = "map1.txt"; break;
   case 2: file = "lvl2.txt"; file2 = "map2.txt"; break;
   case 3: file = "lvl3.txt"; file2 = "map3.txt"; break;
   case 4: file = "lvl4.txt"; file2 = "map4.txt"; break;
   case 5: file = "lvl5.txt"; file2 = "map5.txt"; break;
  }
     
  ifstream fin(file2);
      ifstream flvl(file);
      cout<<(flvl?"":"Уровень не найден");
     
     
     
do  // основной цикл игры
{
     if (sobrano_vishney==nado_sobrat) a[d][s] = 0;
     // собрали все вишни? откроем дверь
     
   for (int i=0; i<nado_sobrat; i++)
   {
   if (sobrano[i]!=1)   // что бы 1 вишню не брали 2 раза
   
   if (x==vishnia_x[i] && y==vishnia_y[i]) 
   {
   sobrano_vishney++; sobrano[i]=1; // подобрали вишню
  } 
  }           
   print_a();
   
   a[x][y] = 0;
   
   motion();
   
   clear_scr();
   
    }
   while (win()==0); // пока не выиграл
   
   print_a();
   
   if (lvl<5) cout<<"Уровень пройден";
     }
   cout<<endl<<"Ты выиграл!"<<endl;
   
   return 0;
    }


[alsoijw]

Гриб_под_зонтом, я же сказал - без магических чисел!

[Гриб_под_зонтом]

alsoijw,
Так я вроде без них сделал...

[alsoijw]

Гриб_под_зонтом, а const int nado_sobrat=4;