peninquen / Modbus-Energy-Monitor-Arduino

A class to collect data from a energy monitor that uses modbus protocol.
Apache License 2.0
46 stars 27 forks source link

Nombres de funciones, objetos y constantes. #5

Closed peninquen closed 8 years ago

peninquen commented 8 years ago

Con el objetivo de que sea 'Arduino-like' los nombres de las funciones en las librerías tienen que ser consistentes con las que existen en otras librerías de Arduino, devolver tipos similares. De esta forma un sketch puede ser casi identico y tomar datos por el bus serial, por ModbusRTU, por un pin de pulsos, etc. Ejemplo MBSerial.read() es equivalente a Serial.read().

Proponer nombres de variables y constantes más descriptivas, o que se ajusten a nombres en otras librerías Arduino.

peninquen commented 8 years ago

MBSerial.available(), debe devolver el numero de datos disponibles actualizados, similar a la libreria Serial. La función MBSerial.read() descuenta 1 el número de datos leidos. Problema: el programa debe asegurarse leer todos los sensores con read() para asegurar de poner a cero el buffer, o ponemos una función ¿reset? que se encarge de poner a cero el contador

peninquen commented 8 years ago

He decidido sacar TIMEOUTcomo variable y pasa a ser una constante definida en en archivo MdbusSensor.h, junto a WAITING_INTERVAL. Son parametros del equipo que el usuario bajo no va a cambiar.