udregning giver fejl pga nul

<?php
if(isset($_POST['Beregn'])) {
    $v = $_REQUEST["v"];
    $hcm = $_REQUEST["hcm"];
    $hm = $_REQUEST["hm"];
    $bmi = $v / ($hcm * $hm);
    echo ("BMI er $bmi");
}
?>

Og så husk lige at give din submit knap et navn :)

<input type="submit" name="Beregn" value="Beregn">

exp>> jeg kan se der skal tilføjes en 'fejlbesked' hvis et af felterne er tomme ellers giver den igen Division by zero, er det bedst i form eller vodden' *S*

hov den skal i øvrigt afrunde til nærmeste hele tal og

    $bmi = $v / ($hcm * $hm);

er

    $bmi = ($v / ($hcm * $hm)) * 10000;

Kan man indsætte en kode så det er ligemeget om man skriver punktum eller komma som decimaltal - jeg kan se at denne beregning kun duer hvis man bruger punktum og de fleste har jo komma på deres taltastatur??

ved ikke lige om du kan lave det du forestpørger, men du kan jo udskifte , med .

$hcm = str_replace(",",".",$hcm);

Her kan du finde noget om validering af et tekstfelt :-)
www.html.dk/scripts/javascript/00013/

exp>>
Jeg har selv fundet løsning på afrunding - bruger 'ceil'
så kun 1 ting tilbage

kode:

<?php
if(isset($_POST['Beregn'])) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
    echo ("<b>$bmi</b>");
}
?>

hvis der ikke udfyldes i begge felter - altså vkg og hcm - så skal der komme en besked herom - hvordan får jeg flettet den ind?

<script type="text/javascript">
<!-- Dette script og mange flere -->
<!-- findes hos http://www.html.dk -->
<!-- Start

function validering()
  {
  error = 0;
  if((document.forms[0].vkg.value=='') || (document.forms[0].hcm.value=='') && (error==0)) 
  {       
    alert('Tekstfeltet skal udfyldes!');
    document.forms[0].tekstfeltnavn.focus();
    error = 1;       
  }           

  if(error == 0)
  document.forms[0].submit();   
  }

// Slut -->
</script>

Ved skam godt det er exp's link - men tror det burde funke...

jaw>> jeg er kun ude efter php-løsninger, sorry

ohh, jamen så kan du vel lave noget a'la:

if($vkg && $hcm) {
  //beregn
} else {
  echo "Du skal udfylde begge felter";
}

Husk lige at opdatere det så det virker med alle tilgængelige php-versioner:

if(isset($_POST['vkg']) && isset($_POST['hcm'])) {
  //beregn
} else {
  echo "Du skal udfylde begge felter";
}

Men hvorfor validere det på serveren, istedet for på klienten?

Tjah, altså jeg har også brugt et javascript i stil med dit Exp flere steder, og synes bestemt det er den mest optimale løsning... Man slipper også for det problem, at felterne bliver tømt, i stedet for at brugeren bare kan indtaste i de manglende felter...

exp>>

denne giver

<?php
if(isset($_POST['vkg']) && isset($_POST['hcm'])) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
    echo ("<b>$bmi</b>");
}
else {
  echo "Du skal udfylde begge felter";
}
?>

at 'du skal osv' står der når siden kaldes og ikke hvis man glemmer et felt.
Samtidig kommer den med Warning: Division by zero in hvis du kun udfylder et felt eller slet ingen - så den lukker altså ikke af for forkert beregning!!!

Du får lige hele koden for en sikkerheds skyld:

----------------------

<form class="formsoeg2" method="post" action="<?php echo $_SERVER['php_self']; ?>">
<table border="0" cellpadding="2" cellspacing="0">
<tr>
<td>Vægt</td>
<td><input name="vkg" type="text" size="1" maxlength="3"> kg<br></td>
</tr><tr>
<td>Højde</td>
<td><input name="hcm" type="text" size="1" maxlength="3"> cm<br></td>
</tr><tr>
<td><input type="submit" name="BMI" value="BMI"></td>
<td>
<?php
if(isset($_POST['vkg']) && isset($_POST['hcm'])) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
    echo ("<b>$bmi</b>");
}
else {
  echo "Du skal udfylde begge felter";
}
?>
</td>
</tr></table>
</form>

