Skrócenie kodu zmiennych

[Gość pziemczyk]

Cześć, w jaki sposób mógłbym to skrócić, na pewno jakoś się da a jest dość sporo rzeczy z formularza.

if ( ! $this->request['pname'] || ! $this->request['birthday'] || ! $this->request['height'] || ! $this->request['carnation'] || ! $this->request['skin'] || ! $this->request['residence'] || ! $this->request['work'] || ! $this->request['phone'] )

Z góry dzięki za pomoc

Pozdrawiam!

Edytowane 27 Listopada 2011 przez pziemczyk

[OmegaOne]

Skrócić? Nie bardzo jest co. Jeśli to jakaś walidacja danych z formularza/urla to powinieneś ja rozbudować by ci user radośnie w np. karnacji 'd*pa' nie wpisał.

[Gość pziemczyk]

to znaczy to tylko sprawdza czy każde pole jest wypełnione.

[DawPi]

No i sprawdza.. Nie rozumiem problemu.. Ew. tak:

<?php

$arrayToCheck = array( 'pname', 'birthday', 'height', 'carnation', 'skin', 'residence', 'work', 'phone' );

foreach( $arrayToCheck as $checkMe )
{
	if( ! $this->request[ $checkMe ] )
	{
		##show error
		break;			
	}
}

[Gość pziemczyk]

Dzięki DawPi, jak zwykle ratujesz

Przy okazji zapytam, żeby już nie tworzyć nowego tematu, bo podobna sytuacja.

W jaki sposób mógłbym podmienić dużą ilość cyfr na dany string? Bo w przypadku gdy mam 200 różnych odpowiedników switch opada, bo to aż 200 linijek

Przykładowo

switch ( $row['v_model'] )
{
   case 400: "Landstalker"; break;
   case 401: "Bravura"; break;
   case 402: "Buffalo"; break;
   ...
}

Z góry dzięki za pomoc.

[Gość pziemczyk]

Eh, znowu nie mam możliwości edytowania poprzedniego posta. Otóż coś tam wyskrobałem jednak to niestety nie działa.

http://forum.invisio...in/1m2pojuww50/

A błąd jaki otrzymuje to

Fatal error: Function name must be a string in /var/www/board/admin/applications_addon/other/lspd/modules_public/criminals/vehicles.php on line 107

// Edit:

Doczytałem, że niestety można tylko albo string, albo integer. Czy ktoś wie czy mogę to jakoś łatwo zamienić na inną funkcję z którą będzie działać?

Edytowane 29 Listopada 2011 przez pziemczyk

[Pavulon]

for ($i = 400; $i <= 611; $i++)
{
$vehicleName = array(
$i => "Landstalker", $i => "Bravura", $i => "Buffalo",

Co to w ogóle ma być?! Zacznij od nauki podstaw php, bo to jakieś herezje.

Ten kod można inaczej zapisać tak:

for ($i = 400; $i <= 611; $i++)
{
$vehicleName[$i] = "Landstalker";
$vehicleName[$i] = "Bravura";
$vehicleName[$i] = "Buffalo";

Widzisz sens tego? Bo ja nie.

$row['v_model_name'] = $vehicleName( $row['v_model'] );

Na prawdę, poucz się co to jest i jak się obchodzi z tablicami o_O

$row['v_model_name'] = $vehicleName[ $row['v_model'] ];

[OmegaOne]

Jak chcesz się bawić z tablica z kluczem od 400 to możesz zrobić coś takiego bez pętli.

$vehicles = array(400 => 'test400', 'test401', 'test402');

a potem

$wybrana_furka = (array_key_exists($szukany_key, $vechicles)) ? $vechicles[ $szukany_key ]  : 'zonk';

Oftop: propozycja nowego działu 'żłobek php';

Edytowane 29 Listopada 2011 przez OmegaOne