Forum PHP.pl

Witam. Mówię z góry że to są moje początki z php i ograniczam się raczej do przeróbek skryptów i dostosowaniem niż wymyślaniem od podstaw. Oto problem.

Kiedyś miałem taki skrypt w JavaScript






Skrypt ładował do IFRAME stronkę którą była wybrana z listy rozwijanej.

Teraz postanowiłem to samo zrobić z php ale żeby ładował includem więc przerobiłem pewien skrypt i zrobiłem coś takiego.

[PHP] pobierz, plaintext 
 
<?php 
 
switch ($_GET['galeria']) 
{ 
	case '': 
	include "galerie/gal10/gal10.php"; 
	break; 
 
	case 'gal10': 
	include "galerie/gal10/gal10.php"; 
	break; 
 
   . 
   . 
   . 
   . 
 
	   case 'gal2': 
	include "galerie/gal2/gal2.php"; 
	break; 
 
	case 'gal1': 
	include "galerie/gal1/gal1.php"; 
	break; 
 
}?> 
 
<FORM ACTION="zdjecia.php" METHOD="GET"> 
<SELECT NAME="galeria" SIZE="1"> 
   <OPTION  VALUE="">&nbsp;Zmień Galerię&nbsp;</OPTION> 
 
	<OPTION VALUE="gal10">pozycja10</OPTION> 
 
   . 
   . 
   .  
   . 
 
	<OPTION VALUE="gal2pozycja2</OPTION> 
	<OPTION VALUE="gal1pozycja1</OPTION> 
</SELECT> 
<INPUT TYPE="SUBMIT" VALUE="Zmień"> 
</FORM> 
[PHP] pobierz, plaintext 

Wszystko działa OK ale jest jedno ale. Muszę naciskać przycisk. Czy jest możliwość wysłania danych bez przycisku tak jak w pierwszym skrypcie? Poprostu zaznaczam i po kliknięci na liście już jest.

Czy można to uprościć? Sporo tych Case'ów a czym galerii będzie więcej tym więcej ich będzie (pierwczy pusty dodałem żeby była jakaś wartość początkowa i od razu żeby się coś ładowało domyślnie)


P.S. A tak na marginesie jak ktoś biegły w CSSie to czy można w takim formularzu zmienić wygląd listy? Chodzi mi konkretnie o wygląd rozwijanej listy i strzałeczkę która służy do tego (kolortyka XPkowa mi się gryzie ze stronką). Bo same pozycje to wiem i przycisk też się da (dodając styl w input lub opyion). Niestety jesli dodam w select jakiś styl to się nic nie zmienia.

Będę wdzęczny za odpowiedź

Normalnie jestem zachwycony php a includ'y mi rozwiąują wiele rzeczy (np jeden plik menu dołączany do każdej podstrony, podobnie nagłowek czy stopka ). A kiedyś był tylko HTML+CSS i skrypty.

wlasnie do tego automatycznego przesylania sluzyl kod JavaScript, ktory usunales. aczkolwiek sadze, ze wystarczy wstawic onchange="this.submit;".

a co do pozbywania sie case'ow i switcha, to coz, wystarczy popatrzec na to i od razu widac, takze jesli nie potrafisz zauwazyc prostych zaleznosci, to niestety nie zadobrze bedziesz sobie radzil, a "przerabianie, zamiast tworzenia", to "plagiat, zamiast programowania", wiec lepiej po prostu zacznij sie zastanawiac nad tym, co robisz, zamiast bezmyslnie kopiowac kod z jednego miejsca na drugie.

podpowiem, ze nazwe dolaczanego pliku mozna zapisac po porstu jako:

[PHP] pobierz, plaintext 
<?php
include('galerie/' . $_GET['galeria'] . '/' . $_GET['galeria'] . '.php');
?>
[PHP] pobierz, plaintext 

