http request and isSMSunread() simultaneously not working. getting struct.

[Narayana-Pathipati]

I am using http get request with tcp and isSMSunread() both are in loop function.
But getting struct, when both are using at a time.
I am expecting solution for this probject.
thank you.

[hoerup]

Please provide more context on how you use this library, together with some code snippets from the failing code

[Narayana-Pathipati]

#include <DFRobot_SIM808.h>
#include <SoftwareSerial.h>
#include <Wire.h>
//#define PHONE_NUMBER "01743175953"
#define MESSAGE_LENGTH 160
char message[MESSAGE_LENGTH];
int messageIndex = 0;
//char MESSAGE[300];
float la;
float lo;
//float speedd;
float wspeed;
String data;
#define MESSAGE"data"
char gprsBuffer[256];
char *s = NULL;
char phone[16];
char datetime[24];
#define PIN_TX 10
#define PIN_RX 11
SoftwareSerial mySerial(PIN_TX,PIN_RX);
DFRobot_SIM808 sim808(&mySerial);//Connect RX,TX,PWR,
char b[100];
void getGPS();
void sendhttp();
String rmsg;
String asish = String("gps");

char buffer[64];
float lati, longi;

void setup()
{
mySerial.begin(9600);
Serial.begin(9600);
//******** Initialize sim808 module *************
while(!sim808.init())
{
Serial.print("Sim808 init error\r\n");
delay(1000);
}
delay(3000);
Serial.println("SIM Init success");
Serial.println("Init Success, please send SMS message to me!");

mySerial.println("AT+CREG?");
delay(100);

mySerial.println("AT+CSQ");
delay(100);

mySerial.println("AT+CGATT?");
delay(100);
}

void sendhttp() {

if(!sim808.connect(TCP,"us-central1-ops-admin-web.cloudfunctions.net", 80)) {
Serial.println("Connect error");
}else{
    Serial.println("Connect us-central1-ops-admin-web.cloudfunctions.net success");
}

// String lnk;
// lnk = "GET /adiramef/api/generic.php?add_to_db&latitude=" + String(lati) + "&longitude=" + String(longi) + "&device_name=Frigorifero" + " HTTP/1.1\r\n\r\n " + " Host: " + "www.ikomoto.com" + "\r\n" + "Connection: close\r\n\r\n";;

// char http_cmd[(lnk.length())+1];
// lnk.toCharArray(http_cmd,(lnk.length()+1));

char http_cmd[] = "GET /adiramef/api/generic.php?add_to_db&latitude=" + String(lati) + "&longitude=" + String(longi) + "&device_name=Frigorifero" + " HTTP/1.1\r\n\r\n " + " Host: " + "www.ikomoto.com" + "\r\n" + "Connection: close\r\n\r\n";;

//*********** Send a GET request *****************

Serial.println("waiting to fetch...");
//Serial.println( sim808.send(http_cmd, sizeof(http_cmd)-1));
sim808.send(http_cmd, sizeof(http_cmd)-1);
while (true) {
int ret = sim808.recv(buffer, sizeof(buffer)-1);
if (ret <= 0){
Serial.println("fetch over...");
break;
}
buffer[ret] = '\0';
Serial.print("Recv: ");
Serial.print(ret);
Serial.print(" bytes: ");
Serial.println(buffer);

    delay(4000);

  break;

}

delay(3000);

}

void loop()
{
getGPS();
delay(200);
readSMS();
delay(200);
sendhttp();
delay(200);
}
void readSMS()
{
//*********** Detecting unread SMS ************************
messageIndex = sim808.isSMSunread();
Serial.print("messageIndex: ");
Serial.println(messageIndex);
//*********** At least, there is one UNREAD SMS ***********
if (messageIndex > 0) {
sim808.readSMS(messageIndex, message, MESSAGE_LENGTH, phone, datetime);
delay(200);
rmsg=String(message);
if(asish == message)
{
sim808.sendSMS(phone,b);
}
//*In order not to full SIM Memory, is better to delete it
sim808.deleteSMS(messageIndex);
Serial.print("From number: ");
Serial.println(phone);
Serial.print("Datetime: ");
Serial.println(datetime);
Serial.print("Recieved Message: ");
Serial.println(message);
}
}
void getGPS()
{
while(!sim808.attachGPS())
{
Serial.println("Open the GPS power failure");
delay(1000);
}
delay(1000);
Serial.println("Open the GPS power success");
while(!sim808.getGPS())
{
}
Serial.print(sim808.GPSdata.year);
Serial.print("/");
Serial.print(sim808.GPSdata.month);
Serial.print("/");
Serial.print(sim808.GPSdata.day);
Serial.print(" ");
Serial.print(sim808.GPSdata.hour);
Serial.print(":");
Serial.print(sim808.GPSdata.minute);
Serial.print(":");
Serial.print(sim808.GPSdata.second);
Serial.print(":");
Serial.println(sim808.GPSdata.centisecond);
Serial.print("latitude :");
Serial.println(sim808.GPSdata.lat);
Serial.print("longitude :");
Serial.println(sim808.GPSdata.lon);
Serial.print("speed_kph :");
Serial.println(sim808.GPSdata.speed_kph);
Serial.print("heading :");
Serial.println(sim808.GPSdata.heading);
Serial.println();
// speedd = sim808.GPSdata.speed_kph;
la = sim808.GPSdata.lat;
lo = sim808.GPSdata.lon;
wspeed = sim808.GPSdata.speed_kph;
// data = ("latitude: " + String(la) + "*C " + " longitude: " + String(lo) + "%" );
data = ("latitude: " + String(la) + " longitude: " + String(lo) + "speed_kph :" + String(wspeed) );
data.toCharArray(b,100);
Serial.println(b);
Serial.println(b);
Serial.println(data);
delay(400);
}

Hi,
I am using above code save gps data webserver using http get request. this is working fine when I use alone. But when I include smsRead(), it was getting hanged and not working properly. If i use separately working fine. Please give me solution.
With smsRead(), I am getting gps route name from text message and update in http request.

[hoerup]

I haven't tried the code - but by a quick look at it, could your problem stem from the fact that you don't close the TCP session after use ?

You need to take a look at what happens if the connect() fails - currently it will try to continue with sending the http request

2 minor nits

1. You have an empty line between request and first header - thats not RFC compliant (and remove the trailing space)
   "HTTP/1.1\r\n\r\n "
2. " Host: " remove space from start

[Narayana-Pathipati]

I am facing issue sim808.readSMS() function inside loop. If I use this function, http request error and sometime hangs.