decoejz / aps1embarcados

APS1embarcados

aps1embarcados's Issues

Existe o define no entanto não está sendo utilizado

No inicio do main, vocês definiram o "BMUS_PIO" corretamente, no entanto em varios trechos de código no main, não se utiliza o define e sim está usando diretamente o "PIOD".

Isto é ruim, pois o usuário troca o BMUS_PIO e percebe que o código não funciona, pois na realidade, não alterou no código todo.

APS1embarcados/apsBuzzer/src/main.c

Line 111 in b2b2de3

while(pio_get(PIOD,PIO_INPUT, BMUS_PIO_IDX_MASK)){}

Transformar trechos repetitivos de código em funções reutilizáveis

No código principal podemos observar, no loop while(1) do main, onde ele faz a lógica para execução de uma três de músicas.

APS1embarcados/apsBuzzer/src/main.c

Lines 115 to 166 in b2b2de3

	//*************MUSICA 1 - TEMA INDIANA JONES*****************
	pio_set(PIOA, LED1_PIO_IDX_MASK);
	pio_clear(PIOA, LED2_PIO_IDX_MASK);
	pio_clear(LED3_PIO, LED3_PIO_IDX_MASK);
	for (int i=2;i<notesI[0];i++){
	if(proxima){
	proxima=0;
	break;
	}
	double periodo = 1.0/notesI[i];
	double duracao=0;
	while(duracao < durationI[i]*notesI[1]){
	if(!pio_get(PIOD,PIO_INPUT, BMUS_PIO_IDX_MASK)){
	pausa = !pausa;
	pio_clear(PIOA, BUZZER_PIO_IDX_MASK);
	while(pausa){
	delay_ms(170);
	if(!pio_get(PIOD,PIO_INPUT, BMUS_PIO_IDX_MASK)){
	pausa = !pausa;
	}
	delay_ms(170);
	}
	}
	if(!pio_get(PIOD,PIO_INPUT, BNEX_PIO_IDX_MASK)){
	proxima = 1;
	break;
	}
	if(notesI[i]==0){
	delay_ms(durationI[i]);
	duracao += durationI[i]*notesI[1];
	pio_clear(PIOA, BUZZER_PIO_IDX_MASK);
	}
	else{
	duracao += periodo*1000;
	pio_set(PIOA, BUZZER_PIO_IDX_MASK);
	delay_us(periodo*1000000/2);
	pio_clear(PIOA, BUZZER_PIO_IDX_MASK);
	delay_us(periodo*1000000/2);
	pio_clear(PIOA, BUZZER_PIO_IDX_MASK);
	}
	}
	}
	delay_ms(200);
	//**************MUSICA 2 - TEMA PIRATAS DO CARIBE************************

Ter três longos trechos de código repetitivos torna o entendimento do código bem mais complexo, dificulta inserir novas músicas, aumenta chance de erros em alterações de código.

Poderia haver uma função especifica para tocar a musica escolhida pelo usuário.

Autoria da música

APS1embarcados/apsBuzzer/src/main.c

Line 115 in b2b2de3

//*************MUSICA 1 - TEMA INDIANA JONES*****************

Faltou indicar o autor/editora e nome da música para dar direitos ao artista.