neta540 / arduinothreadedgsm Goto Github PK

Hi there, thanks for making this library. Could you add an sms send example. You mention that the string to pass to the method should be a PDU but it's not particularly clear what format this should take without an example.

I have to ask you a favor. I want to “hide” Your class ThreadedGSM in my own class. I made a try, but not everything is working. It’s look like:

gms_c.h

#include "ThreadedGSM.h" // Author: Neta Yahav
#include "PduEncoder.h" // Author: Ludovic Laurent

class gsm_c {
private:
//ThreadedGSM *SIM800L;
void signal(ThreadedGSM& modem, ThreadedGSM::SignalLevel& Signal);

public:
    // ----- Constructor -----          
    //gsm_c();
    gsm_c(HardwareSerial& port);        
    void init();
    void sendAlarmSMS();
    void gsm_loop();        

};

gms_c.cpp

#include "gsm_c.h"

//gsm_c::gsm_c() {
// // nothing
// init();
//}

gsm_c::gsm_c(HardwareSerial& port) {
init();
}

void gsm_c::init() {
Serial1.begin(115200);
//SIM800L->begin();
SIM800L.begin();

//SIM800L->setInterval(ThreadedGSM::INTERVAL_SIGNAL, 60000);
SIM800L.setInterval(ThreadedGSM::INTERVAL_SIGNAL, 60000);

SIM800L.setHandlers({
.signal = signal,
.clock = NULL,
.incoming = NULL,
.ready = NULL,
.outgoing = NULL,
.power = NULL
});     

}

void gsm_c::signal(ThreadedGSM& modem, ThreadedGSM::SignalLevel& Signal) {
Serial.print("Modem signal: Dbm:");
Serial.print(Signal.Dbm);
Serial.print(" value: ");
Serial.println(Signal.Value);
}

void gsm_c::sendAlarmSMS() {
PduEncoder PDU;
String sms, encodedPDU;

PDU.setSMSCnumber("+XXXXXXXXXXX");
PDU.setSender("+XXXXXXXXXXX");

sms  = "Piec - ALARM - Temp: ";
sms += currentTemp;
sms += "\rStan pompy: ";

switch(stanPompy) {
    case 0: sms +="OFF"; break;
    case 1: sms +="ON"; break;
    default: sms +="???"; break;
}

sms += "\rTryb pracy pompy: ";
switch(trybPompy) {
    case 0: sms += "AUTO"; break;
    case 1: sms += "ON"; break;
    case 2: sms += "OFF"; break;
    default: sms += "???"; break;
}

PDU.setText(sms);
encodedPDU = PDU.encode();

//SIM800L->sendSMS(encodedPDU);
SIM800L.sendSMS(encodedPDU);

}

void gsm_c::gsm_loop() {
//SIM800L->loop();
SIM800L.loop();
}

The problem is with the SIM800L.setHandlers function. When I put there

SIM800L.setHandlers({
.signal = NULL,
.clock = NULL,
.incoming = NULL,
.ready = NULL,
.outgoing = NULL,
.power = NULL
});     

Sending SMS and loop() is working, but I need signal. If I change .signal = NULL to .signal = signal I’m getting this error:

src\gsm_c.cpp:50:3: error: no matching function for call to 'ThreadedGSM::setHandlers()'
In file included from include/gsm_c.h:10:0,
from src\gsm_c.cpp:1:
src/ThreadedGSM.h:186:7: note: candidate: void ThreadedGSM::setHandlers(ThreadedGSM::conf)
void setHandlers(conf config)
^
src/ThreadedGSM.h:186:7: note: no known conversion for argument 1 from '' to 'ThreadedGSM::conf'

How should I pass config to ThreadedGSM object in this situation? Could You help mi with that please? My knowledge of C++ is rather at basic level. :(