RC передатчик и Arduino UNO

roggedhorse · 11 июн 2013

Хотя нет, я оказался не прав.
Приведенный пример работает, но код неэффективный: слишком много вычислений в обработчике.
Посему я все же закончу свой

Unixon · 11 июн 2013

Разрешите немного позанудствовать...

В этом коде мне решительно не нравится оформление идентификаторов, а именно тот факт, что макроопределения, переменные и функции оформлены в одинаковом стиле. Это затрудняет понимание. Отсутствие разницы в оформлении различных типов идентификаторов не позволяет по одному только виду имени однозначно сказать, что это такое. И еще в условиях оператора if (особенно длинных) я бы рекомендовал расставлять скобки в нужных местах для визуального выделения структуры выражения, даже если приоритет операций позволяет этого не делать.

pyro сказал(а): ↑

Код (C):

#define lmotPWM 10 //Присваеваем имена пинам управления
#define lmotCONT1 9
#define lmotCONT2 8
#define rmotPWM 5
#define rmotCONT1 6
#define rmotCONT2 7
//назначение глобальных переменных
long pulseWidth[3] = {
0,0,0};
void setup()
//настройка вх/вых
{
// Serial.begin(9600);
pinMode(11, INPUT);
pinMode(12, INPUT);
pinMode(13, INPUT);
for (int i = 5; i <= 10; i++)
{
pinMode(i, OUTPUT);
digitalWrite(i, LOW);
}
}
void loop()
{
for (int x=2; x<=4; x++) //измеряем длину импульса
{
unsigned long z = pulseIn(x, HIGH, 20000);
pulseWidth[x-2] = z; //записываем значения в массив
}
if(pulseWidth[2] != 0 && pulseWidth[0] != 0)
{
if(pulseWidth[0] < 1440 || pulseWidth[0] > 1470)
{
if(pulseWidth[0] < 1440)
{
//Serial.println("B");
digitalWrite(lmotCONT1, 1);
digitalWrite(lmotCONT2, 0);
digitalWrite(rmotCONT1, 1);
digitalWrite(rmotCONT2, 0);
digitalWrite(lmotPWM, 1);
digitalWrite(rmotPWM, 1);

}
if(pulseWidth[0] > 1470)
{
//Serial.println("F");

digitalWrite(lmotCONT1, 0);
digitalWrite(lmotCONT2, 1);
digitalWrite(rmotCONT1, 0);
digitalWrite(rmotCONT2, 1);
digitalWrite(lmotPWM, 1);
digitalWrite(rmotPWM, 1);
}
}
else
{
if(pulseWidth[2] < 1440)
{

//Serial.println("L");
digitalWrite(lmotCONT1, 0);
digitalWrite(lmotCONT2, 1);
digitalWrite(rmotCONT1, 1);
digitalWrite(rmotCONT2, 0);
digitalWrite(lmotPWM, 1);
digitalWrite(rmotPWM, 1);
}
if(pulseWidth[2] > 1470)
{

//Serial.println("R");
digitalWrite(lmotCONT1, 1);
digitalWrite(lmotCONT2, 0);
digitalWrite(rmotCONT1, 0);
digitalWrite(rmotCONT2, 1);
digitalWrite(lmotPWM, 1);
digitalWrite(rmotPWM, 1);
}
}
if(pulseWidth[0] > 1440 && pulseWidth[0] < 1470 && pulseWidth[2] > 1440 && pulseWidth[2] < 1470)
{
//Serial.println("STOP");

digitalWrite(lmotPWM, 0);
digitalWrite(rmotPWM, 0);
digitalWrite(lmotCONT1, 0);
digitalWrite(lmotCONT2, 0);
digitalWrite(rmotCONT1, 0);
digitalWrite(rmotCONT2, 0);

//Serial.println(" ");
/*Serial.print(pulseWidth[0]);
Serial.print(" ");
Serial.print(pulseWidth[1]);
Serial.print(" ");
Serial.print(pulseWidth[2]);
Serial.print(" ");
//Serial.print(pulseWidth[3]);
Serial.print(" ");
//Serial.print(pulseWidth[4]);
Serial.println(" ");*/
}
}
else
{
digitalWrite(lmotCONT1, 0);
digitalWrite(lmotCONT2, 1);
digitalWrite(rmotCONT1, 0);
digitalWrite(rmotCONT2, 1);
digitalWrite(lmotPWM, 0);
digitalWrite(rmotPWM, 0);
}
}

Нажмите, чтобы раскрыть...

pyro · 21 июн 2013

Разрешите немного позанудствовать...
Нажмите, чтобы раскрыть...

А можно по конкретнее, что именно не нравиться, какие именно идентификаторы ненравятся. И если есть желание предложите свой вариант. Я только учусь, и мне важны ваши советы.

Unixon · 21 июн 2013

pyro сказал(а): ↑

А можно по конкретнее, что именно не нравиться, какие именно идентификаторы ненравятся. И если есть желание предложите свой вариант. Я только учусь, и мне важны ваши советы.
Нажмите, чтобы раскрыть...

Ну, например, можете ли вы, не глядя в код, в место где определяется идентификатор, сходу сказать lmotPWM или pulseWidth - это переменная или константа, локальная или глобальная, или это вовсе макрос (#define) ?

pyro · 21 июн 2013

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

А что надо сделать, что бы это стало понятно?

Megakoteyka · 21 июн 2013

pyro сказал(а): ↑

А что надо сделать, что бы это стало понятно?
Нажмите, чтобы раскрыть...

Использовать стандарт оформления кода:
Стандарт оформления кода
90 рекомендаций по стилю написания программ на C++
Посмотрите, как написаны примеры в Вики. Там и небольшое отступление было по поводу оформления кода...

zsm · 27 май 2015

Подниму старую тему)
Друзья, подскажите пожалуйста не видел ли кто реализации на arduino регулятора скорости для коллекторного двигателя(с реверсом) управляемого стандартным RC сигналом ?

bigscorp · 3 апр 2016

Доброго!
Можете подкинуть информации по работе с приемником ED7000 Microlite receiver DSM2 на Arduino?
Требуется снимать сигналы для управления сервами и моторами, в общем турель мутим.

Zazila · 23 апр 2016

Всем привет! Я вообще не втыкаю в эти прерывания всякие, а очень нужно понять) Можете простенький скетч написать как, например, на какой то пин подавать сигнал длинной X(мс) с паузой Y(мс) ? Буду очень благодарен)

Игорюшка Владимирович · 17 май 2016

roggedhorse сказал(а): ↑

Посему я все же закончу свой
Нажмите, чтобы раскрыть...

Неужели перестало быть интересным ?

RC передатчик и Arduino UNO

roggedhorse Гик

Unixon Оракул Модератор

pyro Гик

Unixon Оракул Модератор

pyro Гик

Megakoteyka Оракул Модератор

zsm Гик

bigscorp Нуб

Zazila Нерд

Игорюшка Владимирович Гик

Быстрый поиск

RC передатчик и Arduino UNO

roggedhorse Гик

Unixon Оракул Модератор

pyro Гик

Unixon Оракул Модератор

pyro Гик

Megakoteyka Оракул Модератор

zsm Гик

bigscorp Нуб

Zazila Нерд

Игорюшка Владимирович Гик