[PHP] Ścieżka pliku jako zmienna Opcje

[shpaque]

Witam znów serdecznie; mam tym razem taki problem /pewnie jak zwykle prosty ale mnie juz trafia szlag/ .

[PHP] pobierz, plaintext 
<?php
 
ini_set('display_errors', 1);
error_reporting(E_ALL);
 
"Content-Type: text/plain; charset=UTF-8";
$error_message = "";
if ($_SERVER['REQUEST_METHOD'] == 'POST')
{
   $action = isset($_POST['action']) ? $_POST['action'] : '';
   $database = './../.db/data.db';
   $success_page = './nok.php';
 
   if (!file_exists($database))
   {
      echo 'Nie odnaleziono bazy danych!';
      exit;
   }
 
   if ($action == 'dodaj')
   {
      $newkod = $_POST['kod'];
      $newopis = $_POST['opis'];
      $newcena = $_POST['cena'];
      $newfileupload = 
 
      $items = file($database);
      foreach($items as $line)
      {
         list($zdjecie, $kod, $opis, $cena, $status) = explode('|', trim($line));
         if ($newkod == $kod)
         {
            $error_message = 'Wygenerowany kod już istnieje! Odśwież formularz w celu wygenerowania nowego kodu.';
            break;
         }
      }
      if (empty($error_message))
      {
         $file = fopen($database, 'a');
         fwrite($file, $newkod);
         fwrite($file, '.jpg|');
         fwrite($file, $newkod);
         fwrite($file, '|');
         fwrite($file, $newopis);
         fwrite($file, '|');
         fwrite($file, $newcena);
         fwrite($file, '|1');
         fwrite($file, "\r\n");
         fclose($file);
         header('Location: '.$success_page);
         exit;
      }
   }
}
?>
[PHP] pobierz, plaintext 

chodzi mi o linijke

[PHP] pobierz, plaintext 
$database = './../.db/data.db';
[PHP] pobierz, plaintext 

- jesli jest stala sciezka - wszystko gra; tyle że ja mam ok 10 plików *.db i chcialbym w zaleznosci od wartosci zmiennej zebypodmieniał wartosc pliku bazowego - tyle ze to nie dziala!

Najpierw zrobilem tak
na poczatku strony

[PHP] pobierz, plaintext 
<?php
$category = $_GET['category'];
$generate = rand(0000000, 9999999);
$cat = '';
$text = '';
$dbase = '';
  if ($category == balony)
  {
  $cat = "BA";
  $text = "Balony";
  $dbase = "./../db/balony.db";
  }
  else if ($category == ceramika)
  {
  $cat = "CE";
  $text = "Ceramika";
  $dbase = "./../db/ceramika.db";
  }
[PHP] pobierz, plaintext 

i tak dalej = więc zmienna $dbase przybiera rozne wartosci, tyle ze jesli pozniej zrobie

[PHP] pobierz, plaintext 
$database = $dbase;
[PHP] pobierz, plaintext 

to nic nie pobiera... jak to rozwiazac?

[Fifi209]

Popraw warunki, bo stringi trzymamy w ' lub "

[mortus]

Nie prościej byłoby zrobić po prostu tak:

[PHP] pobierz, plaintext 
if(isset($_GET['category']) && $_GET['category'] != '') {
 $dbName = $_GET['category'].'.db';
 $dbDirectory = './../db/';
 $dbPath = $dbDirectory.$dbName;
 // reszta skryptu
} else {
 // coś jest nie tak, więc trzeba ten błąd obsłużyć
}
[PHP] pobierz, plaintext 

lub tym podobnie.
Poza tym spójrz, jak dokonujesz porównań:

[PHP] pobierz, plaintext 
if ($category == balony) {
// ...
} else if ($category == ceramika) {
// ...
}
[PHP] pobierz, plaintext 

Warunki nigdy nie są spełnione, bo porównujesz zmienne z niezdefiniowanymi stałymi (powinno być 'balony', 'ceramika'), a co za tym idzie zmienna $database przyjmuje wartość zmiennej $dbase, która to z kolei jest pustym łańcuchem znaków.
W rozwiązaniu problemu mogło pomóc włączenie wyświetlania i raportowania wszystkich błędów w tym pliku/skrypcie.

Ten post edytował mortus 14.09.2011, 16:08:20

[shpaque]

no tak tyle że kiedy zmienna $category przybiera rozne wartości - wszystkie zmienne w klamrze przybieraja okreslone wartosci - wiec tutaj dziala; problem polegal tylko na pliku $database poniewaz jesli dostawał cos innego niz konkretna sciezke w '' to wariowal...

[mortus]

A na jakiej zasadzie są tworzone te zmienne? Jeśli $cat to dwie pierwsze litery zamienione na duże, a $text to nazwa pisana z dużej litery, to możesz to zrobić tak:

