Can not change a custom variable in ble_our_service_on_ble_evt()

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

Caner Celiktas — Mon, 24 Jun 2019 12:37:03 GMT

Okey, now it's worked. I declared "button_value" as an integer in main.c and i declared as extern in our_service.c. Now it's working fine. Thank you !

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

Einar Thorsrud — Mon, 24 Jun 2019 11:51:45 GMT

Hi,

Like awneil said, you cannot have the button_value as static. That will clearly not work since it will be two completely independent variables in both files, which just happens to have the same name. If you want the same variable in both files, then you need to declare it normally (without static!) in one .c file, and as extern in the other c file or header file.

If you get a build error when you fix this, then you need to resolve it. We cannot help without knowing more about the error, though. What is the full error message?

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

Caner Celiktas — Mon, 24 Jun 2019 11:40:10 GMT

well, i remove "static" and tried "extern". Nothing change.. If i only use "int button_value;" then i'm getting "Build Failed" error.

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

awneil — Mon, 24 Jun 2019 11:14:30 GMT

[quote userid="79370" url="~/f/nordic-q-a/48907/can-not-change-a-custom-variable-in-ble_our_service_on_ble_evt"]i declared the variable "static int button_value;" in our_service.h[/quote]

Declaring variables as 'static' in a header is not generally a good idea!

It means that each source file which #includes that header gets its own private copy!

This is standard 'C' - nothing to do with Nordic or BLE.

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

Caner Celiktas — Mon, 24 Jun 2019 10:28:16 GMT

Fixed.

RE: Can not change a custom variable in ble_our_service_on_ble_evt()

awneil — Mon, 24 Jun 2019 09:34:30 GMT

How to properly post source code (not as an image):