[HTML][PHP]Kalendarz - system rezerwacji - udoskonalenie, Kalendarz z rezerwacjami oparty o tabelę - za dużo iteracji. Opcje

[maxslawek]

Witam,
Skończyłem robić właśnie mini aplikację internetową, w której w łatwy i przyjemny sposób administrator może dodawać/edytować i usuwać rezerwacje.
Stworzyłem również system wyświetlania rezerwacji - wybieramy rok i interesujący nas miesiąc i pobieramy wszystkie rezerwacje pasujące do zapytania.

Wszystko niestety oparłem o tabelę i wyszło dość sporo zagnieżdżeń tj. :

[PHP] pobierz, plaintext 
<table class="kalendarz">
		<thead>
			<tr>
				<th>Pokój</th>
				<th>Max. Os.</th>
				<?php 
				$number = cal_days_in_month(CAL_GREGORIAN, $miesiac, $rok);
				$x = 0;
				do{
 
					$x = $x+1;
 
					echo '<th>'.$x.'</th>';
				}
				while($x<$number)
				?>
			</tr>
		</thead>
		<tbody>
			<?php foreach ($pokoje as $pokoje_spec):?>
// TR dla każdego pokoju
			<tr>
				<td><?php if(isset($pokoje_spec['nazwa_pokoje']) and $pokoje_spec['nazwa_pokoje'] != NULL) {echo $pokoje_spec['nazwa_pokoje'];};?></td>
				<td ><?php if(isset($pokoje_spec['ilosc_osob_pokoje']) and $pokoje_spec['ilosc_osob_pokoje'] != NULL) {echo $pokoje_spec['ilosc_osob_pokoje'];};?></td>
				<?php 
				$number = cal_days_in_month(CAL_GREGORIAN, $miesiac, $rok);
				$x = 0;
				do{
// TD dla każdego dnia miesiąca
					$x = $x+1;
					$data_text = $rok.'-'.$miesiac.'-'.$x;
					$aktualna_data = date("Y-m-d", strtotime($data_text));
 
					echo '<td>';
					foreach ($rezerwacje as $rezerwacje_spec) {
						if($rezerwacje_spec['id_pokoje'] == $pokoje_spec['id_pokoje']){ 
// Foreach sprawdzający czy dana rezerwacja pasuje do założeń - najpierw jeden IF z samym pokojem, a potem 3 IF mniej ważnych (dla daty początkowej, końcowej i w trakcie rezerwacji)
						}
 
					}
					echo '</td>';
				}
				while($x<$number)
				?>
			</tr>
			<?php endforeach; ?>
 
			<tr>
				<td colspan="2"><a class="admin_section_link" title="Anuluj zmiany" href="<?php echo base_url()?>admin_kalendarz">Powrót</a></td>
			</tr>
		</tbody>
	</table>
[PHP] pobierz, plaintext 

I teraz pytanie - można to jakoś uprościć by tych iteracji było znacznie mniej?