-------------------------------------------

glemte lige ()

echo ("Du skal udfylde begge felter");

Du skal lave 2 check:

1. Er der trykket på [ BMI ]?
Er der - gå videre til 2
Er der ikke - foretag intet

2. Er $hcm = 0?
Er den - meld fejl
Er den ikke - beregn

er det noget ligende dette du mener

<?php
if(isset($_POST['vkg']) && isset($_POST['hcm'])) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
                   
      if(empty($_REQUEST['vkg']) || empty($_REQUEST['hcm']))
      {
                echo ("Du skal udfylde begge felter");
      }       

      if($hcm = 0)
      {
                echo ("");
      }       

}
else {
    $bmi = ($bmi);
        echo ("<b>$bmi</b>");
}
?>

ikke helt...

Først skal du checke:

if(isset($_POST['BMI'])) {
  // if(isset($_POST['hcm']) && $_POST['hcm'] != 0) {
    // udfør beregning
    }
  } else {
    echo "Fejl";
  }
}

pseudo-kode...

mener du ikke sådanne:

if(isset($_POST['BMI'])) {
        if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
        echo ("<b>$bmi</b>");
    }
  } else {
    echo "Fejl";
  }

hvor if(isset($_POST['hcm']) && $_POST['hcm'] != 0) {

skal være if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {

hvad mener du med pseudo-kode...?

ovennævnte giver stadig 'Fejl' ved kald af side ellers regner den som den skal.

exp>>

hvis jeg bruger følgende kode, virker den, men den er jo ikke helt korrekt, idet else vises ved kald af side, nu ser man bare ikke noget!!!!

<?php
if(isset($_POST['BMI'])) {
        if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
        echo ("<b>$bmi</b>");
    }
  }
else {
                echo (" ");
}
?>

Kan du genneskue det??? for så er vi færdige*S*

Prøv:

<?php
if(isset($_POST['BMI'])) {
  if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
        echo ("<b>$bmi</b>");
    }
else {
                echo ("Fejl!");
  }
}
?>

jaw>>

tak det hjalp *S*

det var da godt :) Den forrige løsning, ville jo give "fejl" i alle tilfælde, hvor $_POST['BMI'] ikke var sat... Altså, også første gang du åbner siden...

jaw>>
ja, der burde bare være en slags nulstiller efter beregning, for hvis man vil opdatere siden efter en beregning kommer der en box frem med at siden ikke kan kan opdateres uden der sendes oplysninger igen - det er lidt irreterende*S*

ja, det er klart... Den tror jeg ikke, du kan slippe for, uden du skal ud i nogen headers og andre sider...

Den bedste måde at komme uden om det er ved at benytte sig af en udregn.php, hvori der så henvises til beregner.php

header('location:udregn.php');

Derved kommer du også HELT uden om problemet med fejlen at der ikke er indtastet noget når siden loades...

ja, det må jeg så nok leve med så, idet koden er lavet for sig selv og includeres i siden hvor beregning så også finder sted - ellers skal der fremkomme en ny side og det duer ikke i dette tilfælde - eller har jeg misforsået noget?

Brugeren behøver aldrig nogensinde at se en ekstra side...

<form class="formsoeg2" method="post" action="udregn.php">
<table border="0" cellpadding="2" cellspacing="0">
<tr>
<td>Vægt</td>
<td><input name="vkg" type="text" size="1" maxlength="3"> kg<br></td>
</tr><tr>
<td>Højde</td>
<td><input name="hcm" type="text" size="1" maxlength="3"> cm<br></td>
</tr><tr>
<td><input type="submit" name="BMI" value="BMI"></td>
<td>