[PHP] pobierz, plaintext 
if(isset($_GET['category']) && $_GET['category'] != '') {
 $dbName = $_GET['category'].'.db';
 $dbDirectory = './../db/';
 $dbPath = $dbDirectory.$dbName;
 $cat = strtoupper(substr($_GET['category'], 0, 2));
 $text = ucfirst($_GET['category']);
 // reszta skryptu
} else {
 // coś jest nie tak, więc trzeba ten błąd obsłużyć
}
[PHP] pobierz, plaintext 

A jeśli istnieją jakieś szczególne przypadki, to lepiej użyć switch():

[PHP] pobierz, plaintext 
if(isset($_GET['category']) && $_GET['category'] != '') {
 $dbName = $_GET['category'].'.db';
 $dbDirectory = './../db/';
 $dbPath = $dbDirectory.$dbName;
 switch($_GET['category']) {
  'dziwnie_nazwana_kategoria':
    $cat = 'DZI';
    $text = 'DziwnieNazwanaKategoria';
    break;
  default:
    $cat = strtoupper(substr($_GET['category'], 0, 2));
    $text = ucfirst($_GET['category']);
 }
} else {
 // coś jest nie tak, więc trzeba ten błąd obsłużyć
}
[PHP] pobierz, plaintext 

no tak tyle że kiedy zmienna $category przybiera rozne wartości - wszystkie zmienne w klamrze przybieraja okreslone wartosci - wiec tutaj dziala; problem polegal tylko na pliku $database poniewaz jesli dostawał cos innego niz konkretna sciezke w '' to wariowal...

Szczerze wątpię, że Ci to działało, chyba że popełniłeś błąd podczas przepisywania skryptu tutaj na forum. Myślę jednak, że to po prostu nie działało i dlatego zmienna $database nie była tym, czym powinna być. Zresztą co za problem zrobić tak:

[PHP] pobierz, plaintext 
$database = $dbase;
var_dump($database);
[PHP] pobierz, plaintext 

Ten post edytował mortus 14.09.2011, 16:29:47

[luck]

Ja bym to zrobił np. tak:

[PHP] pobierz, plaintext 
$cfg = array(
  'balony' => array(
    'cat' => 'BA',
    'text' => 'Balony',
    'dbase' => '../db/balony.db'
  ),
  'ceramika' => array(
    'cat' => 'CE',
    'text' => 'Ceramika',
    'dbase' => '../db/ceramika.db'
  )
);
 
if(isset($_GET['category']) && isset($cfg[$_GET['category']])) {
  $dbcfg = $cfg[$_GET['category']];
  $cat = $dbcfg['cat'];
  $text = $dbcfg['text'];
  $dbase = $dbcfg['dbase'];
} else {
  /* Nie wybrano prawidlowej kategorii: powrot na strone glowna, wyswietlenie bledu, itp... */
}
[PHP] pobierz, plaintext 

I mam nadzieję, że nie umieszczasz tego kodu w warunku:

[PHP] pobierz, plaintext 
if ($_SERVER['REQUEST_METHOD'] == 'POST')
[PHP] pobierz, plaintext 

bo oczywiście wtedy żaden z podanych tu przykładów nie zadziała.

[shpaque]

no wlasnioe rzecz w tym (zapomnialem nadmienic) ze to wszystko nie jest w warunku POST ale $database tak... Wiec moze jedyny problem tkwi w tym zeby jakos ta zmienna przerzucic do warunku post?

[mortus]

I mam nadzieję, że nie umieszczasz tego kodu w warunku:

[PHP] pobierz, plaintext 
if ($_SERVER['REQUEST_METHOD'] == 'POST')
[PHP] pobierz, plaintext 

bo oczywiście wtedy żaden z podanych tu przykładów nie zadziała.

A to niby dlaczego? Prosty test:

[PHP] plik.php - pobierz, plaintext 
<?php
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
 echo $_GET['zmienna'];
}
?>
<form action="?zmienna=sto" method="post">
 <input type="hidden" name="action" value="check" />
 <input type="submit" value="Check" />
</form>
[PHP] plik.php - pobierz, plaintext 

@shpaque Pokaż cały aktualny kod, bo nikt zgadywał nie będzie, co jest źle.

[shpaque]

Na prośbę przesyłam cały kod:

