Почему не работает?!

Тема в разделе "Arduino & Shields", создана пользователем Рома-Ярик, 12 янв 2017.

Метки:
Страница 1 из 2
  1. Рома-Ярик

    Рома-Ярик Нуб

    И тут не работает -_- . (браузер виснет при попытке вставить код)
    Написал, скомпилировал. Но работают только два первых значения. Уже и пробовал делать их от 1 до 10, от 1 до 255 - не помогает. В чём дело?

    Мой код:

    #define FLEDPIN 4
    #define LLEDPIN 13
    int rotation, number;
    int brightness = 0;
    int fadeAmount = 5;
    void setup() {
    for (int pin = FLEDPIN; pin <= LLEDPIN; ++pin)
    pinMode (pin, OUTPUT);
    pinMode (3, OUTPUT);
    pinMode (A0,INPUT);
    }
    void loop() {
    rotation = analogRead(A0);
    if (rotation < 25) {
    number = 10;
    digitalWrite (13, HIGH);
    digitalWrite (12, LOW);
    digitalWrite (11, LOW);
    digitalWrite (10, LOW);
    digitalWrite (9, LOW);
    digitalWrite (8, LOW);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (25<rotation<50) {
    number = 20;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, LOW);
    digitalWrite (10, LOW);
    digitalWrite (9, LOW);
    digitalWrite (8, LOW);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (50<rotation<75) {
    number = 30;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, LOW);
    digitalWrite (9, LOW);
    digitalWrite (8, LOW);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (75<rotation<100) {
    number = 40;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, LOW);
    digitalWrite (8, LOW);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (100<rotation<125) {
    number = 50;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, LOW);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (125<rotation<150) {
    number = 60;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, HIGH);
    digitalWrite (7, LOW);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (150<rotation<175) {
    number = 70;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, HIGH);
    digitalWrite (7, HIGH);
    digitalWrite (6, LOW);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (175<rotation<200) {
    number = 80;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, HIGH);
    digitalWrite (7, HIGH);
    digitalWrite (6, HIGH);
    digitalWrite (5, LOW);
    digitalWrite (4, LOW);
    } else if (200<rotation<225) {
    number = 90;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, HIGH);
    digitalWrite (7, HIGH);
    digitalWrite (6, HIGH);
    digitalWrite (5, HIGH);
    digitalWrite (4, LOW);
    } else if (225<rotation) {
    number = 100;
    digitalWrite (13, HIGH);
    digitalWrite (12, HIGH);
    digitalWrite (11, HIGH);
    digitalWrite (10, HIGH);
    digitalWrite (9, HIGH);
    digitalWrite (8, HIGH);
    digitalWrite (7, HIGH);
    digitalWrite (6, HIGH);
    digitalWrite (5, HIGH);
    digitalWrite (4, HIGH);
    }
    analogWrite(3, brightness);
    brightness = brightness + fadeAmount;
    if (brightness <= 0 || brightness >= 255) {
    fadeAmount = -fadeAmount;
    }
    delay(number);
    }

    ---------
    Да что такое! Уже и jpg-овские фотки не понимает! Окей, и тут выкрутимся.
    Я сделал такую схему, что используются три вещи: переменный резистор, светодиодная шкала и отдельный светодиод.
    Светодиод сделан он по самой обычной схеме - 3 пин, сам светодиод, резистор на 220 Ом и земля.
    Светодиодная лента - тоже самое, только с 13 по 4 пины. Да, и резистор в последней цепи (от 4 пина) используется так же светодиодом.Не хватило мне резисторов.
    А переменный резистр подключён к А0, 5 вольтам и земле.
    Надеюсь что сумел описать схему.
    А теперь проблемма: как бы я ни крутил ручку переменного резистора, он, похоже, выдаёт не более 200, хотя должен от 0 до 1023. и в итоге у меня загораются только два деления на шкале, и светодиод мерцает только с первой и второй скоростью.
    Прошу помо
     
    Рома-Ярик, 12 янв 2017
    #1
  2. rkit

    rkit Гуру

    А почему вы решили, что ваш резистор может выдавать 0?
     
    rkit, 12 янв 2017
    #2
  3. Рома-Ярик

    Рома-Ярик Нуб

    PS: по поводу оптимизации не ругайтесь, это моя вторая прога для ардуинки. Певая была почти такой-же, но без шкалы. И тоже работала только на первые два значения.
     
    Рома-Ярик, 12 янв 2017
    #3
  4. Рома-Ярик

    Рома-Ярик Нуб

    Так написано в книге, которая прилагалась к набору
     
    Рома-Ярик, 12 янв 2017
    #4
  5. Рома-Ярик

    Рома-Ярик Нуб

    Цитирую книгу: "считываем в rotation напряжение с потенциометра:микроконтроллер выдаст число от 0 до 1023 пропорциональное углу поворота ручки"
     
    Рома-Ярик, 12 янв 2017
    #5
  6. rkit

    rkit Гуру

    Тогда проверяйте качество контактов
     
    rkit, 12 янв 2017
    #6
  7. Рома-Ярик

    Рома-Ярик Нуб

    Отлично, первая версия готова. Ещё какие будут?
     
    Рома-Ярик, 12 янв 2017
    #7
  8. Рома-Ярик

    Рома-Ярик Нуб

    Переставил в другое место переменный резистр - то же самое. Сопротивление проводов по 0.5. А сам п. резистор должен прозваниваться?
     
    Рома-Ярик, 12 янв 2017
    #8
  9. Tomasina

    Tomasina Сушитель лампочек Модератор

    Та хрень, что у вас в коде, работать вообще не должна. Какая то дикая смесь Си и Паскаля.
     
    Tomasina, 12 янв 2017
    #9
  10. VladislavSum

    VladislavSum Нуб

    Код конечно страшненький, но вполне рабочий. В каком месте там Паскаль?

    Тс, попробуй сделать вывод значений с потенциометра в serial и посмотри реальные значения.
     
    VladislavSum, 12 янв 2017
    #10
  11. Tomasina

    Tomasina Сушитель лампочек Модератор

    Код (C++):
    if (25<rotation<50)
    А0 подключен к среднему контакту переменного резистора?
     
    Tomasina, 12 янв 2017
    #11
  12. Unixon

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

    Это "if (25<rotation<50)" в Паскале тоже некорректно.
     
    Unixon, 12 янв 2017
    #12
  13. VladislavSum

    VladislavSum Нуб

    Действительно странно выглядит, однако это компилируются. Проверить на ардуинке сейчас возможности нет
     
    VladislavSum, 12 янв 2017
    #13
  14. Tomasina

    Tomasina Сушитель лампочек Модератор

    Компилируется потому что содержимое скобок не противоречит синтаксису. Но результат обработки такого выражения окажется совсем не тот, что задуман ТС.

    Там вообще весь loop() можно сократить до 12 строк.
     
    Tomasina, 12 янв 2017
    #14
  15. VladislavSum

    VladislavSum Нуб

    Вот тут мне стало интересно, завтра почитаю-поэксперементирую что будет если написать такое выражение.

    UPD. Догадался, значение первого < вернет 0 или 1 и будет сравнено с последним операндом.
     
    Последнее редактирование: 12 янв 2017
    VladislavSum, 12 янв 2017
    #15
  16. Tomasina

    Tomasina Сушитель лампочек Модератор

    Элементарно. Результатом скобок всегда будет true, т.е. будет выполняться только второе if.

    И вот почему:
    (25<rotation<50) - разбор идет слева направо, т.е. у выражения (25<rotation) будет на выходе true или false, а это 1 или 0 соответственно.
    Далее сравнивается вторая половина: (1<50) или (0<50). На выходе в любом варианте получим true, т.к. выражения истинны.
    Поэтому почти всегда выполняется блок кода, где number = 20; (изредка там где number = 10;), а до остальных блоков просто не доходит очередь.
     
    Tomasina, 12 янв 2017
    #16
  17. Airbus

    Airbus Радиохулиган Модератор

    Э!Тут switch-case (rotation) нужен.Это что будет?Типа цифровая шкала?Что делает number?
     
    Airbus, 12 янв 2017
    #17
  18. Tomasina

    Tomasina Сушитель лампочек Модератор

    number не нужен, можно и case использовать, если локально привести к диапазону 1-10.
     
    Tomasina, 12 янв 2017
    #18
  19. MDV

    MDV Гик

    Стоит попробовать так
    if(rotation > 25 && rotation < 50)
     
    MDV, 12 янв 2017
    #19
  20. Tomasina

    Tomasina Сушитель лампочек Модератор

    это лишнее, достаточно одной проверки, т.к. последующее if перекрывает нужный диапазон.

    Да и вообще - ни if, ни case тут не нужны, достаточно map.
    Вот полная замена loop(), с той же функциональностью:
    Код (C++):
    void loop()
    {
      rotation = analogRead(A0);
      number = map(rotation, 0, 1023, FLEDPIN, LLEDPIN + 1);  // ужимаем диапазон до 10 разрядов, минимальная величина - 2 светодиода
      for (byte pin = FLEDPIN; pin <= LLEDPIN; pin++)
        digitalWrite(pin, number >= pin ? HIGH : LOW);        //***
      analogWrite(3, brightness);
     
      brightness = brightness + fadeAmount;
      if (brightness <= 0 || brightness >= 255) fadeAmount = -fadeAmount;
      delay(number * 10);
    }

    //*** это краткая запись такого выражения:
    // if (number >= pin) digitalWrite(pin, HIGH);
    // else digitalWrite(pin, LOW);
     
    Последнее редактирование: 12 янв 2017
    Tomasina, 12 янв 2017
    #20
Страница 1 из 2