[PHP] Zamiana wartości w pętli Opcje

[Kshyhoo]

Coś ciężko mi się myśli. Chcę pobrać dane z pliku, poddać je konwersji i ponownie zapisać do pliku (ew. wyświetlić). Pojedyncza linia działa, ale jak kombinuję w pętli, to ma komunikat:

Warning: Missing argument 2 for rgb2hex(), called in C:\AppServ\www\...\hexToRGB.php on line 29 and defined in C:\AppServ\www\...\hexToRGB.php on line 16

Funkcja:

[PHP] pobierz, plaintext 
function rgb2hex($r, $g, $b){
  return sprintf("#%02X%02X%02X", $r, $g, $b);
}
[PHP] pobierz, plaintext 

Ja wywołam tak:

[PHP] pobierz, plaintext 
$hex = rgb2hex(231,236,107);
[PHP] pobierz, plaintext 

Działa, a jak polecę pętlą:

[PHP] pobierz, plaintext 
$dane = file("plik.txt");
for( $w = 0; $w < count($dane); $w++ ) {
	rgb2hex($dane[$w]);
}
[PHP] pobierz, plaintext 

To wywala błąd, choć (moim zdaniem) dane są ok:

Array
(
[0] => 231,236,107

[1] => 231,191,167

[2] => 233,161,130

[3] => 98,190,174

[4] => 194,169,185

[5] => 135,186,183

[6] => 153,164,200
)

[peter13135]

Znowu podajesz jeden argument :/

Jak wywołujesz tak:

[PHP] pobierz, plaintext 
$hex = rgb2hex(231,236,107);
[PHP] pobierz, plaintext 

to dajesz 3 argumenty.
Jak wywołujez tak :

[PHP] pobierz, plaintext 
$dane = array();
$dane = "124, 255, 11";
$dane = explode(", ", $dane);
rgb2hex($dane);
[PHP] pobierz, plaintext 

To dajesz tylko 1 argument, bo powyższy kod jest równoważny z takim :

[PHP] pobierz, plaintext 
rgb2hex(array('124', '255', '11'));
[PHP] pobierz, plaintext 

Czyli podajesz w pierwszym argumencie tablicę 3 elementową.
A funkcja przyjmuje 3 parametry.
Jakbyś zrobił tak (na początku tak kombinowałeś):

[PHP] pobierz, plaintext 
rgb2hex("124, 255, 11");
[PHP] pobierz, plaintext 

to znowu masz jeden parametr.
"124, 255, 11" - to jest jedna "zmienna" (raczej stała.. no ale mniejsza z tym). A nie 3 wartości, oddzielone przecinkami.

Musisz zrobić tak :

[PHP] pobierz, plaintext 
$dane = array();
$dane = "124, 255, 11";
$dane = explode(", ", $dane);
rgb2hex($dane[0],$dane[1],$dane[2]);
[PHP] pobierz, plaintext 

Lub przerobić funkcję, tak żeby przyjmowała jeden parametr jako tablicę 3 elementową :

[PHP] pobierz, plaintext 
function rgb2hex($rgb){
 
  return sprintf("#%02X%02X%02X", $rgb[0],$rgb[1],$rgb[2],);
 
}
[PHP] pobierz, plaintext 

Ten post edytował peter13135 10.03.2012, 13:31:19