NRF24L01 Problem simplifying code

Question

I am using the NRF24 for receiver/transmitter on a link between a Uno and a Mega. the basis for the code is http://maniacbug.github.io/RF24 I have finally gotten the units working super. My project is to input a four digit code on the transmitter, send it to the receiver, which will echo back the data sent for accuracy check. Then the code will be displayed on a three inch 7 segment display. My problem is: The code from the above is impossible for me to modify without breaking it. It seems that most of the work is done in the library RF24.h. The code is great for debugging and was very helpful for getting the NRF24s working. However, All my efforts to simplify the code to insert the data and extract it results in compiler errors. Does anyone have any help or a source for a more easily modified code for the NRF24?

Thanks to a bunch of great resource and to the dedication to us who are trying to learn.

Bill

screamingtiger · Accepted Answer

Just delete it out of the libraries folder, and restart the IDE

Robert Peter Oakes · Answer

you may find the code here useful for reference, I used the NRF modules in last XMAS contest and provided all the code http://www.element14.com/community/groups/arduino/blog/2015/01/24/byob-party-15-news-year-resolution-in-coding-practices-infineon-uno-the-nrf24l01-coding-walk-through http://www.element14.com/community/groups/arduino/blog/2014/12/29/byob-party-7--mrs-yun-gets-chatty-with-ioteclipseorg-and-nrf24l01

balearicdynamics · Answer

Bill, please in future insert the code with the syntax highlighting so it is more readable and easier to manage (i.e. try to insert in some editor locally). Then also report the compilation errors (the full compilation output where the errors are explained - bad -) 
 bool ok = radio.write( &time, sizeof(unsigned long) ); // this is writing ok but if change anything to 
 // write my data ( 4bytes) it wont compile...

IMO the definition while reading is wrong. Define it then make the operation. It is inside a loop and the defined variables exists just from where they are defined to the more internal parts of the code. If you have not problems of memory, define the boolean ok with another non-misunderstandable name (e.g. my_Ok) outside the loop as global then initialise it in the setup with the correct value you expect initially (true or false) then use it with the read operation in the loop as you are doing. 
 while ( ! radio.available() && ! timeout ) // I think this says " while the radio is available, and not 
 // timeout,

You check the negate value for both, so it says "while the radio is NOT available AND NOT timeout" that seems generating a never available condition. 
 radio.read( &got_time, sizeof(unsigned long) ); // If I try to use my data here, instead 
 // of " &got_time, sizeof(unsigned long) 
 // it wont complile.

Please show how you are using your data here, to have an idea why it does not compile. 'toupper()' usage To be sure that all data in both sides are always checked correctly ignoring the uppercase/lowercase case, despite what is the input string (e.g. all lower case, all uppercase or mixed) the call to toupper forces all the characters to uppercase so the conditions are always checked with the right characters. Enrico

balearicdynamics · Answer

Hi Bill, The following image shows how to catch the syntax highlighting in the editor Enrico

NRF24L01 Problem simplifying code

Top Replies