Arduino.ru

А что не так с delayMicroseconds или с micros? В чём проблема-то?

Если использовать только половину программы, т.е. только пин 2 - вход и 5 выход, программа вроде бы работает, но при добавление в обработку еще 2 пары пинов входа и выхода, "глюки" появляются. 


 #define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 500
int prevValue = 0;
int prevValue2 = 0;
long previousMicros = 0;



void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
}

  void loop()
{
 
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
    
   digitalWrite(led, HIGH);
digitalWrite(led, HIGH);
delayMicroseconds(delayTime); 
digitalWrite(led, LOW);
delayMicroseconds(2000); 
prevValue = LOW;
}

if ((prevValue2 == LOW) && (digitalRead(button2) == HIGH))
{

digitalWrite(led2, HIGH);
delayMicroseconds(delayTime);
digitalWrite(led2, LOW);
delayMicroseconds(2000); 

prevValue2= LOW;
}

}

Двигатель крутится примерно 5000 об/мин, при 3 даттчиках холла которые установлены относительно друг от друга в 120 градусах, идет быстрый опрос. Думаю из-за задержек delayMicroseconds()  или неправильного подбора 

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

Ничего не понял. Где пытаетесь? Почему нет кода с таймером?

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 1000
int prevValue = 0;
int prevValue2 = 0;
long previousMicros = 0;



void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
}

  void loop()
{
 
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
    unsigned long currentMicros = micros();
  if (currentMicros - previousMicros < delayTime)
  {
    previousMicros = currentMicros; 
   digitalWrite(led, HIGH);
  }

prevValue = LOW;
}
else 
{
  digitalWrite(led, HIGH);
}
}

в этом случае он постоянно горит в HIGH.

и правильно делает !

в какой строке вы его в LOW уводите  ?

если перевожу сюда, тоже не меняется.


#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 1000
int prevValue = 0;
int prevValue2 = 0;
long previousMicros = 0;



void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
}

  void loop()
{
 unsigned long currentMicros = micros();
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
      if (currentMicros - previousMicros < delayTime)
  {
    previousMicros = currentMicros; 
   digitalWrite(led, HIGH);
  }
else 
{
  digitalWrite(led, LOW);
}
prevValue = LOW;
}

}

внимательно на 28 строку взгляните

может таймер нужно вынести перед 

if (prevValue == LOW && digitalRead(button) == HIGH)?

какой таймер ?  я вижу вы и представления не имеете что такое таймер и как с ним работать.

про 28 строку я имел ввиду переменную previousMicros.

да если бы я все знал, вряд ли писал  в форумах. Наверное форумы для этого и созданы, чтобы помогать друг другу, а не упрекать человека что он чего-то не знает. ))

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 5000
int prevValue = 0;
int prevValue2 = 0;
long previousMicros = 0;



void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
}

  void loop()
{
 unsigned long currentMicros = micros();
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
      if (micros() - currentMicros < delayTime)
  {
    previousMicros = currentMicros; 
   digitalWrite(led, HIGH);
  }
else 
{
  digitalWrite(led, LOW);
  previousMicros=0;
}
prevValue = LOW;
}

}

если перевожу сюда, тоже не меняется.


#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 1000
int prevValue = 0;
int prevValue2 = 0;
long previousMicros = 0;



void setup()
{
  pinMode(led, OUTPUT);
  pinMode(led2, OUTPUT);
  pinMode(button, INPUT);
  pinMode(button2, INPUT);
}

void loop()
{
  unsigned long currentMicros = micros();
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
    if (currentMicros - previousMicros < delayTime)
    {
      previousMicros = currentMicros;
      digitalWrite(led, HIGH);
    }
    else
    {
      digitalWrite(led, LOW);
    }
    prevValue = LOW;
  }
}

вы думаете, наобум вставляя переменные и фукции, заработает ? нет !

я вам намекал что условие в 28 строке у вас не сработает ни разу !

потому что previousMicros не проинациализирована должным образом.

и это только первый "косяк".

Фо́рум (лат. forum — арх. преддверие гробницы; площадка в давильне для подлежащего обработке винограда; рыночная площадь, городской рынок; торжище, центральная площадь):

Форум (площадь) — типология общественного пространства в древнеримском градостроительстве; центральная городская площадь, где проходила городская жизнь, заключались сделки, велись переговоры.

Из вики. Определение. Где тут слова о ПОМОЩИ? И что понимать под помощью? Написать за вас программу? Или хватит подсказок? Например у вас 3 фазы и казалось бы должны быть три переменные которым присваивается значение времени при появлении сигнала на соответствующем пине и поднимается выход. Проверять три раза время и если где-то оно истекло выход снимать. Но если входной сигнал будет длиннее 300 мс что произойдёт? А если полный цикл будем меньше 900 мс что будет? Как избежать таких ситуаций?

Это Вы так думаете. На самом же деле жизнь устроена по-другому. Вот недавно такому как Вы давали вот эту ссылку. Там в третьем абзаце чётко и понятно написано, что на самом деле (а не в Вашем воображении) происходит, когда человек заходит на форум и требует написать ему код.

А что касается упрёков, так Вам сначала сказали "посмотрите на 28-ю" строку. Вы, вместо того, чтобы посмотреть и решить проблему (или хотя бы спросить "что с ней не так?") начали постить какие-то другие коды и возмущаться. что Вам не помогли. А решение Вам дали  - Вы его проигнорировали.

Регулятор BLDC мотора на датчиках Холла - ТАК не делается. В сети полно готовых прошивок для мег, но они все "на ассемблере" .. догадаетесь "почему"? :)

