Allegro.cc - Online Community

Allegro.cc Forums » Allegro Development » [A5]ALLEGRO_SAMPLE_ID return id -1

This thread is locked; no one can reply to it. rss feed Print
[A5]ALLEGRO_SAMPLE_ID return id -1
Matheus Santos
Member #15,053
April 2013

sorry for the english, I'm brazilian and I need help ...
ALLEGRO_SAMPLE_ID always the id is returning -1
I stated the ALLEGRO_SAMPLE_ID in a header and I am trying to access it in another. cpp but it always returns the id - 1
I'll post the part of the code here:

.h
ALLEGRO_SAMPLE_ID teste;

.cpp

al_play_sample(sample_cowboys, 1, 0, 1, ALLEGRO_PLAYMODE_LOOP, &teste);
al_stop_sample(&teste); // that part of the runtime error!!

help me please!

SiegeLord
Member #7,827
October 2006
avatar

You probably didn't call al_reserve_samples, or called it with too small a number. Also, you should check that al_play_sample returns true before you use the ALLEGRO_SAMPLE_ID for anything.

"For in much wisdom is much grief: and he that increases knowledge increases sorrow."-Ecclesiastes 1:18
[SiegeLord's Abode][Codes]:[DAllegro5]:[RustAllegro]

Matheus Santos
Member #15,053
April 2013

i'm calling al_reserve_samples (1);
al_play_sample is returning true

I tried to increase the number of al_reserve_samples but not solved

code

al_reserve_samples(10);
sample_cowboys = al_load_sample("aud/teste.ogg");

torhu
Member #2,727
September 2002
avatar

Did you also call al_install_audio and al_init_acodec_addon?

EDIT: If you declare a variable in a header, you have to put extern in front of it, to avoid getting multiple definitions.

// Declare in a header:
extern ALLEGRO_SAMPLE_ID teste;

// Define in a single cpp:
ALLEGRO_SAMPLE_ID teste;

Then it can be used in every .cpp file that included the header with the declaration.

Matheus Santos
Member #15,053
April 2013

yes, my only problem is the ALLEGRO_SAMPLE_ID the moment I use al_stop_sample.

#SelectExpand
1 2Viewinit.cpp 3viewInit::viewInit(){ // Class only for init al settings 4 5 if(!al_init()) { 6 fprintf(stderr, "Falha ao iniciar o allegro\n"); 7 } 8 if (!al_install_keyboard()) 9 { 10 fprintf(stderr, "Falha ao inicializar o teclado.\n"); 11 } 12 13 if (!al_install_audio()) 14 { 15 fprintf(stderr, "Falha ao inicializar áudio.\n"); 16 17 } 18 19 if (!al_init_acodec_addon()) 20 { 21 fprintf(stderr, "Falha ao inicializar codecs de áudio.\n"); 22 exit(1); 23 } 24 25 al_init_image_addon(); 26 al_init_font_addon(); 27 al_init_ttf_addon(); 28 29 30 //al_rest(10.0); 31 32 //al_destroy_display(display); 33 34} 35 36viewGame.h 37 38ALLEGRO_SAMPLE_ID teste; 39 40viewGame.cpp 41// this in other class and cpp 42#include "viewGame.h" 43 44viewGame::viewGame(){ // 45 al_reserve_samples(1); 46 sample_cowboys = al_load_sample("aud/bingo_1l.ogg"); 47 48} 49 50void viewGame::draw_anim_cowboys(){ 51 al_play_sample(sample_cowboys, 1, 0, 1, ALLEGRO_PLAYMODE_LOOP, &teste); 52 al_stop_sample(&teste); 53}

EDIT

the ALLEGRO_SAMPLE_ID this in a class

torhu
Member #2,727
September 2002
avatar

And you are sure that viewInit is actually constructed before viewGame? It's hard to tell when you only post snippets of code...

Matheus Santos
Member #15,053
April 2013

#SelectExpand
1//viewGame.h 2#include "libs.h" 3#include "viewGlobal.h" 4#include "controller.h" 5 6class viewGame: public viewGlobal{ 7protected: 8 9 // IMAGENS 10 11 12 ALLEGRO_BITMAP *img_bg[24]; // Imagens bg de acordo com o horário 13 14 ALLEGRO_BITMAP *anim_full_rabo_cavalos; 15 ALLEGRO_BITMAP *anim_rabo_cavalos[35]; 16 17 ALLEGRO_BITMAP *anim_full_trem; 18 ALLEGRO_BITMAP *anim_trem[125]; 19 20 ALLEGRO_BITMAP *anim_full_logo; 21 ALLEGRO_BITMAP *anim_logo[70]; 22 23 ALLEGRO_BITMAP *anim_full_cowboys[4]; 24 ALLEGRO_BITMAP *anim_full_bingo; 25 ALLEGRO_BITMAP *anim_full_acumulado; 26 27 ALLEGRO_BITMAP *anim_cowboys[177]; 28 ALLEGRO_BITMAP *anim_bingo[30]; 29 ALLEGRO_BITMAP *anim_acum[39]; 30 31 ALLEGRO_BITMAP *img_placa_acum; 32 33 34 // FONTES 35 ALLEGRO_FONT *font_display; 36 37 // timer para começar animações 38 ALLEGRO_TIMER *timer_anim_cavalo; 39 ALLEGRO_TIMER *timer_anim_trem; 40 ALLEGRO_TIMER *timer_anim_logo; 41 42 43 // timer para trocar o frame de cada animação 44 ALLEGRO_TIMER *timer_frame_cavalo; 45 ALLEGRO_TIMER *timer_frame_trem; 46 ALLEGRO_TIMER *timer_frame_logo; 47 48 ALLEGRO_TIMER *timer_frame_cowboys; 49 ALLEGRO_TIMER *timer_frame_bingo; 50 ALLEGRO_TIMER *timer_frame_acumulado; 51 52 // Eventos 53 ALLEGRO_EVENT_QUEUE *fila_eventos; 54 55 56 // Frame animações 57 58 int frame_anim[10]; // Controla os frames das animações 59 60 // controle de animações 61 62 int play_anim_cowboys; // 1 se tiver que fazer a animação 63 int play_anim_bingo; 64 int play_anim_acum; 65 66 int status_anim[10]; //1 = começa a animação; 2 = troca o frame 67 68 // Sons 69 70 ALLEGRO_AUDIO_STREAM *music_main; 71 ALLEGRO_AUDIO_STREAM *music_bingo; 72 ALLEGRO_AUDIO_STREAM *music_acumulado; 73 74 ALLEGRO_SAMPLE *sample_tiro; 75 ALLEGRO_SAMPLE *sample_acum; 76 ALLEGRO_SAMPLE *sample_bingo; 77 ALLEGRO_SAMPLE *sample_cowboys; 78 79 ALLEGRO_SAMPLE_ID id_sample_cowboys; 80 81 ALLEGRO_SAMPLE_ID teste; 82 83 // Outros 84 85 char aux[50]; // LoadImage 86 int x; // Loop 87 88 89 char tempo[60]; 90 int i; 91 time_t hora; 92 93 94public: 95 viewGame(); // Carrega todas as imagens e variaveis que ira usar durante a execução do mesmo 96 97 void main(controller refcontroller); // Faz o controle das animações; 98 99 // METODOS DE ANIMAÇÂO 100 void draw_anim_cavalo(); 101 void draw_anim_logo(); 102 void draw_anim_trem(); 103 104 void draw_anim_bingo(); 105 void draw_anim_acum(); 106 107 void draw_anim_cowboys(int status); 108 109 // ---------------------- 110 111 void play_main_music(); 112 113 int get_hour_bg(); 114 // 115 void input(); 116 117 ALLEGRO_EVENT get_event(); 118 119 void get_status(); 120 121 122}; 123 124//viewGame.cpp 125 126#include "viewGame.h" 127 128 129viewGame::viewGame(){ 130 131 //fprintf(stderr, "Teste construtor viewGame\n"); 132 cout << "construtor viewGame OK.\n"; 133 134 135 // ANIMA LOGO 136 for(x = 0; x < 70 ; x++){ // Carregando animação do rabo do cavalo 137 anim_logo[x] = al_create_bitmap(543, 411); 138 al_set_target_bitmap(anim_logo[x]); 139 al_draw_bitmap_region(anim_full_logo, 0 , 411 * x, 543, 411, 0, 0, 0); 140 } 141 142 // ANIMAÇÂO COWBOYS 143 for(x = 0; x < 177; x++){ 144 145 anim_cowboys[x] = al_create_bitmap(750, 294); 146 al_set_target_bitmap(anim_cowboys[x]); 147 148 if(x < 50) al_draw_bitmap_region(anim_full_cowboys[0], 0 , 294 * x, 750, 294, 0, 0, 0); 149 if(x > 49 && x < 100) al_draw_bitmap_region(anim_full_cowboys[1], 0 , 294 *( x - 50 ), 750, 294, 0, 0, 0); 150 if(x > 99 && x < 150) al_draw_bitmap_region(anim_full_cowboys[2], 0 , 294 *( x - 100 ), 750, 294, 0, 0, 0); 151 if(x > 149 && x < 177) al_draw_bitmap_region(anim_full_cowboys[3], 0 , 294 *( x - 150 ), 750, 294, 0, 0, 0); 152 153 al_convert_mask_to_alpha(anim_cowboys[x], al_map_rgb(255,0,255)); 154 155 } 156 157 for(x = 0; x < 30; x++){ 158 anim_bingo[x] = al_create_bitmap(750, 294); 159 al_set_target_bitmap(anim_bingo[x]); 160 161 al_draw_bitmap_region(anim_full_bingo, 0 , 294 * x, 750, 294, 0, 0, 0); 162 al_convert_mask_to_alpha(anim_bingo[x], al_map_rgb(255,0,255)); 163 } 164 165 for(x = 0; x < 39; x++){ 166 anim_acum[x] = al_create_bitmap(750, 294); 167 al_set_target_bitmap(anim_acum[x]); 168 169 al_draw_bitmap_region(anim_full_acumulado, 0 , 294 * x, 750, 294, 0, 0, 0); 170 al_convert_mask_to_alpha(anim_acum[x], al_map_rgb(255,0,255)); 171 } 172 173 al_set_target_bitmap(al_get_backbuffer(display)); 174 175 // FONTES 176 font_display = al_load_font("digital display tfb.ttf", 160, 0); 177 178 // TIMERS 179 180 timer_anim_cavalo = al_create_timer(5.0); 181 timer_frame_cavalo = al_create_timer(0.05); 182 183 timer_anim_trem = al_create_timer(5.0); 184 timer_frame_trem = al_create_timer(0.05); 185 186 timer_anim_logo = al_create_timer(5.0); 187 timer_frame_logo = al_create_timer(0.05); 188 189 timer_frame_cowboys = al_create_timer(0.04); 190 timer_frame_bingo = al_create_timer(0.04); 191 timer_frame_acumulado = al_create_timer(0.04); 192 193 194 // EVENTOS - TIMER 195 fila_eventos = al_create_event_queue(); 196 197 198 al_register_event_source(fila_eventos, al_get_display_event_source(display)); // Registra os eventos da janela 199 200 // Registra evento dos timers 201 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_anim_cavalo)); 202 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_cavalo)); 203 204 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_anim_trem)); 205 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_trem)); 206 207 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_anim_logo)); 208 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_logo)); 209 210 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_cowboys)); 211 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_bingo)); 212 al_register_event_source(fila_eventos, al_get_timer_event_source(timer_frame_acumulado)); 213 214 al_register_event_source(fila_eventos, al_get_keyboard_event_source()); 215 216 217 al_start_timer(timer_anim_cavalo); 218 al_start_timer(timer_anim_trem); 219 al_start_timer(timer_anim_logo); 220 221 // FRAMES 222 for(x = 0; x < 7; x++){ // Zera todos os frames das animações 223 frame_anim[x] = 0; 224 } 225 226 // SONS 227 228 al_reserve_samples(1); 229 230 music_main = al_load_audio_stream("aud/western_theme.ogg", 4, 1024); 231 music_acumulado = al_load_audio_stream("aud/acum_sound.ogg", 4, 1024); 232 music_bingo = al_load_audio_stream("aud/bingo_sound.ogg", 4, 1024); 233 234 sample_cowboys = al_load_sample("aud/bingo_1l.ogg"); 235 236 al_attach_audio_stream_to_mixer(music_main, al_get_default_mixer()); 237 al_attach_audio_stream_to_mixer(music_acumulado, al_get_default_mixer()); 238 al_attach_audio_stream_to_mixer(music_bingo, al_get_default_mixer()); 239 240 241 al_set_audio_stream_playing(music_main, 0); 242 al_set_audio_stream_playing(music_acumulado, 0); 243 al_set_audio_stream_playing(music_bingo, 0); 244 245 246 247 // OUTROS 248 play_anim_cowboys = 0; 249 play_anim_acum = 0; 250 play_anim_bingo = 0; 251 252 ALLEGRO_SAMPLE_ID teste; 253 254} 255// MAIN LOOP 256void viewGame::main(controller controller){ 257 258 al_draw_bitmap(img_bg[ get_hour_bg() ], 0, 0, 0); 259 al_draw_bitmap(img_placa_acum, 0, 0, 0); 260 261 get_status(); 262 263 if(play_anim_cowboys == 0 && play_anim_bingo == 0 && play_anim_acum == 0){ 264 draw_anim_trem(); 265 draw_anim_cavalo(); 266 } 267 268 draw_anim_logo(); 269 270 draw_anim_cowboys(play_anim_cowboys); 271 draw_anim_bingo(); 272 draw_anim_acum(); 273 274 al_draw_text(font_display, al_map_rgb(255, 0, 0), 923, 445, ALLEGRO_ALIGN_CENTER, "888.888,88"); 275 al_draw_text(font_display, al_map_rgb(255, 0, 0), 953, 256, ALLEGRO_ALIGN_CENTER, "88.888,88"); 276 al_draw_text(font_display, al_map_rgb(255, 0, 0), 983, 68, ALLEGRO_ALIGN_CENTER, "8.888,88"); 277 278 al_flip_display(); 279} 280// FUNCTION ERROR 281 282void viewGame::draw_anim_cowboys(int controller_status ){ 283 int retorno_sound; 284 285 if(controller_status == 1 || controller_status == 3){ 286 al_start_timer(timer_frame_cowboys); 287 play_anim_cowboys = controller_status; 288 retorno_sound = al_play_sample(sample_cowboys, 1, 0, 1, ALLEGRO_PLAYMODE_LOOP, &teste); 289 cout << retorno_sound << "\n\n"; 290 } 291 292 if(play_anim_cowboys == 1 || play_anim_cowboys == 3){ 293 al_draw_bitmap(anim_cowboys[frame_anim[IDX_COWBOYS]], 0, 474, 0); 294 295 if(status_anim[IDX_COWBOYS] == 2){ 296 frame_anim[IDX_COWBOYS] = (frame_anim[IDX_COWBOYS] + 1) % 177; 297 298 if(frame_anim[IDX_COWBOYS] == 0){ 299 al_stop_timer(timer_frame_cowboys); 300 if (play_anim_cowboys == 3){ 301 al_start_timer(timer_frame_bingo); 302 play_anim_bingo = 1; 303 } 304 if (play_anim_cowboys == 1){ 305 play_anim_acum = 1; 306 al_start_timer(timer_frame_acumulado); 307 } 308 309 al_stop_sample(&teste); // AT THIS LINE ERRO 310 311 play_anim_cowboys = 0; 312 al_rest(1.0); 313 } 314 } 315 } 316}

EDIT

#SelectExpand
1//IN MAIN.cpp 2int main(int argc, char **argv) 3{ 4 5 6 7 int running = 1; 8 9 viewInit viewInit; 10 viewGame viewGame; 11 controller controller; 12 13 while(running){ 14 controller.main(); 15 viewGame.main( controller ); 16 } 17 18 19 20 return 0; 21}

SiegeLord
Member #7,827
October 2006
avatar

I'm looking at the source of al_play_sample and I don't see any way that it can return true while at the same time setting the id parameter to -1; Try making a small test case that illustrates the bug I guess.

"For in much wisdom is much grief: and he that increases knowledge increases sorrow."-Ecclesiastes 1:18
[SiegeLord's Abode][Codes]:[DAllegro5]:[RustAllegro]

Go to: