Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Atualização para controles de motor #62

Open
vinebarbosa opened this issue Mar 11, 2020 · 1 comment
Open

Atualização para controles de motor #62

vinebarbosa opened this issue Mar 11, 2020 · 1 comment

Comments

@vinebarbosa
Copy link
Contributor

vinebarbosa commented Mar 11, 2020

Saudações a todos. Ultimamente, venho pensando em uma maneira melhor de controlar motores ultilizando esta lib. E por isso, gostaria de propor a senguintes alterações;

// Código de exemplo

#include "Brasilino.h"

Motor meu_motor(2, 3, 4);

funcao configurar() {
  meu_motor.configurarVelocidade(50); 
  meu_motor.girar(PARA_FRENTE);
}

funcao repetir() {

}

1- Que a velocidade que é passada como parâmetro da função configurarVelocidade não seja entre 0 e 255 e sim entre 0 e 100. Acredito que isso possa contribuir para o aprendizado de quem está começando.

2 - Substituir as funções girar e configurarVelocidade por:
acionar(velocidade)
ou somente
girar(velocidade)

O parâmetro velocidade estaria entre -100 e 100. O menos remete-se justamente ao sentido. Por exemplo, se eu passo 50 como parâmetro o motor giraria em um sentido com 50% da velocidade e se, por acaso, eu passar -50 como parâmetro o motor continua girando com 50% da velocidade, só que no sentido oposto.

O que vocês acham dessas modificações?

@SteffanoP
Copy link
Contributor

1- Que a velocidade que é passada como parâmetro da função configurarVelocidade não seja entre 0 e 255 e sim entre 0 e 100. Acredito que isso possa contribuir para o aprendizado de quem está começando.

Acho super válido!

2 - Substituir as funções girar e configurarVelocidade por:
acionar(velocidade)
ou somente
girar(velocidade)

Quanto a configurarVelocidade, me remete a configurar uma velocidade padrão para rotação sempre que acionada, tipo aqueles comandos de configuração que geralmente usamos no setup, que não é o caso aqui.

Quanto a eliminação do comando configurarVelocidade() ou acionar() para implementar em girar, já não acho tão interessante aqui, visto que irá tirar a função de acionamento analógico. Porém, há a possibilidade de adicionar o argumento de velocidade na questão da função girar(), algo como ter a funções: girar(sentido) e girar(sentido, velocidade). Sinceramente, na minha opinião, acho essa a melhor solução aqui para tal problema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants