Problem ze includowaniem i wykorzystaniem CASE, dużo includów, jak to zmiejszyc Opcje

[Kreton]

mam tu stronkę http://dobrepomysly.za.pl

I działy, dajmy na to dział 2.

No i tematy są inludowane w sposób :

[PHP] pobierz, plaintext 
<?php
if (!isset($_GET['dzial'])){
echo" 
<center><b>Szkoła podstawowa</b></center>
<ul><li><a href="index.php?go=szkola&dzial=1"><b>Swobodna twórczość dziecięca</b></a></li>
";
} else {
switch ($_GET['dzial'])
{
case 1:
include("1.html");
break;
case 2:
include("2.html");
break;
<!--  i tak dalej az do 40-->
 }
}
?>
[PHP] pobierz, plaintext 

Jak to zrobic zeby nierobić tych bardzo żmudnych case x: include("blabla.html")
niebardzo wiem jak to zrobic przy użyciu np. pętli while czy cos takiego. Pomóżcie

[zbig13]

Jeśli numery działów odpowiadają nazwom plików, to możesz zrobić po prostu tak:

[PHP] pobierz, plaintext 
<?php
 
$dzial = $_GET['dzial'];
include ('katalog/' . $dzial . '.html');
 
?>
[PHP] pobierz, plaintext 

[Spirit86]

[PHP] pobierz, plaintext 
<?php
 
$dzial = (int) $_GET['dzial'];
if(file_exists('katalog/' . $dzial . '.html')) include ('katalog/' . $dzial . '.html');
 
?>
[PHP] pobierz, plaintext 

może lepiej tak

Ten post edytował Spirit86 24.09.2005, 19:31:44

[M4chu]

Jeśli to nie tylko liczby, to możesz na przykład zrobic tablicę elementów odpowiadająch możliwością, i sprawdzać za pomocją in_array(). Jednak wygląda, że to raczej będą numerki, więc rozwiązanie zbig13'a będzie lepsze, jednak z jednym dodatkiem - filtrowaniem tego $_GET['dzial'].

[Spirit86]

Jeśli to nie tylko liczby, to możesz na przykład zrobic tablicę elementów odpowiadająch możliwością, i sprawdzać za pomocją in_array(). Jednak wygląda, że to raczej będą numerki, więc rozwiązanie zbig13'a będzie lepsze, jednak z jednym dodatkiem - filtrowaniem tego $_GET['dzial'].

rozwiązanie zbig13 wcale nie jest lepsze.
Zastanów się lepiej :]

[zbig13]

ja podsunąłem tylko propozycję jak można to zrobić krócej (zamiast switch) a z zabezpieczeniem tego kodu autor tematu powinien sobie poradzić

@Spirit86 - do tego twojego kodu można zamiast (int) przed zmienną sprawdzić w warunku czy zmienna $dzial jest liczbą (funkcja is_numeric" title="Zobacz w manualu php" target="_manual). Jest to o tyle lepsze, że można kontrolować komunikat wyświetlany, gdy zmienna nie będzie miała wartości liczbowej.

[Kreton]

razej zmienna zawsze będzie miała wartość liczbową, chodzi o te pliki html.
Jest tego dużo i dużo latwiej mi się połąpać. A nieznam sie superowo na php

ja podsunąłem tylko propozycję jak można to zrobić krócej (zamiast switch) a z zabezpieczeniem tego kodu autor tematu powinien sobie poradzić biggrin.gif

Niewiem co mam przez to rozumiec W jaki sposób ten kod zabezpieczyc

Ten post edytował Kreton 25.09.2005, 08:26:26

[zbig13]

Kod niezabezpieczony wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
include ($_GET['dzial'] . '.html');
 
?>
[PHP] pobierz, plaintext 

Potencjalny włamywacz może napisać cos takiego:
http://twoja.strona.pl?dzial=http://strona.hakera.pl/1

i otworzył by się plik, który haknie ci stronę.

Aby zabezpieczyć stronę, trzeba zrobić cos takiego:

[PHP] pobierz, plaintext 
<?php
 
