I have code for starting motor, but delay not work, any one help me ?
#include <gprs.h>
#include <softwareserial.h>
#define TIMEOUT 5000
#define LED_PIN 13
#define ON LOW
#define OFF HIGH
const int Relay1 = 2;
const int Relay2 = 3;
const int Relay3 = 4;
const int Relay4 = 5;
int StatRelay1;
int StatRelay2;
int StatRelay3;
int StatRelay4;
GPRS gprs;
void setup() {
pinMode (Relay1 , OUTPUT); digitalWrite (Relay1, HIGH);
pinMode (Relay2 , OUTPUT); digitalWrite (Relay2, HIGH);
pinMode (Relay3 , OUTPUT); digitalWrite (Relay3, HIGH);
pinMode (Relay4 , OUTPUT); digitalWrite (Relay4, HIGH);
Serial.begin(9600);
while(!Serial);
Serial.println("Starting SIM800 Auto Read SMS");
gprs.preInit();
delay(1000);
while(0 != gprs.init()) {
delay(1000);
Serial.print("init error\r\n");
}
//Set SMS mode to ASCII
if(0 != gprs.sendCmdAndWaitForResp("AT+CMGF=1\r\n", "OK", TIMEOUT)) {
ERROR("ERROR:CNMI");
return;
}
//Start listening to New SMS Message Indications
if(0 != gprs.sendCmdAndWaitForResp("AT+CNMI=1,2,0,0,0\r\n", "OK", TIMEOUT)) {
ERROR("ERROR:CNMI");
return;
}
Serial.println("Init success");
}
//Variable to hold last line of serial output from SIM800
char currentLine[500] = "";
int currentLineIndex = 0;
//Boolean to be set to true if message notificaion was found and next
//line of serial output is the actual SMS message content
bool nextLineIsMessage = false;
void loop() {
//Write current status to LED pin
digitalWrite(Relay1, StatRelay1);
digitalWrite(Relay2, StatRelay2);
digitalWrite(Relay3, StatRelay3);
digitalWrite(Relay4, StatRelay4);
//If there is serial output from SIM800
if(gprs.serialSIM800.available()){
char lastCharRead = gprs.serialSIM800.read();
//Read each character from serial output until \r or \n is reached (which denotes end of line)
if(lastCharRead == '\r' || lastCharRead == '\n'){
String lastLine = String(currentLine);
//If last line read +CMT, New SMS Message Indications was received.
//Hence, next line is the message content.
if(lastLine.startsWith("+CMT:")){
Serial.println(lastLine);
nextLineIsMessage = true;
} else if (lastLine.length() > 0) {
if(nextLineIsMessage) {
Serial.println(lastLine);
// ########## MEMBACA KONTEN SMS DAN MENCARI+MENGARTIKAN KONTEN SMS KE PROGRAM #########
//Kendali Relay
if(lastLine.indexOf("Relay1 ON") >= 0){
StatRelay1 = ON;
Serial.println("Relay1 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay1 DINYALAKAN");
delay(3000);
StatRelay1 = OFF;}
else if(lastLine.indexOf("Relay1 OFF") >= 0) {
StatRelay1 = OFF;
Serial.println("Relay1 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay1 DIMATIKAN");}
if(lastLine.indexOf("Relay2 ON") >= 0){
StatRelay2 = ON;
Serial.println("Relay2 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay2 DINYALAKAN");}
else if(lastLine.indexOf("Relay2 OFF") >= 0) {
StatRelay2 = OFF;
Serial.println("Relay2 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay2 DIMATIKAN");}
if(lastLine.indexOf("Relay3 ON") >= 0){
StatRelay3 = ON;
Serial.println("Relay3 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay3 DINYALAKAN");}
else if(lastLine.indexOf("Relay3 OFF") >= 0) {
StatRelay3 = OFF;
Serial.println("Relay3 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay3 DIMATIKAN");}
if(lastLine.indexOf("Relay4 ON") >= 0){
StatRelay4 = ON;
Serial.println("Relay4 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay4 DINYALAKAN");}
else if(lastLine.indexOf("Relay4 OFF") >= 0) {
StatRelay4 = OFF;
Serial.println("Relay4 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx","Relay4 DIMATIKAN");}
nextLineIsMessage = false;
}
}
//Clear char array for next line of read
for( int i = 0; i < sizeof(currentLine); ++i ) {
currentLine[i] = (char)0;
}
currentLineIndex = 0;
} else {
currentLine[currentLineIndex++] = lastCharRead;
}
}
}
Robin2
September 10, 2017, 6:37pm
2
beryindo:
I don't understand what you mean. What does the program actually do, and what do you want it to do that is different?
And your code is very difficult to read because of the way it is laid out. Please use the AutoFormat tool to make it easier to read and post the reformatted version in your next Reply.
...R
vaj4088
September 10, 2017, 7:25pm
3
"not work" is not useful information. Also, when you are tempted to suffix variable names with numbers, it is time to learn about arrays.
this my code
#include <gprs.h>
#include <softwareserial.h>
#define TIMEOUT 5000
#define LED_PIN 13
#define ON LOW
#define OFF HIGH
const int Relay1 = 2;
const int Relay2 = 3;
const int Relay3 = 4;
const int Relay4 = 5;
int StatRelay1;
int StatRelay2;
int StatRelay3;
int StatRelay4;
GPRS gprs;
void setup() {
pinMode (Relay1 , OUTPUT); digitalWrite (Relay1, HIGH);
pinMode (Relay2 , OUTPUT); digitalWrite (Relay2, HIGH);
pinMode (Relay3 , OUTPUT); digitalWrite (Relay3, HIGH);
pinMode (Relay4 , OUTPUT); digitalWrite (Relay4, HIGH);
Serial.begin(9600);
while (!Serial);
Serial.println("Starting SIM800 Auto Read SMS");
gprs.preInit();
delay(1000);
while (0 != gprs.init()) {
delay(1000);
Serial.print("init error\r\n");
}
//Set SMS mode to ASCII
if (0 != gprs.sendCmdAndWaitForResp("AT+CMGF=1\r\n", "OK", TIMEOUT)) {
ERROR("ERROR:CNMI");
return;
}
//Start listening to New SMS Message Indications
if (0 != gprs.sendCmdAndWaitForResp("AT+CNMI=1,2,0,0,0\r\n", "OK", TIMEOUT)) {
ERROR("ERROR:CNMI");
return;
}
Serial.println("Init success");
}
//Variable to hold last line of serial output from SIM800
char currentLine[500] = "";
int currentLineIndex = 0;
//Boolean to be set to true if message notificaion was found and next
//line of serial output is the actual SMS message content
bool nextLineIsMessage = false;
void loop() {
//Write current status to LED pin
digitalWrite(Relay1, StatRelay1);
digitalWrite(Relay2, StatRelay2);
digitalWrite(Relay3, StatRelay3);
digitalWrite(Relay4, StatRelay4);
//If there is serial output from SIM800
if (gprs.serialSIM800.available()) {
char lastCharRead = gprs.serialSIM800.read();
//Read each character from serial output until \r or \n is reached (which denotes end of line)
if (lastCharRead == '\r' || lastCharRead == '\n') {
String lastLine = String(currentLine);
//If last line read +CMT, New SMS Message Indications was received.
//Hence, next line is the message content.
if (lastLine.startsWith("+CMT:")) {
Serial.println(lastLine);
nextLineIsMessage = true;
} else if (lastLine.length() > 0) {
if (nextLineIsMessage) {
Serial.println(lastLine);
// ########## MEMBACA KONTEN SMS DAN MENCARI+MENGARTIKAN KONTEN SMS KE PROGRAM #########
//Kendali Relay
if (lastLine.indexOf("Relay1 ON") >= 0) {
StatRelay1 = ON;
Serial.println("Relay1 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay1 DINYALAKAN");
delay(3000);
StatRelay1 = OFF;
}
else if (lastLine.indexOf("Relay1 OFF") >= 0) {
StatRelay1 = OFF;
Serial.println("Relay1 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay1 DIMATIKAN");
}
if (lastLine.indexOf("Relay2 ON") >= 0) {
StatRelay2 = ON;
Serial.println("Relay2 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay2 DINYALAKAN");
}
else if (lastLine.indexOf("Relay2 OFF") >= 0) {
StatRelay2 = OFF;
Serial.println("Relay2 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay2 DIMATIKAN");
}
if (lastLine.indexOf("Relay3 ON") >= 0) {
StatRelay3 = ON;
Serial.println("Relay3 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay3 DINYALAKAN");
}
else if (lastLine.indexOf("Relay3 OFF") >= 0) {
StatRelay3 = OFF;
Serial.println("Relay3 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay3 DIMATIKAN");
}
if (lastLine.indexOf("Relay4 ON") >= 0) {
StatRelay4 = ON;
Serial.println("Relay4 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay4 DINYALAKAN");
}
else if (lastLine.indexOf("Relay4 OFF") >= 0) {
StatRelay4 = OFF;
Serial.println("Relay4 DIMATIKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay4 DIMATIKAN");
}
nextLineIsMessage = false;
}
}
//Clear char array for next line of read
for ( int i = 0; i < sizeof(currentLine); ++i ) {
currentLine[i] = (char)0;
}
currentLineIndex = 0;
} else {
currentLine[currentLineIndex++] = lastCharRead;
}
}
}
I have 4 relay, but I need Relay 1 Start Engine motorcycle. and I use delay 3000 second to ON Relay 1 and then Relay 1 OFF.
I use that code, but I can ON relay 1. any one help me. thank you
system
September 11, 2017, 7:38am
5
char currentLine[500] = ""; :o
The code and description of the problem are both quite vague, but I think you want this:
//Kendali Relay
if (lastLine.indexOf("Relay1 ON") >= 0) {
StatRelay1 = ON;
digitalWrite(Relay1, StatRelay1);
Serial.println("Relay1 DINYALAKAN");
gprs.sendSMS ("08xxxxxxxxxx", "Relay1 DINYALAKAN");
delay(3000);
StatRelay1 = OFF;
digitalWrite(Relay1, StatRelay1);
}
Just changing StatRelay1 doesn't actually do anything. You have to use digitalWrite() as well.
emielsteerneman:
//Kendali Relay
if (lastLine.indexOf("Relay1 ON") >= 0) {
Just changing StatRelay1 doesn't actually do anything. You have to use digitalWrite() as well.
Not work bro. relay can't ON
system
September 11, 2017, 9:31am
9
beryindo:
Apparently relay can't ON.
AWOL:
Fix bro
MarkT
September 11, 2017, 10:19am
11
beryindo:
That's 25% of all the RAM in the system for an Uno.
is there any code or any other way?
beryindo:
Isn't this what I suggested?
For the RAM, just change [500] to something smaller, [200] or something, and see if it still works.