[PHP] pobierz, plaintext 
<?php
$category = $_GET['category'];
$generate = rand(0000000, 9999999);
$cat = '';
$text = '';
$dbase = '';
  if ($category == balony)
  {
  $cat = "BA";
  $text = "Balony";
  $dbase = "balony.db";
  }
  else if ($category == ceramika)
  {
  $cat = "CE";
  $text = "Ceramika";
  $dbase = "ceramika.db";
  }
  else if ($category == dekoracje)
  {
  $cat = "DE";
  $text = "Dekoracje";
  $dbase = "dekoracje.db";
  }
  else if ($category == doniczki)
  {
  $cat = "DO";
  $text = "Doniczki";
  $dbase = "doniczki.db";
  }
  else if ($category == figurki)
  {
  $cat = "FI";
  $text = "Figurki";
  $dbase = "figurki.db";
  }
  else if ($category == karnety)
  {
  $cat = "KO";
  $text = "Karnety okolicznościowe";
  $dbase = "karnety.db";
  }
  else if ($category == opakowania)
  {
  $cat = "OP";
  $text = "Opakowania prezentowe";
  $dbase = "opakowania.db";
  }
  else if ($category == slonie)
  {
  $cat = "SE";
  $text = "Słonie";
  $dbase = "slonie.db";
  }
  else if ($category == szklo)
  {
  $cat = "SKL";
  $text = "Szkło";
  $dbase = "szklo.db";
  }
  else if ($category == swiece)
  {
  $cat = "SW";
  $text = "Świece";
  $dbase = "swiece.db";
  }
  else if ($category == swieczniki)
  {
  $cat = "SWC";
  $text = "Świeczniki";
  $dbase = "swieczniki.db";
  }
  else if ($category == wazony)
  {
  $cat = "WA";
  $text = "Wazony";
  $dbase = "wazony.db";
  }
  else if ($category == znicze)
  {
  $cat = "ZN";
  $text = "Znicze";
  $dbase = "znicze.db";
  }
  else if ($category == bozenarodzenie)
  {
  $cat = "AB";
  $text = "Artykuły bożonarodzeniowe";
  $dbase = "bozenarodzenie.db";
  }
  else if ($category == wielkanoc)
  {
  $cat = "AW";
  $text = "Artykuły wielkanocne";
  $dbase = "wielkanoc.db";
  }
  else if ($category == walentynki)
  {
  $cat = "AWA";
  $text = "Artykuły walentynkowe";
  $dbase = "walentynki.db";
  }
  else if ($category == rozne)
  {
  $cat = "AS";
  $text = "Artykuły sezonowe różne";
  $dbase = "rozne.db";
  }
  else if ($category == grosze)
  {
  $cat = "AT";
  $text = "Artykuły poniżej 1,89 PLN";
  $dbase = "grosze.db";
  }
  else
  {
  $cat = "XX";
  $text = "Musisz wybrać kategorię!";
  $dbase = "";
  }
?>
<?php
 
ini_set('display_errors', 1);
error_reporting(E_ALL);
 
"Content-Type: text/plain; charset=UTF-8";
$error_message = "";
 
if ($_SERVER['REQUEST_METHOD'] == 'POST')
{
   $action = isset($_POST['action']) ? $_POST['action'] : '';
   $database = './../db/'.$dbase.'';
   $success_page = './nok.php';
 
   if (!file_exists($database))
   {
      echo 'Nie odnaleziono bazy danych!';
      exit;
   }
 
   if ($action == 'dodaj')
   {
      $newkod = $_POST['kod'];
      $newopis = $_POST['opis'];
      $newcena = $_POST['cena'];
 
      $items = file($database);
      foreach($items as $line)
      {
         list($zdjecie, $kod, $opis, $cena, $status) = explode('|', trim($line));
         if ($newkod == $kod)
         {
            $error_message = 'Wygenerowany kod już istnieje! Odśwież formularz w celu wygenerowania nowego kodu.';
            break;
         }
      }
      if (empty($error_message))
      {
         $file = fopen($database, 'a');
         fwrite($file, $newkod);
         fwrite($file, '.jpg|');
         fwrite($file, $newkod);
         fwrite($file, '|');
         fwrite($file, $newopis);
         fwrite($file, '|');
         fwrite($file, $newcena);
         fwrite($file, '|1');
         fwrite($file, "\r\n");
         fclose($file);
 
         header('Location: '.$success_page);
         exit;
      }
   }
}
?>
[PHP] pobierz, plaintext 

dodam tylko ze WSZYSTKO dziala bez zarzutu oprocz tej sciezki
@mortus - test nie zadzialal...

[mortus]

To teraz przenieś linie 124 i 125 na sam początek (linia 2). A czym w ogóle jest linia 127? Usuń ją. Co to za liczba 0000000 w rand()? Powinno być po prostu 0.

@mortus - test nie zadzialal...

To chyba korzystasz z innego PHP-a niż ja.

Ten post edytował mortus 14.09.2011, 22:50:39

[shpaque]

Dzięki za pomoc - już poradziłem sobie razem z wskazówkami z Twojej strony (zrobiłem tak, żeby formularz w postaci hidden przemycil informacje do tablicy post... i wszystko gra... Teraz pozostał mi ostatni dla mnie zupełnie niezrozumialy problem - ale to w innym temacie... Dzięki. Poniżej daję kod, ktory działa w całości

[PHP] pobierz, plaintext 
<?php
$category = $_GET['category'];
$generate = rand(0, 9999999);
$cat = '';
$text = '';
  if ($category == balony)
  {
  $cat = "BA";
  $text = "Balony";
  }
  else if ($category == ceramika)
  {
  $cat = "CE";
  $text = "Ceramika";
  }
  else if ($category == dekoracje)
  {
  $cat = "DE";
  $text = "Dekoracje";
  }
  else if ($category == doniczki)
  {
  $cat = "DO";
  $text = "Doniczki";
  }
  else if ($category == drewno)
  {
  $cat = "DR";
  $text = "Drewno";
  }
  else if ($category == figurki)
  {
  $cat = "FI";
  $text = "Figurki";
  }
  else if ($category == karnety)
  {
  $cat = "KO";
  $text = "Karnety okolicznościowe";
  }
  else if ($category == opakowania)
  {
  $cat = "OP";
  $text = "Opakowania prezentowe";
  }
  else if ($category == slonie)
  {
  $cat = "SE";
  $text = "Słonie";
  }
  else if ($category == szklo)
  {
  $cat = "SKL";
  $text = "Szkło";
  }
  else if ($category == swiece)
  {
  $cat = "SW";
  $text = "Świece";
  }
  else if ($category == swieczniki)
  {
  $cat = "SWC";
  $text = "Świeczniki";
  }
  else if ($category == wazony)
  {
  $cat = "WA";
  $text = "Wazony";
  }
  else if ($category == znicze)
  {
  $cat = "ZN";
  $text = "Znicze";
  }
  else if ($category == bozenarodzenie)
  {
  $cat = "AB";
  $text = "Artykuły bożonarodzeniowe";
  }
  else if ($category == wielkanoc)
  {
  $cat = "AW";
  $text = "Artykuły wielkanocne";
  }
  else if ($category == walentynki)
  {
  $cat = "AWA";
  $text = "Artykuły walentynkowe";
  }
  else if ($category == rozne)
  {
  $cat = "AS";
  $text = "Artykuły sezonowe różne";
  }
  else if ($category == grosze)
  {
  $cat = "AT";
  $text = "Artykuły poniżej 1,89 PLN";
  }
  else
  {
  $cat = "XX";
  $text = "Musisz wybrać kategorię!";
  }
?>
<?php
 
$error_message = "";
 
if ($_SERVER['REQUEST_METHOD'] == 'POST')
{
   $action = isset($_POST['action']) ? $_POST['action'] : '';
   $kategoria = $_POST['kategoria'];
   $dbasename = $_POST['dbasename'];
   $database = './../db/'.$dbasename.'';
   $success_page = './nok.php';
 
   if (!file_exists($database))
   {
      echo 'Nie odnaleziono bazy danych!';
      exit;
   }
 
   if ($action == 'dodaj')
   {
      $newkod = $_POST['kod'];
      $newopis = $_POST['opis'];
      $newcena = $_POST['cena'];
 
      $items = file($database);
      foreach($items as $line)
      {
         list($zdjecie, $kod, $opis, $cena, $status) = explode('|', trim($line));
         if ($newkod == $kod)
         {
            $error_message = 'Wygenerowany kod już istnieje! Odśwież formularz w celu wygenerowania nowego kodu.';
            break;
         }
      }
      if (empty($error_message))
      {
         $file = fopen($database, 'a');
         fwrite($file, $newkod);
         fwrite($file, '.jpg|');
         fwrite($file, $newkod);
         fwrite($file, '|');
         fwrite($file, $newopis);
         fwrite($file, '|');
         fwrite($file, $newcena);
         fwrite($file, '|1');
         fwrite($file, "\r\n");
         fclose($file);
         header('Location: '.$success_page);
         exit;
      }
   }
}
?>
[PHP] pobierz, plaintext 

a pozniej w formularzu takie wpisy:

[PHP] pobierz, plaintext 
<form name="dodaj" method="post" action="<?php echo basename(__FILE__); ?>" enctype="multipart/form-data" accept-charset="UTF-8" id="nowyForm1">
<input type="hidden" name="action" value="dodaj"/>
<input type="hidden" name="dbasename" value="<?php echo $category; echo '.db'; ?>"/>
<input type="hidden" name="kategoria" value="<?php echo $text; ?>"/>
[PHP] pobierz, plaintext 

i wszystko gra od strony wpisu w baze danych... Dzieki

[mortus]

Tyle, że teraz linie od 1 do 106 włącznie nie są Ci potrzebne.

edit:
Stosuj konkatenacje:

[PHP] pobierz, plaintext 
<input type="hidden" name="dbasename" value="<?php echo $category.'.db'; ?>"/>
[PHP] pobierz, plaintext 

Ten post edytował mortus 15.09.2011, 09:15:27