PHP: Chyba s odkazem typu index.php?id=1..

- takto udělané include stránek je naprosto nevhodné
- slyšel jsi o příkazu SWITCH?
- vypiš si echem, zda má proměnná $id přiřazenou správnou hodnotu?

A teď koukám, že bys ty čísla neměl mít asi v uvozovkách, protože to není řetězec. Zkus takto:

if ($id == 1) {

Ono vůbec se to dá strašně zjednodušit a zpřehlednit:

$id = $_GET["id"];
if($id >= 2 && $id <= 6)
{
include "stranka".($id-1).".php";
}
elseif($id == 1)
{
include "forum/index.php";
}
else
{
include "index.php";
}

A switch už prý hodně programátorů zavrhuje, switch je prý nepřehledný.

V uvozovkách chyba není, PHP je na takovéto prasárny jako stavěné .

A switch už prý hodně programátorů zavrhuje, switch je prý nepřehledný.

Ak niekto zavrhuje switch, tak potom nie je programátor.

http://pc.poradna.net/q/view/432882-jaky-program-j azyk-pro-zacatecnika?page=r432958#r432958
Odtud to mám. Tak co je "správně"?

Správne je použiť to, čo je pre daný prípad vhodnejšie. V tomto prípade porovnávame konštrukciu if..elseif vs. switch..case a samotný switch..case je prehľadnejší ako if..elseif. To nemá s OOP, na ktorý narážal odkazovaný komentár, nič spoločné.

V tom komentári, na ktorý odkazuješ, sa píše, že "switch-case se novodobí programátoři snaží potírat, seč mohou", čo je samozrejme blbosť. To, o čo sa snažia v OOP, je vyčleniť logiku pre rôzne prípady do tried a nemať celé správanie definované v jednej monolitickej metóde pomocou rozhodovania vo funkcii (či už cez if..else alebo switch..case).

OK, díky .

Ach jo, v OOP je switch považován za code smell a v 95% procentech případů se dá nahradit polymorfismem...
Možná by sis mohl zadat do google switch code smell, pokochat se vysledky a třeba se i nečemu přiučit...

EDIT: Tady je hezké video... blog dokonce ukazuje i proč je dobré mit napsané na svůj kód unit testy.

Skús nahradiť switch..case, o ktorom sa bavíme teraz, polymorfizmom. Buď sme sa trafili do tých 5 % (nie), alebo si si tých 95 % vycucal z prsta (áno).

Hezké video ukazuje prípad, ktorý si zaslúžil refaktoring, ale príčinou nebol switch..case ako taký, ale to, že logika pre rôzne prípady bola definovaná v jednej monolitickej metóde. Code smell je len symptóm problému, to ale neznamená, že tam nejaký problém je. Javisti majú ale s enumom nejaký mentálny problém, keďže im aj celkom dlho trvalo (8 rokov?), kým prišli na to, že by mali enum vôbec do toho jazyka zapracovať.

Napr. chlapík vo videu na záver upravil factory metódu zo switch..case na HashMap so slovami, že to je more object-oriented way. Pritom do slovníka ukladal inštancie nových objektov, ktoré vytváral zbytočne a v prípade, že by niektorý z nich bol časovo náročnejší na vytvorenie, tak by sa mu to odrazilo na výkone. Pritom switch..case týmto neduhom netrpel, takže toto konkrétne by som skôr nazval more stupid way. Môžeš síce povedať, že to nie je podstatné (dalo by sa pridať lazy loading a neviem čo ešte), ale takisto aj ja môžem povedať, že celý ten refaktoring bol len stratou času (príliš malý problém, pre ktorý sa našlo "elegantné" a komplexné riešenie, Java way).

Ja se právě vůbec nebavím o tomto problému, jen jsem reagoval na to

Ak niekto zavrhuje switch, tak potom nie je programátor.

protože spíš opak je pravdou...

Myslím, že mentální problém s enumem mají spíš ccékřížkaři, protože si ty enumy sice zavedli hned, ale dost divné. Protože nemůžou mít metody, takže jsou to asi nějaké polovíční objekty nebo co...

Mně řešení komplexní nepřišlo. Zlepšila se čitelnost kódu a jeho udržovatelnost. A navíc, takových příkladu ti vygooglím milion i pro cékřížek...

Tvoje tvrdenie, že opak je pravdou, nie je ničím podložené a spadá do kategórie tvrdení "v 95% procentech případů se dá (switch..case) nahradit polymorfismem". To, že si strávil príliš veľa času s Javou, sa výrazne podpísalo na tvojom spôsobe uvažovania. Vygoogli si radšej aký je rozdiel v enumoch v Jave a ostatných programovacích jazykoch a skús sa na to pozrieť s nadhľadom, pretože v tomto je výnimkou práve Java - v Jave sú to také "polovičné objekty".

Nehovorím, že sa nezlepšila čitateľnosť kódu, ale to riešenie JE komplexnejšie. Udržiavateľnosť sa v tomto konkrétnom prípade príliš nezmenila, zlepšila sa skôr modulárnosť a tým pádom aj rozširovateľnosť. A zavedením HashMapy namiesto switch..case sa k lepšiemu nezmenilo teda vôbec nič.

Príklady mi googliť naozaj nemusíš, takéto základy sa učia v škole. Pokiaľ si si nevšimol, tak dosť jasne rozlišujem medzi prípadom switch..case vs. if..else a prípadom switch..case/if..else vs. polymorfizmus. Tvrdíš, že switch..case je horší ako if..else a že sa dá takmer vždy nahradiť polymorfizmom - ale to je totálny nonsens.

Dám příklad (nekamenujte mě jsem samouk):

switch($_GET['where']){
case "galerie":
write_galerie();
break;
case "clanky":
write_clanky();
break;
case "kalendar":
write_kalendar();
break;
case "lednice":
write_lednice();
break;
case "myčka":
write_mycka();
break;
default:
write_menu();
}

Jak vyhodíte switch ?

Tvrdíš, že switch..case je horší ako if..else a že sa dá takmer vždy nahradiť polymorfizmom - ale to je totálny nonsens.

Takovou hovadinu, jsem nikdy nenapsal. Switch je urcitě lepší jako horda ifů. Tvrdil jsem jenom, že v OOP je lepší použít polymorfizmus místo switche (pokud to jde), kvůli čitelnosti kódu. Nic víc...

Takto to dáva zmysel a o tomto sa nikto neháda. Ale keď v kontexte if..else vs switch..case odpovieš na "Ak niekto zavrhuje switch, tak potom nie je programátor.", že spíš opak je pravdou, tak z toho priamo vyplýva, že by si namiesto switch..case použil polymorfizmus alebo if..else alebo niečo iné. A to už zmysel nedávalo.

Spíš zopakuji co psal host.
Zkus si vypsat tu proměnou id, poté zkus jen takovým způsobem:

$id=intval($_GET["id"]);
if ($id==1){include("./forum/index.php");}
if ($id==2){include("./stranka1.php");}
...

host:
-osobně switch nepoužívám, na větších projektech includuju stránky složitěji (např.30+stránek) s využitím pole
-sice proměnná id má být INT (číslo), ale i když tam dá uvozovky, tak to nevadí. Vadilo by to při porovnávání a přičítání. Ale určitě je to mít hezčí jak to má být
edit// Nebo jak psal marek to půjde ještě líp

tak sem to prepsal pomoci array a case

if (in_array($_GET['id'], array( '1', '2', '3', '4','5','6', '7'))) {
    $id = '1';
  } else {
    $id = $_GET['id'];
  }
  
  switch ($id)
  {
    case '2':
      include "stranka1.php";
      break;
    case '3':
      include "stranka2.php";
      break;
    case '4':
      include "stranka3.php";
      break;
    case '5':
      include "stranka4.php";
      break;
    case '6':
      include "stranka5.php";
      break;
    case '7':
      include "stranka6.php";
      break;
    case '1':
    default:
      include "index.php";
      break;      
    }

ale co je problem je ze kdyz dam echo $id tak to napise ze id neni definovana
psalo to i v predchozi verzi

Tá podmienka nie je dobre - v prípade, že hodnota $_GET["id"] nadobúda hodnotu 1 až 7, priradíš do premennej $id hodnotu "1"; v opačnom prípade tam priradíš hodnotu z $_GET["id"]. Malo by to byť naopak.

Návrh :

$stranky=array() ;

$stranky[1]="./forum/index.php" ;
$stranky[2]="./stranka1.php" ;
// atd...

include($stranky[$id]) ;

Pole bych nadefinoval někde v dalším php a svázal s odkazy v divech .
Udržba a aktualizace se zjednoduší.

Ještě lépe:

$id=intval($_GET["id"]);
$stranky=array("./forum/index.php","./stranka1.php"); // atd
include($stranky[$id]);

To je samozřejmě taky dobře. Ale co kdyby byly odkazy - $stranky definovány jen pro $id=7,15,36 .
Otázka vkusu i situace .

Tak to vypada ze mam asi neco s nastavenim php, protoze at to zapisu jak chcu tak mi echo napise Notice: Undefined variable: id

To testuješ na localhostu? Kam jsi ten příkaz echo přesně umístil?

echo sem strcil do div na te strance a testuju to na localhostu ale se zaplym wamp2 kde ostatni php skripty jdou, jestli tohle muze byt problem tak to zkusim dat na server

To echo $id dej před switch ($id), resp. za $_GET["id"], podle toho, jak to máš udělané.
Taky můžeš zkusit do proměnné id přiřadit nějakou hodnotu natvrdo, jestli to bude vypisovat a fungovat.

Dekuji za trpelivot ale asi muj pocitac je blbej nebo ja, protoze, kdyz tak zadam pevnou hodnotu treba 2 ta to echo spravne zobrazi ale jinak ne

<?php
$id=intval($_GET["id"]);
echo $id;
$stranky=array("./index.php","./forum/index.php","./stranka1.php","./stranka2.php","./stranka3.php","./stranka4.php");
include($stranky[$id]);
?>

Ve výchozím nastavení WAMP2 by to mělo určitě fungovat vše správně.

Chybová hlášení máš zapnutá (E_ALL)?
error_reporting

Ano jsou zaple error_reporting = E_ALL

Tak sem presunul include toho php skriptu primo pod odkaz a vypada to ze odkazy(stihl sem kliknout jen na 1)funguje ale objevil se novy problem a to ze se zacne nacitat nekolik index.php a delaji vstiekv.png

Zkus dát :

include($stranky[$_GET["id"]); 

Přečetl jsem tohleto :

http://www.issociate.de/board/post/444500/Undefine d_variable_passing_vars_via_URL_locally.html

Diky ale bohuzel to nefunguje mam register_globlas = on zkousel sem aji off ale neslo to. Zkusim to vyzkouset nekde jinde nez localhost.

Takze ted sem to testoval mimo localhost a dostal jsem tohle:
Warning: include() [function.include]: Failed opening "for inclusion (include_path='/usr/share/php/smarty/libs:/usr/sha re/fpdf/usr/share/php:.')
podstatny asi je ze to nejak nezahrnuje ten script
Rozhodl jsem se ze odkazy necham staticke ale i tak dekuju vsem za pomoc.

To znamená, že sa snažíš pripojiť pomocou príkazu include nič - t.j. vo volaní include($stranky[$_GET["id"]]) je hodnota argumentu prázdna. Skontroluj si hodnotu argumentu tým, že ju vypíšeš na výstup buď pomocou echo alebo var_dump.

Si si istý, že máš error_reporting zapnutý? Lebo konštrukcie typu $_GET["id"] by ti bez kontroly pomocou isset mali vypísať hlásenie v prípade, že taký prvok v tom poli neexistuje.

V čem mu to pomůže (odkaz jsem nečetl)? Však promenná ID je definovaná na začátku.

Popravdě řečeno jsem z toho taky janek. Ale stačí mrknout na odkaz - symptomy jsou stejné (nebo zdají se) a řešení, jaké jsem psal , tož jsem to v zoufalství zkusil, neboť pokus stojí tazatele nejvýš minutu. Pokud víc, omlouvám se.
Ostatně problém v tomto dotazu je k uzoufání , od začátku do konce se zdá, že řešení už musí přijít. A nic.

Jak uz jsem rikal zkousel jsem to i tvym zposobem require($stranky[$_GET["id"]]); ale pise to stejnou chybu. Dokonce jsem ten script primove vlozil do stranek jestli se neco zmeni a nic. Tak sem pouzil require misto include co mi to napise za chybu a pise to stale stejne co na normalnim serveru Undefined index: id, require() [function.require]: Filename cannot be empty in .../index.php on line 348, ktera je require($stranky[$_GET["id"]]);
require() [function.require]: Failed opening required '' (include_path= tady cesta on line 348
Ja rozumim co ta chyba znamena ale nechapu proc pise ze id neni definovany, kdyz je.

Ako vyzerá URL, kde ti to píše? Lebo ak v nej nie je parameter ?id=hodnota, tak index "id" v poli $_GET bude skutočne nedefinovaný.

URL vypada index.php, tak se ale omlouvam pokud je to timto, ale nevim jak to udelat aby to melo parametr, kdyz jsem jeste na nic neklikl. Dival sem se i na ostatni stranky a i ty kdyz se nacte index, tak nema zadny parametr.

Počkat, pokud na něco kliknete, jde to? Když ne, nejde? Tak to by bylo jiné, stačilo by lehce upravit kód.

Či je index poľa nastavený alebo nie, zistíš pomocou funkcie isset.

Takze skript vypada ze funguje. Ale index.php se mi porad refreshuje a ostatni stranky maji z nejakeho duvodu otevreny lightbox,ktery se nikdy nenacte. Myslel jsem si ze je to konfliktni nejak se skriptem ktery je pouze na index.php. Ten zjisti rozliseni monitoru, refreshne stranku a pomoci get ulozi hodnoty do php promenne, ale i dyz vymazu referenci tak se stranka index.php porad refreshuje
56672186.png

Za otvorený lightbox a refreshovanie bude zodpovedný nejaký JavaScript. Prinajhoršom ho môžeš v prehliadači odkrokovať a zistiť, kvôli čomu sa to tak chová. Na odladenie môžeš použiť napr. Firebug.

Len z obrázka sa bez kódu alebo živej ukážky ťažko bude dať povedať viac.

Takze sem zabalil do archivu index.php a skript,ktery ma includovat ty ostatni stranky
stranky.rar
Doufam ze to pomuze najit problem

Tam nie je žiadne refreshovanie, ale dokola sa includuje tá istá stránka. Ak niekto ide na stránku test.php, tak sa mu includuje stránka index.php, ktorá na konci includuje stránku test.php, ktorá includuje index.php a tak stále dokola.

Takže by malo stačiť vymazať include ("test.php") na konci súboru index.php.

Ten test.php by som napísal tak, že by tam bolo jedno pomocné pole so stránkami, podľa ktorého by si sa rozhodoval. Indexy toho poľa by si potom mohol upraviť na niečo rozumnejšie, takže by odkazy boli čitateľnejšie (napr. test.php?id=galerie). Mohlo by to vyzerať napr. takto:

$pages = array(
	"2" => "forum/index.php",
	"3" => "galerie.php",
	"4" => "page3.php",
	"5" => "page4.php"
);

if (isset($pages[$_GET["id"]]))
	$page = $pages[$_GET["id"]];
else
	$page = "index.php";
include $page;

To množstvo divov vyzerá v kóde strašne, ale tak snáď vieš, čo robíš.

Tak i toto mi vypisuje Undefined variable: id
Zkusil jsem udelat jednoduchou stranku jetli to pujde tam

Jmenuje se to test.php
<html>
<head></head>
<body>
<?
if (in_array($_GET['id'], array( '1', 'forum', 'galerie', 'page1','page2'))) {
    $id = $_GET['id'];
  } else {
    $id = '1';
  }
  
  switch ($id)
  {
    case 'forum':
      include "forum/index.php";
      break;
    case 'galerie':
      include "galerie.php";
      break;
    case 'page1':
      include "page1.php";
      break;
    case 'page2':
      include "page2.php";
      break;
    case '1':
    default:
      include "test.php";
      break;      
    }

?>
<b>TOTO je :
<a href="test.php?id=forum" target="_self">forum</a>
<a href="test.php?id=galerie" target="_self">galerie</a>
<a href="test.php?id=page1" target="_self">skola</a>
<a href="test.php?id=page2" target="_self">ucitele</a>
</body>
</html>

a tady to nevypise zadnou chybu ale taky to porad zustava na test.php

<html>
<head></head>
<body>
<?
$pages = array(
	"forum" => "forum/index.php",
	"galerie" => "galerie.php",
	"4" => "page3.php",
	"5" => "page4.php"
);

if (isset($pages[$_GET["id"]]))
	$page = $pages[$_GET["id"]];
else
	$page = "test.php";
include $page;
?>
<b>TOTO je :
<a href="test.php?id=forum" target="_self">forum</a>
<a href="test.php?id=galerie" target="_self">galerie</a>
<a href="test.php?id=4" target="_self">page1</a>
<a href="test.php?id=5" target="_self">page2</a>
</body>
</html>

mi na obrazovku vypise v prohlizeci
"forum/index.php", "galerie" => "galerie.php", "4" => "page3.php", "5" => "page4.php" ); if (isset($pages[$_GET["id"]])) $page = $pages[$_GET["id"]]; else $page = "test.php"; include $page; ?> TOTO je : forum galerie page1 page2

Používaš shorttagy - máš ich zapnuté?

Omlouvam se mel sem omylem napsane <? misto <?php ale po zmene se objevila zase hlaska Notice: Undefined index: id in D:\wamp\www\cssseite\test.php on line 12
a line 12 je:

if (isset($pages[$_GET["id"]]))

Myslim ze by bylo lepsi kdyby byl nekdo tak hodny a napsal neco podobneho jednoducheho-index s linkama a scriptem +2stranky treba s textem toto je stranka 1, na ktery bych se dostal z indexu, otestoval to ze mu to funguje a uploadnul to nekde a ja to zkusil otestovat, mozna je chyba nekde u me v nastaveni.

Zadarmo ?

Tak jestli udelat stranku(rekneme test.php) na ktere jsou 2 odkazy ve tvaru test.php?id=galerie a treba test.php?id=forum, k tomu script ktery podle id includuje tu urcitou stranku je tak tezky ze bys za to chtel platit tak na to radsi ani neodpovidej. Protoze ty scripty(ktery si myslim ze jsou funkcni) jsou napsany o par radku vys.

Tak to uprav ešte na:

if (isset($_GET["id"]) && isset($pages[$_GET["id"]]))