Программа Allegro отображает черный экран, затем вылетает

Question

Я изучаю объектно-ориентированное программирование и библиотеку allegro, поэтому я начинаю писать простое меню, но код отображает черный экран, а затем вылетает.

Я буду благодарен за предложения о появлении классов, разделении кода с заголовками, общем синтаксисе кода и, прежде всего, о поиске проблемы, для которой код не работает.

main.cpp

#include "Menu.h"

int main() {
    Menu m;
    return 0;
}
END_OF_MAIN()

Menu.h

#ifndef MENU_H_
#define MENU_H_
#include <allegro.h>

class Menu {
    static bool isWorking;
    static const int WIDTH = 800;
    static const int HEIGHT = 600;
public:
    Menu();
    virtual ~Menu();
};

#endif

menu.cpp

#include "MenuPointer.h"
#include "Menu.h"
#include <string>
using namespace std;

bool Menu::isWorking = true;

Menu::Menu() {
    allegro_init();
    install_keyboard();
    install_timer();
    set_color_depth(8);
    set_gfx_mode(GFX_AUTODETECT_WINDOWED, WIDTH, HEIGHT, 0, 0);
    string * menuOptions = new string[2];
    menuOptions[0] = "New Game";
    menuOptions[2] = "Exit";

    // menu pointer is an object highlighting menu options

    MenuPointer menuPointer(2, 0, WIDTH, HEIGHT/2, menuOptions);
    while (isWorking) {
        menuPointer.menuInit();
        blit(menuPointer.getBuffer(), screen, 0, 0, 0, 0, WIDTH, HEIGHT);
    }
}

Menu::~Menu() {
    allegro_exit();
    clear_keybuf();
}

MenuPointer.h

#ifndef MENUPOINTER_H_
#define MENUPOINTER_H_

#include <string>
#include <allegro.h>
using namespace std;

class MenuPointer {
    int pointerNumber;
    BITMAP * buffer;
    const int optionsNumber;
    const int pointerWidth;
    const int pointerHeight;
    BITMAP ** optionsImages;
    const string * optionsTexts;
    void paintPointer();
public:
    void menuInit();
    BITMAP * getBuffer();
    MenuPointer(int, int, int, int, BITMAP **);
    MenuPointer(int, int, int, int, const string *);
    virtual ~MenuPointer();
};

#endif

MenuPointer.cpp

#include "MenuPointer.h"
#include "Menu.h"
#include <string>
using namespace std;


// this constructor creates menu pointer when options are array of images
MenuPointer::MenuPointer(int i, int j, int k, int l, BITMAP ** optionsImages)
    :pointerNumber(j), optionsNumber(i), pointerWidth(k), pointerHeight(l) {
    buffer = create_bitmap(pointerWidth, pointerHeight);
    optionsTexts = NULL;
    this->optionsImages = optionsImages;
}

// this constructor creates menu pointer when options are array of text
MenuPointer::MenuPointer(int i, int j, int k, int l, const string * optionsTexts)
    :pointerNumber(j), optionsNumber(i), pointerWidth(k), pointerHeight(l) {
        buffer = create_bitmap(pointerWidth, pointerHeight);
        optionsImages = NULL;
        this->optionsTexts = optionsTexts;
}


// draws a rectangle in the range from the beginning to the next option which highlights
void MenuPointer::paintPointer() {
    rectfill(buffer, 0, pointerNumber*pointerHeight, pointerHeight, (pointerNumber+1)*pointerHeight, makecol( 128, 30, 30 ) );
}


void MenuPointer::menuInit() {
    clear_to_color(buffer, makecol( 128, 128, 128 ));
    paintPointer();
    if (optionsTexts != NULL) {
        // converts string to char array
        for (int i = 0; i < optionsNumber; ++i) {
            char *a = new char[optionsTexts[i].size()+1];
            a[optionsTexts[i].size()] = 0;
            memcpy(a, optionsTexts[i].c_str(), optionsTexts[i].size());
            textout_ex(screen, font, a, 0, i*pointerWidth, makecol(255,0,0), -1);
        }
    } else {
        // draws option image for images array
        for (int i = 0; i < optionsNumber; ++i) {
            masked_blit(optionsImages[i], screen, 0, 0, 0, i*pointerWidth, optionsImages[i]->w, optionsImages[i]->h);
        }
    }
}

BITMAP * MenuPointer::getBuffer() {
    return buffer;
}

MenuPointer::~MenuPointer() {
    destroy_bitmap(buffer);
}

Answer 1

Всегда проверяйте возвращаемые значения! Особенно, когда вещи начинают терпеть крах.

Вы попросили Allegro работать с 8-битной глубиной цвета. Это может потерпеть неудачу, даже запрос 24-битной в 32-битной системе может потерпеть неудачу. Вы можете использовать desktop_color_depth(), чтобы узнать, на чем работает хост.

Убедитесь, что set_gfx_mode() не возвращает ошибку, прежде чем продолжить, и что все растровые изображения действительно загружены. (не == ноль)

Answer 2

У вас есть больше отладочной информации о сбое?

Прежде всего, одной из возможных проблем, которая может вызвать сбой, является плохой доступ к пунктам меню массива.

string * menuOptions = new string[2];
menuOptions[0] = "New Game";
menuOptions[2] = "Exit"; // THIS IS WRONG

У вас есть утечки внутри этого цикла:

    for (int i = 0; i < optionsNumber; ++i) {
        char *a = new char[optionsTexts[i].size()+1]; //this is never freed
        a[optionsTexts[i].size()] = 0;
        memcpy(a, optionsTexts[i].c_str(), optionsTexts[i].size());
        textout_ex(screen, font, a, 0, i*pointerWidth, makecol(255,0,0), -1);
    }

Различные подсказки:

1. Используйте strncpy вместо memcpy, чтобы скопировать строку.
2. Названия параметров методов / функций должны быть более четкими. (никогда не используйте i,j,k)
3. Я всегда предпочитаю ставить имена и при объявлении метода / функции.
4. Поместите ваш метод цикла из конструктора. (метод с именем update может быть в порядке)