что не так с программой? delay игнорируется

Ytm · Сообщение **Ytm** » Чт ноя 07, 2013 09:22:48

Собственно текст:

Спойлер

int switchPin = 2;
int ledPin = 3;
boolean lastButton = LOW;
boolean currentButton = LOW;
int ledLevel = 0;
unsigned long dTime = 0;
boolean runningScript = LOW;
unsigned long Delay = 5000;

void setup()
{
  pinMode(switchPin, INPUT);
  pinMode(ledPin, OUTPUT);
}

boolean debounce(boolean last)
{
  boolean current = digitalRead(switchPin);
  if (last !=current)
  {
    delay(5);
    current = digitalRead(switchPin);
  }
  return current;
}

void loop()
{
  currentButton = debounce(lastButton);

  if (lastButton == LOW && currentButton == HIGH)
  {
    if (runningScript == LOW)
    {
      runningScript = HIGH;
      ledLevel = 40;
      dTime = millis() + Delay;
    }
    else
    {
      runningScript = LOW;
    }
  }

  lastButton = currentButton;

  if ((runningScript == HIGH) && ledLevel && (ledLevel < 255))
  {
    if ((millis() - dTime) > 50)
    {
      ledLevel++;
      dTime = millis();
    }
  }
  if ((runningScript == LOW) && ledLevel && (ledLevel > 0))
  {
    if ((millis() - dTime) > 50)
    {
      ledLevel--;
      dTime = millis();
    }
  }

  analogWrite(ledPin, ledLevel);
}

А задача этого Delay сделать так чтобы при включении диод останавливался на 5 секунд в положении 40, а уже потом начиналось увеличение. Работает все кроме этой самой задержки в начале

Реклама

Flasher

а аргумент delay задается разве в секундах?

Реклама

eess9

Код: Выделить всё

delay(5);

собственная функция? сами писали?
попробуйте так:

Код: Выделить всё

#include<util/delay.h>

for(uint8_t i=0;i<50;i++) { _delay_ms(100);}

Ytm · Сообщение **Ytm** » Чт ноя 07, 2013 13:56:47

Flasher писал(а):а аргумент delay задается разве в секундах?

в милисекундах

eess9 писал(а):
Код: Выделить всё
delay(5);
собственная функция? сами писали?

нет, зачем? да и не delay, а Delay - это просто переменная

eess9 писал(а): попробуйте так:
Код: Выделить всё
#include<util/delay.h>

for(uint8_t i=0;i<50;i++) { _delay_ms(100);}

чет не соображу, к чему это все?

Реклама

vek · Сообщение **vek** » Чт ноя 07, 2013 14:01:26

Может я чего-то недопонимаю?
А под задержкой вы подразумеваете эту строчку:

Код: Выделить всё

delay(5)

???? Она не работает? А что там под юбкой у этой функции?

Реклама

Ytm · Сообщение **Ytm** » Чт ноя 07, 2013 15:38:08

нет. эти 5 милисекунд от дребезжания кнопки. там же написано "debounce". к светодиодам это не имеет никакого отношения.

Реклама

Flasher

Тогда приводите полный код и укажите точно нерабочий кусок.

eess9

Ничего не пойму. Ytm в самом первом посте идется о 5 секундах, теперь уже появились милисекунды. Вы определитесь
Функция delay() написана вами? Если да, то ее код в студию. Если нет, то просвятите меня в каком стандартном заголовочном файле она объявлена?

Ytm · Сообщение **Ytm** » Чт ноя 07, 2013 18:30:25

Наверное надо было начать с того что это ардуина

у нее свои библиотеки. Эта delay(5); никаким раком не относится к проблеме, она там просто чтобы дребезг кнопки убрать. и она действительно 5 милисекунд. Вот более полную версию выдрал из временной папки

Спойлер

Код: Выделить всё

#line 1 "test13.ino"
#include "Arduino.h"
void setup();
boolean debounce(boolean last);
void loop();
#line 1

int switchPin = 2;
int ledPin = 3;
boolean lastButton = LOW;
boolean currentButton = LOW;
int ledLevel = 0;
unsigned long dTime = 0;
boolean runningScript = LOW;
unsigned long Delay = 5000;

void setup()
{
  pinMode(switchPin, INPUT);
  pinMode(ledPin, OUTPUT);
}

boolean debounce(boolean last)
{
  boolean current = digitalRead(switchPin);
  if (last !=current)
  {
    delay(5);
    current = digitalRead(switchPin);
  }
  return current;
}