a wiec nie potrzeba zadnego switcha, ale musisz wtedy pamietac o bezpieczenstwie (przesylanie wartosci, ktorych nie wyswietlasz normalnie w formularzu).

Ten post edytował wrzasq 20.03.2006, 16:20:12

Usunąłem bo wrzucał w ramkę a nie chcę ramek.



Nie działa



Gdybym widział to bym nie pytał



Tak się człowiek uczy użytecznych rzeczy, poza tym php jest podobny do C++ którego MUSIAŁEM się uczyć kiedyś ale tego nie lubiłem (ma trochę inne zapisy) . Poza tym po co wyważać otwarte drzwi.



Plagiat jest gdy przepisujesz kropka w kropkę a nie gdy coś przerabiasz. Jest to już coś nowego.



Myślałem ale jeśli czegoś się nie lubi to choćby się myślało 1000 dni to się nie wymyśli. Niestety nie lubię programowania (jakby tak bylo byłbym programistą) wolę design czyli grafika + html. php ułatwia sprawę i niektórych rzeczy nie da się inaczej obejść. Może kiedyś polubię programowanie ale nienawidzę jak błąd tkwi 30 lini wyżej i nie wiadomo gdzie bo i tak coś nie działa. W grafice i HTML jak się coś skopie to od razu widać że coś krzywo czy się nie wyświetla itp.

No ale ok mniejsza z tym DZIKI WIELKIE za to spojrzenie, nie wpadłbym że można wykożystać konkatenację (choć kiedyś używałem tego do innych skryptów z hasłem czy mailem.) Jednak skrypt ma wadę bo nie reaguje na wartość domyślną. Ale wymyśliłem to

[PHP] pobierz, plaintext 
<?php
if ($_GET['galeria'] == '') {
include "galerie/gal10/gal10.php"; 
}
else {
include('galerie/' . $_GET['galeria'] . '/' . $_GET['galeria'] . '.php');
}
?>
[PHP] pobierz, plaintext 

Po czym stwierdziłem że uproszczę jeszcze i ostatecznie mam to:

[PHP] pobierz, plaintext 
<?php
if ($_GET['galeria'] == '') 
$_GET['galeria'] = 'gal10';
include('galerie/' . $_GET['galeria'] . '/' . $_GET['galeria'] . '.php');
?>
[PHP] pobierz, plaintext 

Jak widzisz myślę choć nie wiem czy wszystko jest poprawnie ale działa.



Tak wiem oryginalnie w skrypcie było POST ale zmieniłem na GET bo wtedy widać w adresie przesyłane zmienne a więc można zapisać stronę w ulubionych z prametrami (np link do konkretnej galerii)

P.S. Dzięki za szybką odpowiedź i czkem na pomysły z przyciskiem. (Chyba że sam coś wymyślę bo pokombinuję.)

Sorka za błędy. Nie jestem dysortografem (chyba) ale poprostu zbyt szybko naciskam "Wyślij"

No to sobie sam poradziłem w sumie coś tam mi podpowiedziałeś ale nie do konca było OK. Metoda prób i błędów wstawiłem coś takiego



i działa.

Dzięki za naprowadzenie i chyba EOT jak dla mnie sprawa rozwiązana

Mała uwaga

W petli switch akcje domyslna lepiej wykonac przy pomocy default zamiast case ''

[PHP] pobierz, plaintext 
<?php 
switch ($_GET['galeria']) 
{ 
 
	case 'gal2': 
	include "galerie/gal2/gal2.php"; 
	break; 
 
	case 'gal1': 
	include "galerie/gal1/gal1.php"; 
	break; 
 
	case 'gal10':
	default: 
	include "galerie/gal10/gal10.php"; 
	break; 
 
}?>
[PHP] pobierz, plaintext 

Co do przerabiania - to jesli autor nie ma nic przeciwko temu - to ja nie widzie w tym nic zdroznego. Inna rzecz, ze jak napiszesz cos sam od poczatku do konca to sie wiecej nauczysz.