First off, the modules require a long wire antenna. What you have connected is a whip antenna which has a ground terminal too. This is not required.

Second, a good preamble is a must.

Third, make sure your code works when wired between tx and rx. Only then, go to the wireless part.

Fourth, make sure your data rate is acheivable by the modules.

JF