<?php
if(isset($_SESSION['bmi']) && $_SESSION['bmi'] != 0) {
  echo "<b>".$_SESSION['bmi']."</b>";
}
?>

</td>
</tr></table>
</form>


Udregn.php
----------------------------------------

<?php
session_start();

if(isset($_POST['BMI'])) {
  if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
    $_SESSION['bmi'] = $bmi;
    header('location:'.$_SERVER['HTTP_REFERER']);
    exit;
  } else {
    $_SESSION['bmi'] = "Fejl!";
    header('location:'.$_SERVER['HTTP_REFERER']);
  }
}
?>

exp>>
Det er jo en fed løsning, som kan bruges til meget, men jeg kan bare ikke få den til at virke. Når jeg trykker på knap BMI kommer der intet retur - kan du se om der er indsneget sig en fejl???

------------------------------------

bmi_beregn.php

<html><head>
<title>beregning</title>
<link rel="stylesheet" href="../css/hamlet.css" type="text/css">
</head><body>
<b>BMI beregning</b>
<form class="formsoeg2" method="post" action="bmi_udregn.php">
<table border="0" cellpadding="2" cellspacing="0">
<tr>
<td>Vægt</td>
<td><input name="vkg" type="text" size="1" maxlength="3"> kg<br></td>
</tr><tr>
<td>Højde</td>
<td><input name="hcm" type="text" size="1" maxlength="3"> cm<br></td>
</tr><tr>
<td><input type="submit" name="BMI" value="BMI" style="font-size : 9px;"></td>
<td>
<?php
if(isset($_SESSION['bmi']) && $_SESSION['bmi'] != 0) {
  echo "<b>".$_SESSION['bmi']."</b>";
}
?>
</td>
</tr></table>
</form>   
</body></html>

---------------

bmi_udregn.php

<?php
session_start();

if(isset($_POST['BMI'])) {
  if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
    $vkg = $_REQUEST["vkg"];
    $hcm = $_REQUEST["hcm"];
    $bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);
    $_SESSION['bmi'] = $bmi;
    header('location:'.$_SERVER['HTTP_REFERER']);
    exit;
  } else {
    $_SESSION['bmi'] = "Fejl!";
    header('location:'.$_SERVER['HTTP_REFERER']);
  }
}
?>

-----------------------------

Så vidt jeg lige kan se, mangler du session_start(); i din bmi_beregn.php

nej den giver bare:

Cannot send session cache limiter - headers already sent

og du sætter den aller først i filen? Før <html><head> og hvad der ellers er...

<?php session_start();  ?> i top det hjalp, men den udskriver ikke mere "Fejl!"

trykker man bare på BMI knappen kommer der ikke fejl eller hvis felt ikke udfyldt, mon der skal være ( ) om fejl!

men den returnerer udregningen korrekt?

ja

Hej igen, for at lære æidt kan I så fortælle mig, hvorfor man med session godt kan regne med komma, hvor den anden løsning ikke kunne , men kun med punktum???

hvis jeg gerne vil have

$bmi = ceil(($vkg / ($hcm * $hcm)) * 10000);

med en decimal, så skal jeg vel bruge round, men hvor skriver man decimalen

$bmi = round(($vkg / ($hcm * $hcm)) * 10000);

burde denne ikke virke?

$bmi = round((($vkg / ($hcm * $hcm)) * 10000), 1);

fandt ud af det, den virker hvis jeg gør sådanne:

$vkg = str_replace(",",".",$vkg);       
$bmi = round((($vkg / ($hcm * $hcm)) * 10000), 1);

men hvorfor vil den ikke udskrive dette udsagn

else {
    $_SESSION['bmi'] = "Fejl!";
    header('location:'.$_SERVER['HTTP_REFERER']);
  }

den retunerer slet ikke Fejl! på noget tidspunkt????

hvad hvis du fjerner headeren midlertidigt, og indsætter en echo, får du så "Fejl!" udskrevet?

