Relay gets stuck and RF issue

[PerryBebbington]

I suggest you remove the radios for now and concentrate on getting serial communications working without them, then when you are happy with that add the radios in. At that point you will know the communications work OK without the radios, so any problems will be radio related, not related to anything else.

Start with this:

Serial Input Basics - updated

Updated Version Please note that this is a revised version of the advice in this earlier Thread which has become very long. As far as possible I have kept the code examples identical or simplifed them slightly. It should not be necessary to refer to the older Thread, but feel free to do so. Contents The following sections are in this Tutorial Introduction Serial data is slow by Arduino standards Example 1 - Receiving single characters Why code is organized into functions Exampl…

You could also do with knowing how to read buttons properly, Here is my tutorial:

Buttons and other electro-mechanical inputs (introduction)

Part 2 Part 1 shows how to read a single button. This part explains the code and circuit. Debouncing You have probably heard of debouncing and are perhaps wondering what it is and why it is needed. The oscilloscope trace below was taken from a single button wired as in part 1. The trace shows high and low transitions - some lasting over 4ms - before the signal settles at low. This is caused by mechanical bouncing of the button's contacts. We can use software to remove these unwanted transiti…

There's also the state change example in the IDE file / examples / digital / state change detection
Reading buttons correctly is a common topic here, so some reading of the forum will also help.

You need to get rid of the delay and learn how to code without them. Delay might be a quick fix at the moment, but the more you use it the more it will cause you problems.

Using millis() for timing. A beginners guide

Part 1 It is not usually long before new Arduino users discover that although the delay() function is easy to use it has side effects, the main one of which is that its stops all activity on the Arduino until the delay is finished (not quite true, I know, but that is usually how the problem presents itself). When this occurs the new user is usually directed to the BlinkWithoutDelay example sketch (BWoD) and/or the excellent thread entitled Several things at the same time. Several things at the…

What is this for:

pinMode(Button3, INPUT_PULLUP); // DS18B20

Why are you reading a DS18B20? How has that got anything to do with what you are doing? A DS18B20 is a 1 wire device and isn't read like that anyway.