Nextion button - arduino problem.

anon90604980 · December 7, 2017, 5:44pm

Hello,

I'v had a problem all day whit a button in my Nextion 2.4 inch. I only want to increment a variable in case "hour_" when the "but" button is pushed, but on the serial monitor hasn't showed the incremented number.

Could you help me about this problem?

I use:

The latest version of Arduino 1.8.
The latest version of nextion editor V0.52
The ITEADLIB_Arduino_Nextion-master's library

And code that I've used is:

#include <Nextion.h>
#include <Time.h>
#include <TimeLib.h>
#include <SoftwareSerial.h>
int hour_ = 16;
int min_ = 0;

SoftwareSerial nex_com(10, 11); // RX, TX

NexButton b0 = NexButton(3, 3, "b0");
NexNumber n2 = NexNumber(3, 2, "n2");
NexButton b1 = NexButton(0, 5, "but");

NexTouch *nex_listen_list[] =
{
&b0, // Button added
&b1,
NULL // String terminated
} ;

void setup() {
nex_com.begin(9600);
Serial.begin(9600);
delay(500);

nex_com.print("baud=115200");
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
//
nex_com.end();
nex_com.begin(115200);
setTime(hour_,min_,0,12,4,17);
b0.attachPush(b0PushCallback);
b1.attachPush(b1PushCallback);
}

void loop()
{
delay(10);
nexLoop(nex_listen_list);
Serial.println(hour_);
}
void b0PushCallback(void *ptr)
{

hour_= hour_ - 1;

nex_com.print("n0.val=");
nex_com.print(hour_);
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
}
void b1PushCallback(void *ptr)
{
hour_ = hour_ + 1;

nex_com.print("n2.val=");
nex_com.print(hour_);
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);

} // End of press event

Whandall · December 7, 2017, 5:49pm

SoftwareSerial will not work with 115200 baud.

anon90604980 · December 7, 2017, 5:54pm

Thank's for that.

I changed it to 9600 baud, but a problem hadn't solved.

There is something else that stop me.

Whandall · December 7, 2017, 6:01pm

Barca1991:
I only want to increment a variable in case "hour_" when the "but" button is pushed, but on the serial monitor hasn't showed the incremented number.

So why don't you increment/decrement hour_, but an undefined variable max_ ?

anon90604980 · December 7, 2017, 6:06pm

My mistake, but the problem isn't that

Whandall · December 7, 2017, 6:09pm

So don't you think that you should post the new code?

And please use code tags (the </> button).

This looks strange too

NexButton b0 = NexButton(3, 3, "b0");
NexButton b1 = NexButton(0, 5, "but");

are you shure the page ids and component ids are correct?

Don't change already posted code, that renders the answers meaningless.

Post the code again if you change it.

anon90604980 · December 7, 2017, 6:21pm

I corrected my mistake in the code above. I've tried whit this code:

#include <Nextion.h>

#include <Time.h>
#include <TimeLib.h>
#include <SoftwareSerial.h>

int hour_ = 16;
int min_ = 0;

SoftwareSerial nex_com(10, 11); // RX, TX

NexButton b0 = NexButton(3, 3, "b0"); // Button added
//NexButton b1 = NexButton(3, 4, "b1"); // Button added
NexNumber n2 = NexNumber(3, 2, "n2");
NexButton b1 = NexButton(0, 5, "but");

NexTouch *nex_listen_list[] =
{
&b0,
&b1,
NULL
} ;

void setup() {
nex_com.begin(9600);
Serial.begin(9600);
delay(500);

nex_com.print("baud=9600");

nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);

setTime(hour_,min_,0,12,4,17);
b0.attachPush(b0PushCallback); // Button press
b1.attachPush(b1PushCallback); // Button press

}

void loop()
{
delay(10);
//nexLoop(nex_listen_list);
nex_com.print("n0.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
delay(2);
nex_com.print("n1.val=");
nex_com.print(minute());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
nexLoop(nex_listen_list);

Serial.println(hour_);

}

void b0PushCallback(void *ptr)
{

max_ = hour_ - 1;

nex_com.print("n0.val=");
nex_com.print(hour_);
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
}
void b1PushCallback(void *ptr)
{
hour_ = (hour_ + 1);

nex_com.print("n2.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
}

And when I use this functions

nex_com.print("n2.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);

I successfully send the date and visualize the hours and minutes.

But when I push a button on the screen the hour_ variable isn't increment.????

Whandall · December 7, 2017, 6:23pm

//nexLoop(nex_listen_list);

?

No listen - no action.

And why don't you use code tags?

anon90604980 · December 7, 2017, 6:27pm

NexTouch *nex_listen_list[] =
{
&b0,
&b1,
NULL
} ;

It's the touch event.

Is it correct that?

Whandall · December 7, 2017, 6:52pm

Which touch event?

You don't even listen to SoftwareSerial (you commented that statement, see #7).

anon90604980 · December 7, 2017, 6:54pm

void loop()
{
delay(10);
//nexLoop(nex_listen_list);
nex_com.print("n0.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
delay(2);
nex_com.print("n1.val=");
nex_com.print(minute());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
nexLoop(nex_listen_list); --------- I use this here.

Serial.println(hour_);

}

What happened. I'm so desperate. ;(;(

Whandall · December 7, 2017, 7:01pm

You don't answer my questions, you ignore the requested usage of code tags.

Remove all commented statements that you don't wish to use in the code later or for debug,
they only distract and confuse the reader (at least me).

Don't use delay for timing, you should try to keep loop running as fast as possible.

anon90604980 · December 7, 2017, 7:11pm

Ok Whandall, sorry.

#include <Nextion.h>
#include <Time.h>
#include <TimeLib.h>
#include <SoftwareSerial.h>
int hour_ = 16;
int min_ = 0;
SoftwareSerial nex_com(10, 11); // RX, TX

NexButton b1 = NexButton(0, 5, "but");

NexTouch *nex_listen_list[] =
{
&b1,
NULL
} ;

void setup() {
nex_com.begin(9600);
Serial.begin(9600);
delay(500);
nex_com.print("baud=9600");
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
setTime(hour_,min_,0,12,4,17);
b1.attachPush(b1PushCallback);
}

void loop()
{
nex_com.print("n0.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.print("n1.val=");
nex_com.print(minute());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
nexLoop(nex_listen_list);
Serial.println(hour_);
}

void b1PushCallback(void *ptr)
{
hour_ = (hour_ + 1);
Serial.println(hour_);
}

It's the compact code, Do you see anything wrong? Because I'm not able to do it so far. ;(

Whandall · December 7, 2017, 7:21pm

You did not use code tags, again.

I can not see your HMI, you don't answer questions regarding it.
Are you shure that the button is on page 0 with component id 5?

anon90604980 · December 7, 2017, 7:29pm

Hi, again.

You can see attachments, especially the pic.

Whandall · December 7, 2017, 7:44pm

You are writing the values of hour_ and min_ much too often,
they should only be written when they change.

If you have fixed that (that keeps you in a blocked state on SoftwareSerial),
print hour_ only when it has changed (or once a second) so you will not block on Serial.

After these changes press Ctrl-T in the IDE to get a better formatting and post the resulting
compiling code.

If you will not use code tags (the button </>), I will stop wasting time here.

anon90604980 · December 7, 2017, 8:06pm

Sorry, but I don't understand , when I should use "code tags (the button </>)", Is it here on the forum?

#include <Nextion.h>
#include <Time.h>
#include <TimeLib.h>
#include <SoftwareSerial.h>
int hour_ = 16;
int min_ = 0;
SoftwareSerial nex_com(10, 11);

NexButton b1 = NexButton(0, 5, "but");

NexTouch *nex_listen_list[] =
{
&b1,
NULL
} ;

void setup() {
nex_com.begin(9600);
Serial.begin(9600);
delay(500);
nex_com.print("baud=9600");
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
setTime(hour_, min_, 0, 12, 4, 17);
b1.attachPush(b1PushCallback);
}

void loop()
{
nex_com.print("n0.val=");
nex_com.print(hour());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.print("n1.val=");
nex_com.print(minute());
nex_com.write(0xff);
nex_com.write(0xff);
nex_com.write(0xff);

nexLoop(nex_listen_list);
}

void b1PushCallback(void *ptr)
{
hour_ = hour_ + 1;
Serial.println(hour_);
}

It is, but nothing............ ;( ;(

sterretje · December 7, 2017, 8:15pm

Yes, that </> button is here on the forum. You click the reply button and it opens a new page to compose a reply; you will see it there.

Not familiar with nexion but ain't you supposed to have a callback attached to the but button?

Whandall · December 7, 2017, 8:15pm

Barca1991:
Sorry, but I don't understand , when I should use "code tags (the button </>)", Is it here on the forum?

I you had bothered reading

Read this before posting a programming question ... #6

you knew.

Whandall:
You are writing the values of hour_ and min_ much too often,
they should only be written when they change.
..
After these changes press Ctrl-T in the IDE to get a better formatting and post the resulting
compiling code.

Whandall · December 7, 2017, 8:19pm

sterretje:
Not familiar with nexion but ain't you supposed to have a callback attached to the but button?

It is in there.

 b1.attachPush(b1PushCallback);

I don't use SoftwareSerial with the Nextion, but I think one will run into problems
when blocking on the transmit and hoping to receive something at the same time.

Maybe a different SoftwareSerial works better, like AltSoftSerial or the like.

A better communication strategy (sending only changes) would help anyway.

Topic		Replies	Views
Hold Button down to increase number Programming	18	4208	May 5, 2021
Nextion daten zu arduino var schicken Deutsch	3	711	May 6, 2021
Arduino won't detect release of button Displays	4	983	May 6, 2021
I can't manage to send data (a number) from Arduino to Nextion Displays	9	503	December 25, 2023
Nextion button Programming	2	1121	May 5, 2021

Nextion button - arduino problem.

Related topics