Arduino.ru

Код приложите, а то как то видно плохо строки с 15 по 25

#include <EEPROM.h>
#include <enc28j60.h>
#include <EtherCard.h>
#include <net.h>
#include <IPAddress.h>
#include <HX711.h>
#define STATIC 0  // set to 1 to disable DHCP (adjust myip/gwip values below)

///HX711///////
HX711 scale(3, 2);                          

float calibration_factor;          // калибровка!
float units;
float ounces;
float divider;
float pluser;
//////////////

#if STATIC
// ethernet interface ip address
static byte myip[] = { 192,168,182,251 };
// gateway ip address
static byte gwip[] = { 192,168,182,111 };
static byte destip[]= {192,168,182,250};
static int myport=4220,destport=4020;
#endif
static byte mymac[] = { 0x1A,0x2B,0x3C,0x4D,0x5E,0x01 };
byte Ethernet::buffer[700];
static uint32_t timer;
static uint32_t timer1;
static byte hisip[] = { 192,168,182,250 };
static byte dnsip[] = { 192,168,182,111 };
//const char website[] PROGMEM = "192.168.182.220";
const int dstPort PROGMEM = 4020;

const int srcPort PROGMEM = 4220;

String str;
int num = 0;
char cstr[20];

String str1;
int num1 = 0;
char cstr1[20];

float calibr;

//callback that prints received packets to the serial port
void udpSerialPrint(uint16_t dest_port, uint8_t src_ip[IP_LEN], uint16_t src_port, const char *data, uint16_t len){
  IPAddress src(src_ip[0],src_ip[1],src_ip[2],src_ip[3]);

  Serial.print("dest_port: ");
  Serial.println(dest_port);
  Serial.print("src_port: ");
  Serial.println(src_port);
  
  
  Serial.print("src_ip: ");
  ether.printIp(src_ip);
  Serial.println();
  Serial.print("data: ");
  Serial.println(data);

String d1;
float d2;
d1 = strtok(data,":");
d2 = atof(strtok(NULL,":"));

float eeprom_write = d2;

if (d1 == "CALIBR") {
  EEPROM_float_write(0, eeprom_write);
  calibration_factor = EEPROM_float_read(0);
  scale.set_scale();
  scale.set_scale(calibration_factor);       //Применяем калибровку
  Serial.print("EEPROM calibration_factor: ");
  Serial.println(calibration_factor);
  }

if (d1 == "DIVIDER") {
  EEPROM_float_write(4, eeprom_write);
  divider = EEPROM_float_read(4);
  Serial.print("EEPROM divider: ");
  Serial.println(divider);
  }
if (d1 == "PLUSER") {
  EEPROM_float_write(8, eeprom_write);
  pluser = EEPROM_float_read(8);
  Serial.print("EEPROM pluser: ");
  Serial.println(pluser);
  }

if (d1 == "RESET") {digitalWrite(4, LOW);}
if (d1 == "TARE") {
  scale.tare();
  pluser=d2;
  }

}

void setup () {
digitalWrite(4, HIGH);
delay(200); 
pinMode(4, OUTPUT);
  Serial.begin(9600);

  if (ether.begin(sizeof Ethernet::buffer, mymac) == 0) 
    Serial.println( "Failed to access Ethernet controller");
#if STATIC
  ether.staticSetup(myip, gwip);
ether.copyIp(ether.dnsip, dnsip);
#else
  if (!ether.dhcpSetup())
    Serial.println(F("DHCP failed"));
#endif
ether.copyIp(ether.hisip, hisip);
  ether.printIp("IP:  ", ether.myip);
  ether.printIp("GW:  ", ether.gwip);  
  ether.printIp("DNS: ", ether.dnsip);  
  ether.printIp("SRV: ", ether.hisip);

  //register udpSerialPrint() to port 4220
  ether.udpServerListenOnPort(&udpSerialPrint, 4220);

//////HX711//////////
  scale.set_scale();
  scale.set_scale(calibration_factor);       //Применяем калибровку
////////////////////
}
char tag[] = {"WEIGHT:"};