hvis jeg ændre til:

else {
    $_SESSION['bmi'] = ("Fejl!");
    echo "fejl her";
  }

skriver den fejl her, hvis der ikke er indtastet noget i begge felter og hvis der mangler i hcm, men ikke hvis der mangler i vkg.

Fejlen skrives heller ikke på samme side men på en ny.

:-)

Forstår jeg ikke helt :/

hvilken header mente du da

jeg mente den du piller ved...
Den her linie:  if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
prøv at ændre den til  if($_POST['vkg'] != 0 && $_POST['hcm'] != 0) {

hvis jeg ændre til
if($_POST['vkg'] != 0 && $_POST['hcm'] != 0) {
giver det samme resultat, ingen 'fejl' med

else {
    $_SESSION['bmi'] = ("Fejl!");
    header('location:'.$_SERVER['HTTP_REFERER']);
  }

og 'fejl her' på blank side ved

else {
    $_SESSION['bmi'] = ("Fejl!");
    echo "fejl her";
  }

Fejlen ligger hvor du echoer den på din bmi_beregn.php
if(isset($_SESSION['bmi']) && $_SESSION['bmi'] != 0) { -> ændre til:
if(isset($_SESSION['bmi']) && $_SESSION['bmi'] != "0") {

ja nu kommer 'fejl!'

men desværre også når man kalder siden bmi_beregn.php, ellers virker alt ok

Prøv at lukke din(e) browsere og åbne dem igen og prøv igen... Skyldes det ikke bare, at du allerede har "Fejl" stående i din session?

hvad er mest korrekt at bruge

if(isset($_POST['vkg']) && $_POST['hcm'] != 0) {
eller
if($_POST['vkg'] != 0 && $_POST['hcm'] != 0) {

det ser ud til at give samme resultat!!!

Den første vil jeg nok mene er den bedste...

Det andet var bare lidt for at prøve, hvor fejlen lå... Ikke altid, jeg man lige kan gennemskue det i første gæt :(

jeg havde ellers tømt midlertidige filer hver gang men ikke lukket browser og så virker det skisme.

Tak for din ihærdige hjælp - hvordan kan jeg nu give jer begge point?

Hehe, no problemo... Hvis du får et svar fra os begge, kan du bare acceptere os begge, så burde vi dele pointene...

http://expfaq.1go.dk/?id=21#mere_end_et_svar

Nå, så du lurer alligevel, Exp?! :)

ja sikken en*G*

nu har jeg givet jer 60 hver 2 gange men den slutter ikke, så jeg må nok bare prøve at dele

Vil i have karma

hehe, det der karma har jeg vist ikke så meget af - men det ser jo meget pænt ud :D Og tak for point...

jaw>> så har du også fået karma, tak endnu engang til jer begge

Wow, tak - og selv tak :)

det var bestemt så lidt, men tænk at sådan et lille regnestykke kan volde så mange problemer, godt det ikke er et stort matematisk regnestykke jeg ville lave*S*

Heh, tjah... Sidder på skolen og laver projekt, så havde ikke lige til at bygge det op for at teste det og selv kunne bytte rundt... Ellers havde det (måske!) nok være lidt nemmere...

jaw >>
Sidder på arbejde, så kan ikke så godt yde 100%, vel ;-)

exp - hehe, det var også kun ment som en joke, at du lige smed en lille diskret kommentar midt i det hele :) Og man skal jo passe på med at overanstrenge sig, FØJ!

:-)

hej igen exp og jaw,

jeg ville gerne udbygge den lidt mere, (vise et svar alt efter det udregnede tal)
så hvis I er på tænkte jeg at oprette et nyt ? med flere point
og så vise linket her.

Gider I... :-)

Jeg er her :)

Du opretter bare et nyt, så skal vi nok finde det :-)

Bare husk, at det er tilladt for alle at svare...

ok jeg opretter og vender tilbage :-)

her er den så http://www.eksperten.dk/spm/508603