if (is_numeric ( $_GET['dzial'] )) {
 
$dzial = (int) $_GET['dzial'];
if (file_exists('./katalog/' . $dzial . 'html') { include ('./katalog/' . $dzial . '.html'); }
 
} else {
echo 'Nieprawidłowe dane';
}
 
?>
[PHP] pobierz, plaintext 

czyli połączenie mojego kodu i kodu Spirit86

Edit:

@mike_mech Racja, to z pośpiechu Już poprawiam

Ten post edytował zbig13 25.09.2005, 16:31:47

[Ociu]

ja bym tak dał die" title="Zobacz w manualu php" target="_manual zamiast echo

[Kreton]

Mam gorącą prośbe, mogłbyś mi bardziej dogłębnie wytłumaczyc, jak mogłby hacknnąć tą stronkę W jaki sposób ten kod napisany przez ciebie chroni przed włamem.

Nie chcę robić poźniej takiego błędu
z góry THX

[mike]

Mam gorącą prośbe, mogłbyś mi bardziej dogłębnie wytłumaczyc, jak mogłby hacknnąć tą stronkę W jaki sposób ten kod napisany przez ciebie chroni przed włamem.

Chodzi o to że bez kontroli dołanczasz jakiś plik, to wcale nie musi byc plik u Ciebie na serwerze tylko u kogoś. Kotś może Ci tak dołączyć plik, w którym kasuje wszystkie foldery w bierzącej lokalizacji a to już problem, nie :?:

Sprawdzenie polego na tym czy zmienna przychodząca z $_POST'a jest liczbą i czy u Ciebie na serwerze istnieje plik, który masz zamiar dołączyć (masz pewność że dołanczasz seoje pliki).
Mam tylko dwa ale:
1. Do sprawdzania czy dana jest liczbe użyłbym fukncji z rozszerzenia ctype.
2. @zbig13 ma błąd w kodzie: Sprawdza istnienie pliku na poziomie (folderów) gdzie jest index.php a dołancza plik z folderu niżej. Oczywiste jest że nigy takiego pliku nie będzie. Trzeba sprawdzać czy plik istnieje w folderze niżej.

[Spirit86]

ja bym tak dał die" title="Zobacz w manualu php" target="_manual zamiast echo

a cóż to za różnica, jeżeli i tak udaremniliśmy próbę Code Injection. Teraz to i tak nie ma znaczenia, więc można wykonywac stronę dalej.

[Kreton]

[PHP] pobierz, plaintext 
<meta http-equiv="Content-type" content="text/html; charset=iso-8859-2" />
<?php
if (!isset($_GET['dzial'])){
echo" 
 
<center><b>Szkoła podstawowa</b></center>
<ul>
<li><a href="index.php?go=efekty&dzial=1"><b>Egzamin w klasach trzecich</b></a></li>
<li><a href="index.php?go=efekty&dzial=2"><b>Szkolny system diagnozowania.</b></a></li>
<li><a href="index.php?go=efekty&dzial=7"><b>Świetlicowy Sejmik (spotkania społeczności świetlicowej)</b></a></li>
<li><a href="index.php?go=efekty&dzial=10"><b>Kolorowy  kraj „Ćwierćlandia” – impreza w ramach obchodów Dnia Matki i Dnia Dziecka w naszej szkole</b></a></li>
</ul>
<center><b>Gimnazjum</b></center>
<ul>
<li><a href="index.php?go=efekty&dzial=3"><b>„Wartość dodana” zamiast tabel lekkoatletycznych</b></a></li>
<li><a href="index.php?go=efekty&dzial=5"><b>Zastosowanie minimum ze wszystkich przedmiotów w nauczaniu  w gimnazjum</b></a></li>
<li><a href="index.php?go=efekty&dzial=8"><b>Klasa partnerska</b></a></li>
<li><a href="index.php?go=efekty&dzial=9"><b>„Miasto uczuć” – jakie uczucia wzbudza w nas nasza szkoła?</b></a></li>
<li><a href="index.php?go=efekty&dzial=11"><b>Promowanie osiągnięć uczniów szkoły wśród Rodziców.</b></a></li>
</ul>
<center><b>Szkoła ponadgimnazjalna</b></center>
<ul>
<li><a href="index.php?go=efekty&dzial=4"><b>Zwiększanie efektywności uczenia się.</b></a></li>
<li><a href="index.php?go=efekty&dzial=6"><b>Porównywanie wyników nauczania w poszczególnych klasach.</b></a></li>
</ul>
";
} else {
if (is_numeric ( $_GET['dzial'] )) {
 
$dzial = (int) $_GET['dzial'];
if (file_exists('./katalog/' . $dzial . 'html') { include ('./katalog/' . $dzial . '.html'); }
 
} else {
echo 'Nieprawidłowe dane';
}
 
?>
[PHP] pobierz, plaintext 

i wyrzuca błąd

Kod

Parse error: syntax error, unexpected '{' in C:\Program Files\WebServ\httpd\Działy\I.Efekty\działy.php on line 31

A niemoge się tam doszukac jakiegoś błędnego nawiasu, co jest nie tak HELP

[nospor]

Na samyum koncu brakuje }. Czy to tak ciezko policzyc lkiczbe otwrtych, liczbe zmaknietych i sprobowac samemu znalexc?

[Kreton]

dla ciebie wnikliwego programisty to moze i proste, ale dla mnie.

Moze jak jakbym ci zaczoł nawalać taktyki w Warcrafcie to dla ciebie niebyło by tak oczywiste jak dla mnie.

I tak nic to niedaje ...

Ten post edytował Kreton 26.09.2005, 16:34:56

[kszychu]

dla ciebie wnikliwego programisty to moze i proste, ale dla mnie.

Moze jak jakbym ci zaczoł nawalać taktyki w Warcrafcie to dla ciebie niebyło by tak oczywiste jak dla mnie.

I tak nic to niedaje ...

Kolego, troszkę szacunku dla kogoś, kto próbuje Ci pomóc.
I zamiast tekstów "i tak nic to nie daje" lepiej powiedz, co jest jeszcze nie tak jak powinno być, czy wywala jakiś błąd?

[Spirit86]

[PHP] pobierz, plaintext 
<meta http-equiv="Content-type" content="text/html; charset=iso-8859-2" />
<?
if (!isset($_GET['dzial']))
{
	echo"
 
	<center><b>Szkoła podstawowa</b></center>
	<ul>
	<li><a href="index.php?go=efekty&dzial=1"><b>Egzamin w klasach trzecich</b></a></li>
	<li><a href="index.php?go=efekty&dzial=2"><b>Szkolny system diagnozowania.</b></a></li>
	<li><a href="index.php?go=efekty&dzial=7"><b>Świetlicowy Sejmik (spotkania społeczności świetlicowej)</b></a></li>
	<li><a href="index.php?go=efekty&dzial=10"><b>Kolorowy  kraj „Ćwierćlandia” – impreza w ramach obchodów Dnia Matki i Dnia Dziecka w naszej szkole</b></a></li>
	</ul>
	<center><b>Gimnazjum</b></center>
	<ul>
	<li><a href="index.php?go=efekty&dzial=3"><b>„Wartość dodana” zamiast tabel lekkoatletycznych</b></a></li>
	<li><a href="index.php?go=efekty&dzial=5"><b>Zastosowanie minimum ze wszystkich przedmiotów w nauczaniu  w gimnazjum</b></a></li>
	<li><a href="index.php?go=efekty&dzial=8"><b>Klasa partnerska</b></a></li>
	<li><a href="index.php?go=efekty&dzial=9"><b>„Miasto uczuć” – jakie uczucia wzbudza w nas nasza szkoła?</b></a></li>
	<li><a href="index.php?go=efekty&dzial=11"><b>Promowanie osiągnięć uczniów szkoły wśród Rodziców.</b></a></li>
	</ul>
	<center><b>Szkoła ponadgimnazjalna</b></center>
	<ul>
	<li><a href="index.php?go=efekty&dzial=4"><b>Zwiększanie efektywności uczenia się.</b></a></li>
	<li><a href="index.php?go=efekty&dzial=6"><b>Porównywanie wyników nauczania w poszczególnych klasach.</b></a></li>
	</ul>
	";
}
else
{
	if(is_numeric ( $_GET['dzial'] ))
	{
		$dzial = (int) $_GET['dzial'];
		if(file_exists('./katalog/' . $dzial . 'html'))
		{
		  include ('./katalog/' . $dzial . '.html');
		}
		else
		{
			echo 'Nieprawidłowe dane';
		}
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Spirit86 26.09.2005, 17:12:56

[Kreton]

Mam bardzo duży szacunek do was, ale wy niemacie szacunku do mnie.
Bo niejestem dobrym programistą w php.

Ok działa.
Bardzo wam dziękuję za pomoc.

Aha, i mam pytanie, jeżeli mam tn działy.php i w tym samym katalogu te pliki 1.html itd.

To jak zamienić tą komendę include bo próbuję coś tu zmieniać, i zawsze wyskakuje " nieprawidłowe dane " :/

Ten post edytował Kreton 26.09.2005, 17:52:41

[kszychu]

Usuń /katalog/ z file_exists() i .....
ZACZNIJ MYŚLEĆ!!!

[Berni]

wrzuc sobie do tablicy nazwy dostepnych blokow z odpowiadajaca im nazwa plikow, potem wyciganij za pomoca foreach i porownaj z getem jesli blok bedzie dostepny to incuduj plik .... mniejwiecej tak:

[PHP] pobierz, plaintext 
<?php
$a_block = array('blok_1'=>'blok');
 
foreach($a_block as $block_k => $block_v){
 
	if($_GET['block']==$block_k){
 
		 include_once('".$block_v.".php');	
 
	}
 
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Berni 28.09.2005, 00:17:38