The sketch with the new integrer stepsToGo:
// Example 3 - Receive with start- and end-markers
const byte numChars = 32;
char receivedChars[numChars];
int stepsToGo =0 ;
boolean newData = false;
void setup() {
Serial.begin(9600);
Serial.println("<Arduino is ready>");
}
void loop() {
recvWithStartEndMarkers();
showNewData();
}
void recvWithStartEndMarkers() {
static boolean recvInProgress = false;
static byte ndx = 0;
char startMarker = '<';
char endMarker = '>';
char rc;
while (Serial.available() > 0 && newData == false ) {
rc = Serial.read();
if (recvInProgress == true) {
if (rc != endMarker) {
receivedChars[ndx] = rc;
ndx++;
if (ndx >= numChars) {
ndx = numChars - 1;
}
}
else {
receivedChars[ndx] = '\0'; // terminate the string
recvInProgress = false;
ndx = 0;
newData = true;
}
}
else if (rc == startMarker) {
recvInProgress = true;
}
}
}
void showNewData() {
if (newData == true) {
Serial.print("This just in ... ");
Serial.println(receivedChars);
Serial.println(stepsToGo);
newData = false;
}
delay(1000);
}
Robin2
September 10, 2017, 3:30pm
22
Brinarpi:
That's because you have not added any code to take the data in receivedChars[] and convert it to an integer and save it into your variable stepsToGo.
I don't understand how you thought that the program could work without doing that?
Have a look at what I have added in this version
// Example 3 - Receive with start- and end-markers
const byte numChars = 32;
char receivedChars[numChars];
int stepsToGo =0 ;
boolean newData = false;
void setup() {
Serial.begin(9600);
Serial.println("<Arduino is ready>");
}
void loop() {
recvWithStartEndMarkers();
parseData(); // new
showNewData();
}
void recvWithStartEndMarkers() {
static boolean recvInProgress = false;
static byte ndx = 0;
char startMarker = '<';
char endMarker = '>';
char rc;
while (Serial.available() > 0 && newData == false ) {
rc = Serial.read();
if (recvInProgress == true) {
if (rc != endMarker) {
receivedChars[ndx] = rc;
ndx++;
if (ndx >= numChars) {
ndx = numChars - 1;
}
}
else {
receivedChars[ndx] = '\0'; // terminate the string
recvInProgress = false;
ndx = 0;
newData = true;
}
}
else if (rc == startMarker) {
recvInProgress = true;
}
}
}
void showNewData() {
if (newData == true) {
Serial.print("This just in ... ");
Serial.println(receivedChars);
Serial.println(stepsToGo);
newData = false;
}
// delay(1000); // this should not be here
}
// new
void parseData() {
stepsToGo = atoi(receivedChars);
}
Also NOTE that I told you to put the delay(1000); in the other program - the sending program.
...R
Ohh dear Robin. The sender is a smaller problem . I have already added the delay as you ask.
There is only 1 segment left of the project. To implement in the sender, the push buttons, and the LCD16/2 and its finish. Try to do it alone......
Anyway thanks a lot for everything what you help to me. You are really a MAN (with capital characters).
Arpad from Hungary
Robin2
September 10, 2017, 6:47pm
24
Brinarpi:
You are 13 years younger than me
...R
Hello again
I was analize your way of the program build. First in the loop you tell what kind of jobs are like adress.
void loop() {
recvWithStartEndMarkers();
parseData();
showNewData();
And after explain all of them in separated void like:
void recvWithStartEndMarkers()
{
}
Is that workable if I put in the loop the stepping program like adress(as others are) and open a void on the end like last void?
void loop() {
recvWithStartEndMarkers();
parseData(); // new
showNewData();
stepping();
void stepping()
{
// put an accel stepper program here
}
Is that way is correct?
A
Robin2
September 11, 2017, 12:43pm
26
Brinarpi:
Yes. Organizing programs into separate single purpose functions makes them much easier to develop, test and debug.
Of course you also need to get the details right
Maybe have a look at Planning and Implementing a Program
...R