Pokoi jest 50. Dni w miesiącu mamy średnio 30. Rezerwacji niech będzie ~400 na miesiąc co daje ok. 600 000 operacji ;(
Foreach dla pokoi - potem do/while dla każdego dnia miesiąca no i potem przy każdym dniu sprawdzenie każdego zamówienia czy pasuje do założeń (dzień i pokój).
Jak na razie wszystko działa, ale chciałbym to rozwiązać nieco bardziej "profesjonalnie"

Powód edycji: [Kshyhoo]: bb-code

[JakubBab]

ale Panie namieszales

Sprobuj cos w tym stylu (ponizszy kod nie bedzie dzialac bo nie znam Twoich tablic )

[PHP] pobierz, plaintext 
 
 
$number = cal_days_in_month(CAL_GREGORIAN, $miesiac, $rok);
 
 
$table = '<table class ="kalendarz"><tr><th>Pokój</th><th>Max. Os.</th>';
for ($i=0;$i<$number;$i++){
 
 
	$table.= '<th>'.$i.'</th>';
 
 
 
}
 
$table .= '</tr>'<td>;
		//nie wiem jak wyglada tablica pokoje jesli ma w sobie dwie tabele to 
 
foreach($pokoje as $pokojeKey => $pokojeVal){
 
	if($pokojeVal != ''){
		$table.='<td>'.$pokojeVa.'<td>';
	}
 
}
 
$table .='</td>';
[PHP] pobierz, plaintext 

Bez sensu jest podawanie zalozen w if a nie kodu. CIezko pomagac kiedy musimy sie domyslac-> popraw

[PHP] pobierz, plaintext 
foreach ($rezerwacje as $rezerwacje_spec) {
						if($rezerwacje_spec['id_pokoje'] == $pokoje_spec['id_pokoje']){ 
// Foreach sprawdzający czy dana rezerwacja pasuje do założeń - najpierw jeden IF z samym pokojem, a potem 3 IF mniej ważnych (dla daty początkowej, końcowej i w trakcie rezerwacji)
						}
 
					}
[PHP] pobierz, plaintext 

<-- wynik tego foreach-a wstaw w zmienna w pierwszej petli ( *for* bo i tu i to uzywasz tej samej zmiennej $x do iteracji wiec po co 2 razy to samo. i w odpowiednim momencie (podejrzewam, ze na koncu doklej do reszty.

Ten post edytował JakubBab 2.06.2016, 21:27:39

[maxslawek]

Już tłumaczę.
Pokoje:
Id_Pokoje
Nazwa_pokoje
cena_pokoje
opis_pokoje
zdjecie_pokoje

Muszą wyświetlać się wszystkie pokoje - nawet te nie zarezerwowane.

Co do if wewnątrz sprawdzenia rezerwacji:

Kod

                 foreach ($rezerwacje as $rezerwacje_spec) {
                        if($rezerwacje_spec['id_pokoje'] == $pokoje_spec['id_pokoje']){  // Jeżeli jest rezerwacja dla danego pokoju

                            if ($rezerwacje_spec['data_poczatkowa_rezerwacje'] == $aktualna_data){ // Jeżeli data poczatkowa rezerwacji jest taka jak aktualna data przypisana do komórki w tabeli tzn. ,że to pierwszy dzień rezerwacji
                                echo '<div class="rezerwacje_min_start" title="'.$rezerwacje_spec['id_rezerwacje'].'"><a href="'.base_url().'admin_rezerwacje/rezerwacje_edit/'.$rezerwacje_spec['id_rezerwacje'].'"></a></div>';
                            }

                            elseif(($rezerwacje_spec['data_poczatkowa_rezerwacje'] < $aktualna_data) and
                                ($rezerwacje_spec['data_koncowa_rezerwacje'] > $aktualna_data)){  // Jeżeli data jest pomiędzy data poczatkowa i koncowa rezerwacji
                            echo '<div class="rezerwacje" title="'.$rezerwacje_spec['id_rezerwacje'].'"><a href="'.base_url().'admin_rezerwacje/rezerwacje_edit/'.$rezerwacje_spec['id_rezerwacje'].'"></a></div>';
                            }
                            elseif ($rezerwacje_spec['data_koncowa_rezerwacje'] == $aktualna_data){   // Jeżeli data koncowa rezerwacji jest taka jak aktualna data przypisana do komórki w tabeli tzn. ,że to ostatni dzień rezerwacji
                                echo '<div class="rezerwacje_min_end" title="'.$rezerwacje_spec['id_rezerwacje'].'"><a href="'.base_url().'admin_rezerwacje/rezerwacje_edit/'.$rezerwacje_spec['id_rezerwacje'].'"></a></div>';
                            }
                        }
                        
                    }

W skrócie chodzi o specyficzne oznaczenie rezerwacji tj pierwszy i ostatni dzień w siatce zajmuje tylko pół komórki a środkowe dni całą komórkę <td> wraz z przyciskiem umożliwiającym szybkie przejscie do edycji danej rezerwacji

Chodzi mi przede wszystkim czy można jakoś zminimalizować ilość zagnieżdżeń foreach/do while, bo nic nie przychodzi mi do głowy

[JakubBab]

W skrócie chodzi o specyficzne oznaczenie rezerwacji tj pierwszy i ostatni dzień w siatce zajmuje tylko pół komórki a środkowe dni całą komórkę <td> wraz z przyciskiem umożliwiającym szybkie przejscie do edycji danej rezerwacji wink.gif

Jesli dobrze rozumiem, to chodzi o formatowanie stylu <td>? jesli tak to skorzystaj z : http://www.w3schools.com/html/html_tables.asp . Jesli natomiast chcsze formatowac ktoras z komorek to wtedt ustaw $counter =0 przed foreach a nastepnie w foreach $counter++ i warunek :

[PHP] pobierz, plaintext 
if($counter == ($x -< ?ktora komorka?) {
$table.= '<td " zajebisty kod css">'.$zmienna.'</td>';
}else{
$table.='<td>'.zmienna.'</td>';
}
 
[PHP] pobierz, plaintext 

Co do zagniezdzen uzyj tego co napisalem powyzej i bedzie git.

Co do ostatniego postu to tutaj zrobiles ten sam warunek co w peirwszej petli ( x< $number) tak wiec, mozesz skorzystac z jego dobrodziejstw i tam go wstawic tylko, ze w innej zmiennej:

[PHP] pobierz, plaintext 
  foreach ($rezerwacje as $rezerwacje_spec) {
                        if($rezerwacje_spec['id_pokoje'] == $pokoje_spec['id_pokoje']){  // Jeżeli jest rezerwacja dla danego pokoju
 
                            if ($rezerwacje_spec['data_poczatkowa_rezerwacje'] == $aktualna_data){ // Jeżeli data poczatkowa rezerwacji jest taka jak aktualna data przypisana do komórki w tabeli tzn. ,że to pierwszy dzień rezerwacji
                                $tableDiv.= '<div class="rezerwacje_min_start" title="'.$rezerwacje_spec['id_rezerwacje'].'"><a href="'.base_url().'admin_rezerwacje/rezerwacje_edit/'.$rezerwacje_spec['id_rezerwacje'].'"></a></div>';
                            }
 
[PHP] pobierz, plaintext 

itd itp. Tak wiec $tableDiv bedzie przechowywac juz sformatowane '<div>' -y gotowe zeby byly doklejone pod glowna $table.



Po skonczonym skrypcie wyswietlisz tylko raz kod bez potrzby zagniezdzania go w pierdylion linijek html-a.

Ten post edytował JakubBab 2.06.2016, 21:34:12

[maxslawek]

Dziękuję za pomoc - kilka kwestii przetestuję Co do pytania odnośnie formatowania <td> - nie, całego TD nie mogę formatować, ponieważ pierwszy i ostatni dzień danej rezerwacji nie może zajmować całego pola td - dlatego podesłałem informacje co mieści się w warunkach if(...) - w <td>, w którym jest jakaś rezerwacja występuje <div> którego klasa zależna jest od tego czy to pierwszy/ostatni czy środkowy dzień rezerwacji. Inne pola - bez rezerwacji - są czyste - tj. bez <div>.

Teraz pytanie - na ile rozwiązanie dotyczące przypisania całego kodu html do zmiennej a potem tylko wypisanie jej całej wpływa na szybkość działania całego serwisu? Można jakoś realnie określić wzrost wydajności?