opstilling, indrykning osv

Meget flot opsætning, dog har jeg to rettelser, du har følgende:

if(isset($push)) {
    function send_mail() {
        //declares the variables there is in the send mail function
        $email        =    $_POST['email'];
        $subject    =    'test';
        $message    =    'hallowwe';

$_POST['email']; er rykket for langt ud, den er sikkert en fejl. Følgende:
if(isset($push)) {

Der plejer jeg at lave mellemrum mellem if og betingelsen:
if (isset($push)) {

Jeg peljer desuden ikke at tabulerer mellem variablen og dens 'lig med' tegn.
$subject    =    'test';
Skriver jeg blot:
$subject = 'test';

den sidste du har skrevet med tab mell. var og = tegn er for jeg kan få stillet det pænt på en linie og få det til at være overskueligt

Jeg bruger kun en indrykning på 2 tegn. Smag og behag.

Det er godt at set et forsøg på at lave al beregning før evt HTML, men
du udnytter det dog ikke. Din

echo 'you are in the army<br />';

Kunne have været en

  $text = 'you are in the army<br />';

med en efterfølgende

  <? print $text; ?>

ned i HTML-en. Man kan faktisk adskille udregninger og så præsentationen
i HTML. Og bør - syn's jeg.

Men i realiteten kan jeg ikke se formålet med at 'pakke' din funktion ind i:
if(isset($push)) {
  function send_mail() {
  .........
}
Det må være nok du bruger den her:

if(isset($push)) {
    send_mail();
}

Og så skal du nok bruge:

if(isset($_POST['push'])) {
    send_mail();
}

så det osse virker med register_globals=Off.

Jeg kunne godt lige se fejlen der i function send_email.
og til dig erik jeg

og erik har rettet det du snakkede om men $text

dog kan jeg ikke helt forstå den her linie --->

ned i HTML-en. Man kan faktisk adskille udregninger og så præsentationen
i HTML. Og bør - syn's jeg.

Du har ca. denne struktur:

<?
  // en masse beregninger i PHP
?>
<html>
<!--en masse HTML, evt med udskrifter af variabler/arrays fra PHP-->
</html>

Det er den opdeling jeg mener man altid skal bruge - hold de to ting adskilt.

Rart at se at du overvejer hvordan struktureringen af dine kode er.
Personligt er jeg ikke så meget for:
<?PHP echo $text; ?>
Jeg plejer at bruge:
<?= $text; ?>
Da det, IMA, er en del nemmer at læse. Men smag og behag ;-)

Det er ikke smag og behag. Din version, <?= ... ?>, virker kun hvis
den aktuelle PHP-implementation har short-tags slået til. Man bør
selvfølgelig skrive kode, der altid virker, uanset sådan en indstilling.

Afslutningen af din funktionserklæring er indrykket. Det er ganske modsat "standard", og er derfor lidt forvirrende.

Nogle kan bedst lide det således:

function hejsa($variable)
{
  lavnoget;
}


Selv foretrækker jeg

function hejsa($variable) {
  lavnoget;
}

Men dit eksempel havde

function hejsa($variable) {
  lavnoget;
  }

Og så er det svært at få øje på slutningen af funktionen, når man bare skimmer ned over koden ved samme indrykningsgrad som begyndelsen af funktionsdefinitionen.

Derudover kan jeg tilslutte mig Eriks kommentarer om opdeling af kode og output. (Jeg er selv dårlig til den del, men jeg ved godt at det er den rigtige måde at gøre det.)

Til sidst vil jeg ønske dig tillykke med beslutningen om at skrive læselig kode. Det vil for det første gøre det nemmere for dig selv at overskue koden, og for det andet vil det også gøre det nemmere for dig at få hjælp her. Der er ingen, der gider skulle rode igennem flere hundrede liniers ustruktureret kode for måske at finde en manglende }.

hehe nej det var osse derfor jeg besluttede mig for det. jeg kan se det kun er ahv der har lavet svar her så ham får point takker alle sammen for indlagende