void loop()
{
  currentButton = debounce(lastButton);

  if (lastButton == LOW && currentButton == HIGH)
  {
    if (runningScript == LOW)
    { 
      ledLevel = 40;
    runningScript = HIGH;
      dTime = millis() + Delay;
    }
    else
    {
      runningScript = LOW;
    }
  }

  lastButton = currentButton;

  if ((runningScript == HIGH) && ledLevel && (ledLevel < 255))
  {
    if ((millis() - dTime) > 10)
    {
      ledLevel++;
      dTime = millis();
    }
  }
  if ((runningScript == LOW) && ledLevel && (ledLevel > 0))
  {
    if ((millis() - dTime) > 5)
    {
      ledLevel--;
      dTime = millis();
    }
  }

  analogWrite(ledPin, ledLevel);
}

Вот Arduino.h:

Спойлер

Код: Выделить всё

#ifndef Arduino_h
#define Arduino_h

#include <stdlib.h>
#include <string.h>
#include <math.h>

#include <avr/pgmspace.h>
#include <avr/io.h>
#include <avr/interrupt.h>

#include "binary.h"

#ifdef __cplusplus
extern "C"{
#endif

#define HIGH 0x1
#define LOW  0x0

#define INPUT 0x0
#define OUTPUT 0x1
#define INPUT_PULLUP 0x2

#define true 0x1
#define false 0x0

#define PI 3.1415926535897932384626433832795
#define HALF_PI 1.5707963267948966192313216916398
#define TWO_PI 6.283185307179586476925286766559
#define DEG_TO_RAD 0.017453292519943295769236907684886
#define RAD_TO_DEG 57.295779513082320876798154814105

#define SERIAL  0x0
#define DISPLAY 0x1

#define LSBFIRST 0
#define MSBFIRST 1

#define CHANGE 1
#define FALLING 2
#define RISING 3

#if defined(__AVR_ATtiny24__) || defined(__AVR_ATtiny44__) || defined(__AVR_ATtiny84__) || defined(__AVR_ATtiny25__) || defined(__AVR_ATtiny45__) || defined(__AVR_ATtiny85__)
#define DEFAULT 0
#define EXTERNAL 1
#define INTERNAL 2
#else  
#if defined(__AVR_ATmega1280__) || defined(__AVR_ATmega2560__) || defined(__AVR_ATmega1284__) || defined(__AVR_ATmega1284P__) || defined(__AVR_ATmega644__) || defined(__AVR_ATmega644A__) || defined(__AVR_ATmega644P__) || defined(__AVR_ATmega644PA__)
#define INTERNAL1V1 2
#define INTERNAL2V56 3
#else
#define INTERNAL 3
#endif
#define DEFAULT 1
#define EXTERNAL 0
#endif

// undefine stdlib's abs if encountered
#ifdef abs
#undef abs
#endif

#define min(a,b) ((a)<(b)?(a):(b))
#define max(a,b) ((a)>(b)?(a):(b))
#define abs(x) ((x)>0?(x):-(x))
#define constrain(amt,low,high) ((amt)<(low)?(low):((amt)>(high)?(high):(amt)))
#define round(x)     ((x)>=0?(long)((x)+0.5):(long)((x)-0.5))
#define radians(deg) ((deg)*DEG_TO_RAD)
#define degrees(rad) ((rad)*RAD_TO_DEG)
#define sq(x) ((x)*(x))

#define interrupts() sei()
#define noInterrupts() cli()

#define clockCyclesPerMicrosecond() ( F_CPU / 1000000L )
#define clockCyclesToMicroseconds(a) ( (a) / clockCyclesPerMicrosecond() )
#define microsecondsToClockCycles(a) ( (a) * clockCyclesPerMicrosecond() )

#define lowByte(w) ((uint8_t) ((w) & 0xff))
#define highByte(w) ((uint8_t) ((w) >> 8))

#define bitRead(value, bit) (((value) >> (bit)) & 0x01)
#define bitSet(value, bit) ((value) |= (1UL << (bit)))
#define bitClear(value, bit) ((value) &= ~(1UL << (bit)))
#define bitWrite(value, bit, bitvalue) (bitvalue ? bitSet(value, bit) : bitClear(value, bit))


typedef unsigned int word;

#define bit(b) (1UL << (b))

typedef uint8_t boolean;
typedef uint8_t byte;

void init(void);

void pinMode(uint8_t, uint8_t);
void digitalWrite(uint8_t, uint8_t);
int digitalRead(uint8_t);
int analogRead(uint8_t);
void analogReference(uint8_t mode);
void analogWrite(uint8_t, int);

unsigned long millis(void);
unsigned long micros(void);
void delay(unsigned long);
void delayMicroseconds(unsigned int us);
unsigned long pulseIn(uint8_t pin, uint8_t state, unsigned long timeout);

void shiftOut(uint8_t dataPin, uint8_t clockPin, uint8_t bitOrder, uint8_t val);
uint8_t shiftIn(uint8_t dataPin, uint8_t clockPin, uint8_t bitOrder);

void attachInterrupt(uint8_t, void (*)(void), int mode);
void detachInterrupt(uint8_t);

void setup(void);
void loop(void);

// Get the bit location within the hardware port of the given virtual pin.
// This comes from the pins_*.c file for the active board configuration.

#define analogInPinToBit(P) (P)

// On the ATmega1280, the addresses of some of the port registers are
// greater than 255, so we can't store them in uint8_t's.
extern const uint16_t PROGMEM port_to_mode_PGM[];
extern const uint16_t PROGMEM port_to_input_PGM[];
extern const uint16_t PROGMEM port_to_output_PGM[];

extern const uint8_t PROGMEM digital_pin_to_port_PGM[];
// extern const uint8_t PROGMEM digital_pin_to_bit_PGM[];
extern const uint8_t PROGMEM digital_pin_to_bit_mask_PGM[];
extern const uint8_t PROGMEM digital_pin_to_timer_PGM[];

// Get the bit location within the hardware port of the given virtual pin.
// This comes from the pins_*.c file for the active board configuration.
// 
// These perform slightly better as macros compared to inline functions
//
#define digitalPinToPort(P) ( pgm_read_byte( digital_pin_to_port_PGM + (P) ) )
#define digitalPinToBitMask(P) ( pgm_read_byte( digital_pin_to_bit_mask_PGM + (P) ) )
#define digitalPinToTimer(P) ( pgm_read_byte( digital_pin_to_timer_PGM + (P) ) )
#define analogInPinToBit(P) (P)
#define portOutputRegister(P) ( (volatile uint8_t *)( pgm_read_word( port_to_output_PGM + (P))) )
#define portInputRegister(P) ( (volatile uint8_t *)( pgm_read_word( port_to_input_PGM + (P))) )
#define portModeRegister(P) ( (volatile uint8_t *)( pgm_read_word( port_to_mode_PGM + (P))) )

#define NOT_A_PIN 0
#define NOT_A_PORT 0

#ifdef ARDUINO_MAIN
#define PA 1
#define PB 2
#define PC 3
#define PD 4
#define PE 5
#define PF 6
#define PG 7
#define PH 8
#define PJ 10
#define PK 11
#define PL 12
#endif

#define NOT_ON_TIMER 0
#define TIMER0A 1
#define TIMER0B 2
#define TIMER1A 3
#define TIMER1B 4
#define TIMER2  5
#define TIMER2A 6
#define TIMER2B 7

#define TIMER3A 8
#define TIMER3B 9
#define TIMER3C 10
#define TIMER4A 11
#define TIMER4B 12
#define TIMER4C 13
#define TIMER4D 14	
#define TIMER5A 15
#define TIMER5B 16
#define TIMER5C 17

#ifdef __cplusplus
} // extern "C"
#endif