void loop () {  
    if (millis() > timer1) {
      timer1 = millis() + 250; 
units = scale.get_units(), 10;
ounces = units * 0.035274;

float w2 = (ounces/divider)+pluser;
str1 = tag+String(w2);
str1.toCharArray(cstr1,20);

ether.sendUdp(cstr1, sizeof cstr1, 4001, ether.hisip, 4120 );
    }
ether.packetLoop(ether.packetReceive());
    if (millis() > timer) {
      timer = millis() + 1000;  
      
units = scale.get_units(), 10;
ounces = units * 0.035274;

float w1 = (ounces/divider)+pluser;
str = tag+String(w1);
str.toCharArray(cstr,20);

     ether.sendUdp(cstr, sizeof cstr, srcPort, ether.hisip, dstPort );   
  }

}///end loop

void EEPROM_float_write(int addr, float val) // запись в ЕЕПРОМ
{  
  byte *x = (byte *)&val;
  for(byte i = 0; i < 4; i++) EEPROM.write(i+addr, x[i]);
}

float EEPROM_float_read(int addr) // чтение из ЕЕПРОМ
{    
  byte x[4];
  for(byte i = 0; i < 4; i++) x[i] = EEPROM.read(i+addr);
  float *y = (float *)&x;
  return y[0];
}

Пример "неправильного" вывода, плииз

Схема на базе этого:

Пояснение к коду:
К этому подключен АЦП HX711, конфигурируется, настраивается UDP порт на приём 4220 и два UDP порта на передачу, 4020 и 4120. IP адрес получает автоматически, также предусмотрено включение конфигурации статического адреса. При старте считываются калибровоные значения, значение деления и значение смещения. В цикле частотой 0.25 сек. измеряется вес и оправляется на IP адрес 192,168,182,250 на порт 4120  и с частотой 1 сек. отправляется на тот-же адрес на порт 4020. И слушает порт 4220, чтобы принимать команды управления. При поступлении команд, выполняются операции изменения калибровочного значения, делителя и смещения шкалы и записываются в EEPROM.

Из всего сказанного, делаем смелый вывод что указанный "в терминале выводит" некое устройство, а в Serial все выводит правильно. Так? Тогда ищите в терминале.Для очистки совести проконтрольте что уходит в передачу в 141 и 154

Пример неправильного вывода:
26.51.23
19.45.23
26.43.23
23.68.23
22.77.23
Вывод после сброса:
23.54
26.51
21.85
17.05
25.02

del

А если в районе 50 строки вставить 

data[len]=0;

;)

[q]

может быть это нормальное явление с float числами.

[/quote]

float  тут не при чем. Где-то со строками нахомутали

Одинаково что? неправильный вывод?

Добавьте в 141 (154) вывод в Serial  cstr (cstr1)

Да, неправильный вывод со временем появляется везде и по сети и в сом порту, после сброса, всё востанавливается, начинает выводить правильно.
Вывод в сериал мне не нужен, потому, что мне нужно то, что он отсылает в сеть. Вывод в сериал я добавлял, когда таестировал логику работы, как только всё стало работать, я удалил эти строки из кода. Думаю зачем он зря будет в сериал еще выводить?
Да, я понял, только дошло, сейчас проверю. Но искажается уже в переменной w2.
Сейчас наверно переароверю все переменные наверно, выведу их все и посмотрю...

сейчас у Вас вывод в сом только в коллбеке. А надо проверить, что именно отдаем на передачу. Хотя дело то Ваше.

Да ладно!!!! float не может быть с двумя дробными частями :)

Я бы от String избавился напрочь для начала и попробовал изобразить что-то из этих запчастей:

char buffer[20];
char tag[]={"WEIGHT:"};
float floatVar=123.456;
...
memset(buffer, 0x00, sizeof(buffer))
memcpy(buffer, tag);
...
dtostrf(floatVar,4,2,&buffer[sizeof(tag)]);
ether.sendUdp(buffer, strlen(buffer), ...);
..

PS. Правда, я float избегать пытаюсь в 98% случаев, а для конвертации псевдофлоата в char[] у меня своя функция...

Спасибо, вы меня хоть оживили немного, а то я вообще тут немного зашился)

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

кто-то тебе в строке терминальный \0 затираеть

Да и вообще - вычисляли бы на стороне сервера свои дробные числа. Ардуине бы попроще было и вам тоже.

HX711.h:

// waits for the chip to be ready and returns a reading
long read();
...
// returns (read_average() - OFFSET), that is the current value without the tare weight; times = how many readings to do
double get_value(byte times = 1);

// returns get_value() divided by SCALE, that is the raw value divided by a value obtained via calibration
// times = how many readings to do
float get_units(byte times = 1);

Romhik, по-моему все дело в элементарной ошибке. Если я правильно прочитал описание, функция String.toCharArray() не добавляет к полученной строке нулевой терминатор.

Вставте на строку 141 вот такой оператор и будет вам счастье:

cstr1[str1.length()] = '\0';

Если перед неправильным выводом был правильный типа "12345.23", то b707 прав.

Ааааа... ну тут же суржик :) Все дружно смотрим String.c_str()

И последний вопрос, как обнулить или очистить быстро string array, то есть массив строк cstr1?
Что-то не могу найти быстрое решение пока.

Так пойдёт?

for( int i = 0; i < 20;  ++i ){cstr1[i]= {""};}

for( int i = 0; i < 20;  ++i ){cstr1[i]= {""};}

Лучше все позиции в массиве забить нулями '\0'

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

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

А если строка пишется в массив в цикле - как у автора? На каждой итерации весь буфер заново нулями забивать? Неужели так трудно вычислить длину строки, которую только что сам и создал?  :)

cstr1[20]= {'\0'};

Так?

Да я вообще не особо вчитывался в эту адскую смесь со String и Ethercard. stlren() все равно бежит по массиву, так же как и memset(), так что оверхед небольшой на помещение 0x00 в ячейку, а перфекционизм удовлетворен. Я и так и сяк пользуюсь. 

cstr1[str1.length()] = '\0';

У меня сейчас так и сделано, правда один раз выскочила вторая запятая, без подключенного АЦП, потом сбросил и всё пропало, сейчас уже долго показывает нормально.
Если всё нормально будет, то оставлю так, если снова приклеится вторая запятая, то попробую очищать весь массив.

cstr1[20]= {'\0'};

неправильно - в конце записываемых данных.

Я вам строчку уже давал.

memset(str1, 0x00, sizeof(str1));

У меня выдаёт ошибку:
exit status 1
cannot convert 'String' to 'void*' for argument '1' to 'void* memset(void*, int, size_t)'
Наверно я не понял как этой функцией пользоваться, пойду курить мануалы...
 

Что-то вы то за str1 , то за cstr1 хватаетесь. 

...лучше поймите, что String class вам тут все карты путает - вы то туда, то сюда. Имхо - связались с Ethercard - используйте везде char[].

М-да уж), сбивают с толку все эти преобразования, да и сам себя запутал, взял строку вместо массива)))
Пора отдыхать)
Спасибо!
 

memset(str1, 0x00, sizeof(str1));

Этот код работает.
Нужно действительно всё на массив переделать, спасибо еще раз!

Здравствуйте, всё хорошо работает, прошли сутки, нареканий нет!
У меня остался еще один вопрос к ГУРУ, не подскажите, что означает последняя цифра "10" в этой строке:

units = scale.get_units(), 10;

Работает одинаково как с цифрой 10, так и без неё.
Заранее спасибо.

в том виде, как написано это означает:  "присвоить переменной units значение функции scale.get_units(). И 10." 

примерно так. 

https://msdn.microsoft.com/ru-ru/library/zs06xbxh.aspx

void loop() { 

  Serial.print("Reading: ");
  
  for(int i = 0;i < 10; i ++) units =+ scale.get_units(), 10;   // усредняем показания считав 10 раз 
  units / 10;                                                   // делим на 10
   
  ounces = units * 0.035274;                                    // переводим унции в граммы              
  Serial.print(ounces);                                          // отправляем в монитор порта
  Serial.print(" grams");  
  Serial.println();

 
}

Я ничего не усредняю, потому что происходит сильная задержка из-за этого усреднения...

дак и в 6 строке ничего не усредняется.  Переменная units делится на 10, результат отбрасывается. 

просто рядом стоит

усреднение вот: 

  for(int i = 0;i < 10; i ++) units =+ scale.get_units();  
  units /= 10; 

можно убрать

исправлю, кста

  for(int i = 0;i < 10; i ++) units += scale.get_units();
  units /= 10.0;