[PHP]Problem z echo Opcje

[kubax33]

mam taki kod:

[PHP] pobierz, plaintext 
<?php
$szukane = $_GET['info'];
$plik = "db.txt";
$dane = file($plik); 
 
if ((isset($_POST['nazw'])) AND (trim($_POST['nazw']) != "")) 
{
 
  for($i=0;$i<count($dane);$i++) 
    list($imie[$i], $nazwisko[$i], $miejscowosc[$i]) = explode(" || ", $dane[$i]);
 
 
  for($i=0;$i<count($nazwisko);$i++)
   if (strpos($nazwisko[$i],$_POST['nazw']) !== false && $_GET['info'] == $id[$i])
 
 
     echo $imie[$i]." ".$nazwisko[$i].", ".$miejscowosc[$i]."<br />";
 
/* tutaj else echo!!!!!!!!!!!!!!!!!!!!!!!!!!!!
}
     
}
?>
[PHP] pobierz, plaintext 

i kombinuje na wszelkie sposoby, ale nie moge dopisać else echo w miejscu, którym wskazałem czyli chodzi mi o to jeśli nie będzie spełniony "if" pokaże się informacja! Proszę o pomoc

Ten post edytował kubax33 1.02.2012, 15:28:10

[seba123]

Polecam stosować nawiasy { } to będzie wszystko jasne.

[slawek3422]

[PHP] pobierz, plaintext 
<?php
$szukane = $_GET['info'];
$plik = "db.txt";
$dane = file($plik);
 
if ((isset($_POST['nazw'])) AND (trim($_POST['nazw']) != "")){
 
    for($i=0;$i<count($dane);$i++){
        list($imie[$i], $nazwisko[$i], $miejscowosc[$i]) = explode(" || ", $dane[$i]);
    }
 
    for($i=0;$i<count($nazwisko);$i++){
        if (strpos($nazwisko[$i],$_POST['nazw']) !== false && $_GET['info'] == $id[$i]){
            echo $imie[$i]." ".$nazwisko[$i].", ".$miejscowosc[$i]."<br />";
        }   else    {
            echo "HELP !!";
        }
 
 
    }
 
}
?>
[PHP] pobierz, plaintext 

[Sephirus]

Naucz się używać klamr {}

[PHP] pobierz, plaintext 
    <?php
    $szukane = $_GET['info'];
    $plik = "db.txt";
    $dane = file($plik);
 
    if ((isset($_POST['nazw'])) AND (trim($_POST['nazw']) != ""))
    {
 
         for($i=0;$i<count($dane);$i++)
         {
                    list($imie[$i], $nazwisko[$i], $miejscowosc[$i]) = explode(" || ", $dane[$i]);
         }
 
         for($i=0;$i<count($nazwisko);$i++)
         {
                if (strpos($nazwisko[$i],$_POST['nazw']) !== false && $_GET['info'] == $id[$i])
                        echo $imie[$i]." ".$nazwisko[$i].", ".$miejscowosc[$i]."<br />";
                else
                         echo '!!!';
          }
    }
    ?>
[PHP] pobierz, plaintext 

[by_ikar]

Użyj klamerek. Przy jednym bloku możesz sobie pozwolić na nie danie klamerek. Przy zagnieżdżeniu wielu bloków, lepiej użyć klamerki, żeby potem nie głowić się dlaczego sypie mi errorem.

[kubax33]

mam spore problemy z tymi klamrami przyznaje!

już trace cierpliwość co tutaj jest źle?

[PHP] pobierz, plaintext 
 
$szukane = $_GET['zamowienie'];
   if ($szukane)
   {
 
$plik = "db.txt";
$dane = file($plik);
    {
if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != "")) {
         {
  for($i=0;$i<count($dane);$i++) 
{
    list($nr[$i], $nick[$i], $ilosc[$i]) = explode("|", $dane[$i]);
  }
  for($i=0;$i<count($nick);$i++)
{
   if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
{
     echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
 
else
echo '!!!';
 
}
 
}
 
[PHP] pobierz, plaintext 

i czy kto jest wstanie mi logicznie wytlumaczyć jak stosuje się te klamry albo jakiś dobry link podesłać bo szczerze mówiąc często robie je metoda prób i błędów edytując jakiegos gotowca :/

[Sephirus]

Zrób inaczej - skoro masz z tym problem to używaj ich wszędzie i zawsze obojętnie od liczby linijek w nich

[PHP] pobierz, plaintext 
// dla warunków :
 
if(...) {
    // kod...
    // kod...
}
 
// pętli :
 
foreach(...) {
    // kod...
}
 
[PHP] pobierz, plaintext 

i tak dalej... i pamiętaj o dobrym formatowaniu kodu i wcięciach! Czytelny kod to taki w którym łatwiej znaleźć błąd, zarówno logiczny jak i składniowy

Twój kod po "próbie" sformatowania:

[PHP] pobierz, plaintext 
$szukane = $_GET['zamowienie'];
if ($szukane)
{
	$plik = "db.txt";
	$dane = file($plik);
	{ // WTF? po co tutaj ta klamerka
		if((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != "")) 
		{
			{ // i po co tutaj??
				for($i=0;$i<count($dane);$i++)
				{
					list($nr[$i], $nick[$i], $ilosc[$i]) = explode("|", $dane[$i]);
				}	
				for($i=0;$i<coun($nick);$i++)
				{
					if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
				{
				echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
 				else // do czego to else niby??
				echo '!!!';
				}
			}
// gdzie reszta klamerek :)
[PHP] pobierz, plaintext 

Ten post edytował Sephirus 1.02.2012, 16:13:11

[kubax33]

Jakoś trace siły żeby się męczyć 3 h z klamrami ehhh ciężko mi to opanować. Zrobiłem tak jak mówiłeś tylko...

[PHP] pobierz, plaintext 
$szukane = $_GET['zamowienie'];
 
   if ($szukane)
{
 
$plik = "db.txt";
$dane = file($plik); 
 
if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != "")) 
         {
  for($i=0;$i<count($dane);$i++) 
{
    list($nr[$i], $nick[$i], $ilosc[$i]);
  }
  for($i=0;$i<count($nick);$i++)
{
   if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
{
     echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
else
echo '!!!'; // musze mieć else zeby zwracało błąd jeśli nie są spełnione warunki if-a (  if (strpos($nick[$i],$_GET['zamowienie'] itd. )
}
 
}
} // te klamry miałem zamknąć wcześniej otwierając górne?
}
 
 
[PHP] pobierz, plaintext 

[untorched]

[PHP] pobierz, plaintext 
$szukane = $_GET['zamowienie'];
 
if ($szukane)
{
	$plik = "db.txt";
	$dane = file($plik);
 
	if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != ""))
	{
		for($i=0;$i<count($dane);$i++)
		{
			list($nr[$i], $nick[$i], $ilosc[$i]);
		}
 
		for($i=0;$i<count($nick);$i++)
		{
			if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
			{
				echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
			}
			else
			{
				echo '!!!'; // musze mieć else zeby zwracało błąd jeśli nie są spełnione warunki if-a ( if (strpos($nick[$i],$_GET['zamowienie'] itd. )
			}
		}
	}
}
[PHP] pobierz, plaintext 

[kubax33]

zadziałało dziękuję! Jest tylko jeden problem że w pomencie kiedy zostaje wywoływane else echo zamiast "!!!" pokazuje mi ich :
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

czyli kilkunastokrotnie wpisuje to co jest w else echo. Czego to moze byc wina?

do tego teraz jednocześnie pokazuje na stronie to co jest w samym echo oraz w else echo :/ cos jest nie tak...

[PHP] pobierz, plaintext 
 
<?php
$szukane = $_GET['zamowienie'];
 
   if ($szukane)
{
 
$plik = "db.txt";
$dane = file($plik); 
 
if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != "")) 
         {
  for($i=0;$i<count($dane);$i++) 
{
    list($nr[$i], $nick[$i], $ilosc[$i]) = explode("|", $dane[$i]);
  }
  for($i=0;$i<count($id);$i++)
{
   if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
{
     echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
}
else
{
echo '!!!';
 
}
 
}
}
}
 
?>
[PHP] pobierz, plaintext 

[untorched]

To jest wina tego, że instrukcja for(15 linijka) dalej się wykonuje. Jeśli chcesz ją przerwać gdy wystąpi echo '!!!'; to pod tym echo daj break;, lub napisz to w ten sposób:

[PHP] pobierz, plaintext 
$szukane = $_GET['zamowienie'];
$error = 0;
 
if ($szukane)
{
	$plik = "db.txt";
	$dane = file($plik);
 
	if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != ""))
	{
		for($i=0;$i<count($dane);$i++)
		{
			list($nr[$i], $nick[$i], $ilosc[$i]);
		}
 
		for($i=0;$i<count($nick);$i++)
		{
			if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
			{
				$zwrot .= $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
			}
			else
			{
				$error = 1;
			}
		}
 
		if($error == 1)
		{
			echo 'Wystąpił błąd!';
		}
		else
		{
			echo $zwrot;
		}
	}
}
[PHP] pobierz, plaintext 

PS. Obstawiam że ta linijka:

[PHP] pobierz, plaintext 
for($i=0;$i<count($nick);$i++)
[PHP] pobierz, plaintext 

Ma wyglądać tak:

[PHP] pobierz, plaintext 
for($i=0;$i<count($nick-1);$i++)
[PHP] pobierz, plaintext 

Ten post edytował untorched 1.02.2012, 17:01:49

[kubax33]

a jednak nadal szwankuje coś

[PHP] pobierz, plaintext 
<?php
$szukane = $_GET['zamowienie'];
 
   if ($szukane)
{
 
$plik = "db.txt";
$dane = file($plik); 
 
if ((isset($_GET['zamowienie'])) AND (trim($_GET['zamowienie']) != "")) 
         {
  for($i=0;$i<count($dane);$i++) 
{
    list($nr[$i], $nick[$i], $ilosc[$i]) = explode("|", $dane[$i]);
  }
  for($i=0;$i<count($id);$i++)
{
   if (strpos($nick[$i],$_GET['zamowienie']) !== false && $_GET['zamowienie'] == $nick[$i])
{
     echo $nr[$i]." ".$nick[$i].", ".$ilosc[$i]."<br />";
break;
}
else
{
echo '!!!';
brak;
}
 
}
}
}
 
?>
 
 
[PHP] pobierz, plaintext 

mianowicie jeśli wyśle przez get i wyswietli dane to zawsze jest na poczatku else echo czyli : "!!!!!!!!!!" i do tego dopiero echo wyswietlone (prawidlowo)
Jeśli wywołam else echo to jest ono znowu kilkunastokrotnie powtorzone czyli "!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!"

A ten sposób drugi co wysłałeś "untorched" wyrzuca bład cały czas przy każdej akcji.

Jeśli ma ktos jakies pomysły będę wdzięczny...

Ten post edytował kubax33 1.02.2012, 17:37:56

[untorched]

Nie łaska wrzucić jakie błędy? Na oko nie zawsze wszystko da się zauważyć.. Poniżej masz działający, testowany skrypt

[PHP] pobierz, plaintext 
$szukane = (isset($_GET['zamowienie']) ? $_GET['zamowienie'] : NULL);
 
if ($szukane !== NULL)
{
	$plik = 'db.txt';
	$dane = file($plik);
 
	if (trim($szukane) != '')
	{
		foreach($dane as $linia => $text)
		{
			list($nr[$linia], $nick[$linia], $ilosc[$linia]) = explode('|', $text);
		}
 
		$zwrot = '';
 
		for($i=0; $i<count($nr); $i++)
		{
			if (strpos($nick[$i],$szukane) !== false && $szukane == $nick[$i])
			{
				$zwrot .= $nr[$i].', '.$nick[$i].', '.$ilosc[$i].'<br />';
			}
 
		}
 
		if($zwrot != '')
		{
			echo $zwrot;
		}
		else
		{
			echo '!!!';
		}
	}
}
[PHP] pobierz, plaintext 

Ten post edytował untorched 1.02.2012, 20:38:51

[kubax33]

Dziękuję uprzejmie działa bez zarzutów o to chodziło.

Pozdrawiam

[--Bono--]

Uczę się PHP za pomocą pewnej książki HELIONU, mam tam kod

[PHP] pobierz, plaintext 
<?php
  echo "<p>Zamówienie przyjęte o:</p>";
  echo date('H:i. jS  F');
  echo '</p>';
  echo '<blink><h2>Zamówienie Państwa wygląda następująco:</h2></blink>';
  echo $iloscopon.'<center> opon</center><br />';
  echo $iloscoleju.'<center> butelek oleju</center><br />';
  echo $iloscswiec.'<center> Swiec zapłonowych</center><br />';
  $ilosc = 0;
  $ilosc = $iloscopon + $iloscoleju + $iloscswiec;
  echo 'Zamówionych częsci:   '.$ilosc.'<br />';
 
  $wartosc = 0.00;
 
  define('CENAOPON'. 400);
  define('CENAOLEJU'. 100);
  define('CENASWIEC'. 40);
 
  $wartosc = $iloscopon * CENAOPON  /* Tu Rozpoczyna się problem */
             + $iloscoleju * CENAOLEJU    /* np.zamiast "CENAOLEJU" podam 100 to obliczy bez problemu*/
             + $iloscswiec * CENASWIEC;
 
  echo 'Cena netto: '[font="Arial Black"].[b]$wartosc[/b].[/font]' PLN<br />';/*tu daje format type*/
  $stawkavat = 0.23;
  $wartosc = $wartosc * ( 1 + $stawkavat );
  echo 'Cena brutto: '[font="Arial Black"].[b]$wartosc[/b].[/font]' PLN<br />'; /*Tu daje format_type*/
?>
[PHP] pobierz, plaintext 

A mianowicie gdy zdefiniowałem słowa CENA-||- mimo to nie potrafi wyczytać że cena np. CENAOPON to to samo co 400
po prostu daje wynik 0.
Problem rozwiązuje danie kodu
$wartosc = $iloscopon * 400 I TAK DALEJ.
Ale mnie to nie zadowala. Czemu mi nie działa "define"?
Mam jeszcze problem z
'.format_type($wartosc. 2).'
Co powinno mi dać (ja tak to rozumie) 2 wyrazy po przecinku tak jak to np.(3.21)=e
a daje mi tak (0.321)=e
Czemu?
Próbowałem to rozgryźć na wszelkie sposoby nie widzę błędu? Może w php.ini?

[by_ikar]

define ci nie działa, bo użyłeś zamiast przecinku żeby podać drugi parametr, to użyłeś kropki, powinieneś dostać warning, ale pewnie masz wyłączone raportowanie błędów. Tak samo jest w format_type tam również podałeś kropkę zamiast przecinku.

[kubax33]

i napotkałem kolejny błąd odnośnie mojego kodu. W firefoxie jest ok, ale w IE wyrzuca błąd :
Notice: Undefined offset:

tyczy się to linijki:

[PHP] pobierz, plaintext 
list($nr[$linia], $nick[$linia], $ilosc[$linia]) = explode('|', $text);
[PHP] pobierz, plaintext 

Co tutaj jest źle? Niby wyczytałem, że pętla powtarza się w nieskończoność? jak temu zaradzić.

Zastanawiam się czy związku z tym nie ma też:

[PHP] pobierz, plaintext 
$szukane = (isset($_GET['zamowienie']) ? $_GET['zamowienie'] : NULL);
[PHP] pobierz, plaintext 

Ten post edytował kubax33 5.02.2012, 12:13:41