#ifdef __cplusplus
#include "WCharacter.h"
#include "WString.h"
#include "HardwareSerial.h"

uint16_t makeWord(uint16_t w);
uint16_t makeWord(byte h, byte l);

#define word(...) makeWord(__VA_ARGS__)

unsigned long pulseIn(uint8_t pin, uint8_t state, unsigned long timeout = 1000000L);

void tone(uint8_t _pin, unsigned int frequency, unsigned long duration = 0);
void noTone(uint8_t _pin);

// WMath prototypes
long random(long);
long random(long, long);
void randomSeed(unsigned int);
long map(long, long, long, long, long);

#endif

#include "pins_arduino.h"

#endif

Подсказали мне, в общем поменял я

Код: Выделить всё

      ledLevel = 40;
      runningScript = HIGH;
      dTime = millis() + Delay;

на

Код: Выделить всё

      ledLevel = 40;
      analogWrite(ledPin, ledLevel);
      runningScript = HIGH;
      delay(5000);
      dTime = millis();

eess9

Ну, поменял и...
Как результат?

Ytm · Сообщение **Ytm** » Чт ноя 07, 2013 21:48:24

помогло

vek · Сообщение **vek** » Пт ноя 08, 2013 01:49:53

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

Без хрустального шара не обойтись

Ytm · Сообщение **Ytm** » Пт ноя 08, 2013 11:26:13

ну библиотеки у всех ардуин стандартные, я подумал что это не проблема.

blackx

Да, только надо было еще в первом посте указать что у вас ардуино. Да и с ней лучше на форум ардуинщиков идти спрашивать.

Ytm · Сообщение **Ytm** » Пт ноя 08, 2013 11:35:21

да-да, моя ошибка. Вот только на форуме ардуинщиков этот вопрос висел два дня уже перед тем как я сюда его написал. да и в целом то язык тот-же самый. В общем получилось то что получилось. В любом случае спасибо всем кто отозвался.

eess9

Язык тот же, а библиотеки другие...
поди разберись