[wolontariat]Instrukcja if, problem Opcje

[mlody93]

Witam.
Dlaczego ten skrypt nie działa?

zmienna $lum jest podawana przez formularz

if ( 3400 < $lum ) {
$lampa = "lampa 50 W";
} elseif ( 4500 > $lum ){
$lampa = "lampa 50W";
} elseif ( 4501 < $lum ){
$lampa = "lampa 70 W";
} elseif ( 7000 > $lum ){
$lampa = "lampa 70 W";
} else if ( 7001 < $lum ){
$lampa = "lampa 100 W";
} elseif ( 12000 > $lum ){
$lampa = "lampa 100 W";
}


Aby oświetlić pokój potrzebujesz: $lampa

Jeżeli ktoś mi pomoże będę wdzięczny

[michal_robak]

a walnij echo $lum przed tą formułą i powiedz co ci wyskoczy. i daj w kazdym ifie coś takie echo "jestem w x warunku" i sprawdzisz czy ci wchodzi do ktorejś pętli. Debugować debugować i jeszcze raz debugować w takich wypadkach (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) !!

Ten post edytował michal_robak 2.07.2009, 12:10:28

[viking]

Zapewne $_POST['lum']. I poczytaj o register_globals.

[MateuszS]

A jaki błąd? Przypisałeś zawartość tablicy $_POST/$_GET do $lum?

$lum = $_POST['lum'];

[mlody93]

a walnij echo $lum przed tą formułą i powiedz co ci wyskoczy.

Wyskoczy mi wartość podana w formularzu

[Spawnm]

korzystaj z bbcode.
daj na początku

[PHP] pobierz, plaintext 
<?php
$lum=(int)$lum;
?>
[PHP] pobierz, plaintext 

[MateuszS]

dodaj na koncu zamiast ostatniego elseif

[PHP] pobierz, plaintext 
<?php
else { $lampa = "lampa 100 W"; }
?>
[PHP] pobierz, plaintext 

PS ten skrypt nie pokazuje jaka lampe potrzeba, musisz dodac pod kazdym warunkiem
echo $lampa;

[Wicepsik]

[PHP] pobierz, plaintext 
<?php
$lum=(int)$_POST['lum'];
 
if ($lum < 3400){
   $lampa = 'Wartość lum jest mniejsza niz 3400';
}elseif(3400 <= $lum and $lum <= 4500){  // Gdy zmienna lum jest między 3400 a 4500
    $lampa = "lampa 50W";
}elseif (4501 <= $lum and $lum <= 7000){ // Gdy zmienna lum jest między 4501 a 7000
   $lampa = "lampa 70 W";
}else if ( 7001 <= $lum and $lum <= 12000){ // Gdy zmienna lum jest między 7001 a 12000
   $lampa = "lampa 100 W";
}else{
   $lampa = 'Wartość lum przekroczyła 12000';
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Wicepsik 2.07.2009, 12:19:00

[mlody93]

Dzięki Wicepsik!
Lecz zrobiłeś 2 błędy bodajże w swoim kodzie.
Wielkie dzięki.

A co zrobić żeby.
W formularzu gdzie jest zmienna $lum podaje ktoś literę a nie liczbę.
Żeby zwróciło błąd?

[Spawnm]

albo odbieraj jak pisałem czyli z (int)
albo na końcu daj

[PHP] pobierz, plaintext 
<?php
}else{
  echo 'error';
}
?>
[PHP] pobierz, plaintext 

[Pawel_W]

is_numeric()

zwraca true albo false

[mlody93]

A mógłbym poprosić o jakiegoś gotowca?

[Pawel_W]

if(is_numeric($lum))
{
// kod ten co masz wyżej
}
else
{
echo 'błąd';
}

[Spawnm]

Przenoszę.

Przypominam o stosowaniu bbcode...

[mlody93]

Paweł, coś nie tak z tym... Co ma być na miejscu tekstu za slashami?

[Wicepsik]

[PHP] pobierz, plaintext 
<?php
$lum=$_POST['lum'];
if(is_numeric($lum))
{
 
if ($lum < 3400){
  $lampa = 'Wartość lum jest mniejsza niz 3400';
}elseif(3400 <= $lum and $lum <= 4500){  // Gdy zmienna lum jest między 3400 a 4500
   $lampa = "lampa 50W";
}elseif (4501 <= $lum and $lum <= 7000){ // Gdy zmienna lum jest między 4501 a 7000
  $lampa = "lampa 70 W";
}else if ( 7001 <= $lum and $lum <= 12000){ // Gdy zmienna lum jest między 7001 a 12000
  $lampa = "lampa 100 W";
}else{
  $lampa = 'Wartość lum przekroczyła 12000';
}
 
}
else
{
echo 'błąd';
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Wicepsik 2.07.2009, 12:53:47