В целом, не майтесь. купите готовый контроллер мотора - это дешевле и проще.

я Вас понял....

посмотрите пожалуйста этот код. Выходной сигнал появился, но он никак не реагирует на мою переменную delayTime(длительность выходного сигнала), просто выходной сигнал в точности повторяет входной. 

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 500
int prevValue = 0;
int prevValue2 = 0;
uint32_t timeRising;


void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
//previousMicros = micros();
}

  void loop()
{
    if (prevValue == LOW && digitalRead(button) == HIGH)
  {
      digitalWrite(led, HIGH);
      timeRising = micros();
      
  }
      if ((micros() - timeRising)< delayTime)
  {
    digitalWrite(led, LOW);
    prevValue = LOW;
}
}

Может так

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 500
bool prevValue = 0;
int prevValue2 = 0;
uint32_t timeRising;



void setup() 
{
  pinMode(led, OUTPUT);
  pinMode(led2, OUTPUT);
  pinMode(button, INPUT);
  pinMode(button2, INPUT);
  //previousMicros = micros();
}

void loop()
{
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
    digitalWrite(led, HIGH);
    timeRising = micros();
    prevValue = 1;

  }
  if (prevValue == 1 && (micros() - timeRising)>= delayTime)
  {
    digitalWrite(led, LOW);
  }
  if (digitalRead(button)== LOW) prevValue = 0;
}

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 500
int prevValue = 0;
int prevValue2 = 0;
uint32_t timeRising;


void setup() 
{
pinMode(led, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(button, INPUT);
pinMode(button2, INPUT);
//previousMicros = micros();
}

  void loop()
{
    if (prevValue == LOW && digitalRead(button) == HIGH)
  {
      digitalWrite(led, HIGH);
      timeRising = micros();
      
  }
      if ((micros() - timeRising)< delayTime)
  {
    digitalWrite(led, LOW);
    prevValue = LOW;
}
}

реагирует, просто вы не видите погасание диода на единицы микросекунд. 

оооо получилось. Спасибо огромное всем!!!

я на осциллографе смотрю. видно))

Чуть подправил

#define button 2
#define button2 3
#define led 5
#define led2 6
#define delayTime 500
bool prevValue = 0;
int prevValue2 = 0;
uint32_t timeRising;



void setup() 
{
  pinMode(led, OUTPUT);
  pinMode(led2, OUTPUT);
  pinMode(button, INPUT);
  pinMode(button2, INPUT);
  //previousMicros = micros();
}

void loop()
{
  if (prevValue == LOW && digitalRead(button) == HIGH)
  {
    digitalWrite(led, HIGH);
    timeRising = micros();
    prevValue = 1;

  }
  if (prevValue == 1 && (micros() - timeRising)>= delayTime)
  {
    digitalWrite(led, LOW);
  }
  if (digitalRead(button)== LOW && digitalRead(led)== LOW ) prevValue = 0;
}

Давайте сначала малость ускорим:

Строка 22 if (prevValue == LOW && (PIND&B00000100==B00000100))

Строка 24 PORTD!=B00100000

Строка 30 ....

Строка 32 ....

Строка 34 .... не нужна нулите в 32

Если время между входными сигналами заведомо больше 300мс, то можно уменьшить количество проверок и действовать по следующему алгоритму:

1. При поступлении сигнала включаем выход и считываем значение времени

2. При прохождении 300мс выключаем выход.

По хорошему - это будет работать и при интервалах входных сигналов менее 300мс. Просто выход не будет выключаться.

Где-то так

void loop() {
if (digitalRead(button) == HIGH){
digitalWrite(led, HIGH);
timeRising = micros();
}
if ( (micros() - timeRising)>= delayTime) digitalWrite(led, LOW);
}

void loop() {
if (digitalRead(button) == HIGH){
digitalWrite(led, HIGH);
timeRising = micros();
}
if ( (micros() - timeRising)>= delayTime) digitalWrite(led, LOW);
}

выход здесь включается при нажатии на кнопку и и отключается через delayTime после отпускания ея.

Судя по программе delayTime будет осчитываться от момента когда button упадёт в LOW? А если он милисекунд 200 продержится, то на выходе импульс будет 300+200 = 500 мс? 

Уел :)

Правда атор поста не указывал, по какому фронту начинать отсчет и я принял время прохождения датчика несущественным. Но можно ковырнуть и так:

void loop() {
if (!prevValue && digitalRead(button)){
digitalWrite(led, HIGH);
prevValue=TRUE;
timeRising = micros();
}
if ( (micros() - timeRising)>= delayTime){
 digitalWrite(led, LOW);
prevValue=FALSE;
}
}

Согласен - будет срабатывать как попало, но тогда вся эта шняга смысла не имеет.

Кстати можно еще ускорить для более точного определения интервала, убрав операцию чтения во время подсчета времени:

void loop() {
if (!prevValue){
if (digitalRead(button)){
digitalWrite(led, HIGH);
prevValue=TRUE;
timeRising = micros();
}
}
if ( (micros() - timeRising)>= delayTime){
 digitalWrite(led, LOW);
prevValue=FALSE;
}
}

Никого  я не ел. Это ж элементарная логика из стартового сообщения:  нужно при входном HIGH сформировать сигнал длиной 300 мкс...

А теперь ещё два таких же блока для ещё двух входов и что будет, если длительность loop будет больше входного импульса?  

Скетч в #26 (http://arduino.ru/forum/programmirovanie/signal-opredelennoi-dliny-na-vy...) посту будет работать коректно при импульсе любой длины, но mykaida прав, при работе с микрос желательно переходить на управление портом на прямую!