Power LED's using RF transmitter and receiver

Using Arduino Programming Questions
1

I'm working on a project that I have two sensors, two Arduinos (a YUN and an UNO), two LED's, and an RF transmitter/receiver

It's a pretty simple procedure: Activate a force sensor, send it via RF to the receiver and light up the corresponding LED. So my design looks like this -

Sensor 1 and Sensor 2 ---> Arduino YUN ---> RF Transmitter --> Wireless

---> RF Receiver ---> Arduino UNO ---> LED 1 and LED 2

So if Sensor 1 is activated LED 1 lights up and if Sensor 2 is activated LED 2 lights up.

So far I am able to send the signal and distinguish between each sensor by viewing the serial monitor. If I hit sensor 1, the Uno receives sensor 1's signal. Same with sensor 2. I'm just stuck on how to get the LED's to activate. I tried using buf =='1' code so whenever I send "1", it would read the buf for 1 then digitalWrite the led high. But that didn't work. Any advice would be much appreciated. Here's my code if you would like to look at it.
Transmitter:
```
*    #include <VirtualWire.h>
    int pin0 = 0;
    int pin1 = 1;

void setup()
    {
        vw_set_ptt_inverted(true);  // Required by the RF module
        vw_setup(2000);            // bps connection speed
        Serial.begin (9600);
        vw_set_tx_pin(3);        // Arduino pin to connect the transmitter data pin
    }

void loop()
    {
      int force = analogRead(pin0); //  analog reading from the Force sense resistor
      int force1 = analogRead(pin1); //  analog reading from the Force1 sense resistor

const char *msg = "1";
      const char *ms = "2";

if (force > 100) {
      vw_send((uint8_t *)msg, strlen(msg));

vw_wait_tx();        // We wait to finish sending the message
      delay(200);        // We wait to send the message again               
    }

if (force1 > 100) {
      vw_send((uint8_t *)ms, strlen(ms));

vw_wait_tx();        // We wait to finish sending the message
      delay(200);        // We wait to send the message again               
    }

}*
* *Receiver:* *
*#inlude <VirtualWire.h>

const int led = 3;
const int led1 = 4;

void setup()
{
Serial.begin(9600);
vw_set_ptt_inverted(true);
vw_setup(2000);
vw_set_rx_pin(2);
vw_rx_start();

pinMode(3, OUTPUT);
pinMode(4, OUTPUT);

}

void loop()
{

uint8_t buf[VW_MAX_MESSAGE_LEN];
uint8_t buflen = VW_MAX_MESSAGE_LEN;

if (vw_get_message(buf, &buflen)))
{
int i;

for (i = 0; i < buflen; i++)
{
	Serial.print(buf[i]);
	if(buf[i] == '1'){digitalWrite(3, HIGH);}
	if(buf[i] == '2'){digitalWrite(4, HIGH);}
}
}
else
{
	digitalWrite(3, LOW);
	digitalWrite(4, LOW);
}

}*
```
Thank you for any help at all.

2

Thank you for any help at all.

Probably not, but I'm going to do it, anyway.

    int pin0 = 0;
    int pin1 = 1;

Two numbered variables. Fine.

       int force = analogRead(pin0); //  analog reading from the Force sense resistor
       int force1 = analogRead(pin1); //  analog reading from the Force1 sense resistor

WTF happened to the numbering?

       const char *msg = "1";
       const char *ms = "2";

Ditto.

const int led = 3;
const int led1 = 4;

Ditto.

		Serial.print(buf[i]);

Not going to tell us, huh. Fine. Be that way.

Do you know that the LEDs are wired correctly? Do you KNOW that you are sending anything? If you do, do you know how long the LED will stay on? I'm guessing that the answer to this one is no. Your indenting does not help.

3

Wow, I feel bad whoever works with you in industry. You sound like a pretentious asshole.

Anyways, I figured it out - I had to use

if(buf[0] == '1'){digitalWrite(3, HIGH);}

On the receiver instead of

if(buf[i] == '1'){digitalWrite(3, HIGH);}

So thanks for the GREAT constructive criticism! (That was sarcasm incase you couldn't pick up on that)

4

You sound like a pretentious asshole.

Oh, damn. My secret's out. Though I'm not at all pretentious.

5

@ OP, at least Paul can count. Apparently you can't... or for that matter read your code 'carefully' before asking for help...
Typical N00B attitude...
The Attitude and I've had my share... is meant to kick your loose and flaccid grey matter into action...
I'll warrant that you will be more careful in the future when writing code...Checking for DUMB errors SHOULD be done First....

Doc