[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]

Bo podajesz jeden parametr, a powinieneś trzy podać (IMG:style_emoticons/default/tongue.gif)
Rozbij ten string : "231,236,107" explodem, po przecinku i podaj 3 argumenty.

[Kshyhoo]

Czegoś tu nie łapię:

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

co daje:

Array
(
[0] => 124
[1] => 255
[2] => 11
)

a wynik:

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

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

[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

[Kshyhoo]

Czyli powinienem zrobić tak:

[PHP] pobierz, plaintext 
function rgb2hex($rgb) {
  return sprintf("#%02X%02X%02X", $rgb[0],$rgb[1],$rgb[2]);
}
 
$dane = file("plik.txt");
 
for( $w = 0; $w < count($dane); $w++ ) {
  rgb2hex($dane[$w]);
  echo $dane[$w].'<br />';;
}
[PHP] pobierz, plaintext 

[peter13135]

Za co Ty dostałeś te 118 pochwał ? (IMG:style_emoticons/default/biggrin.gif)
Nie tak.

Ta funkcja zgodnie z tym co pisałem, przyjmuje jako parametr 3 elementową tablicę. Ty podajesz string.

Więc robisz albo tak :

[PHP] pobierz, plaintext 
function rgb2hex($rgb) {
 
  return sprintf("#%02X%02X%02X", $rgb[0],$rgb[1],$rgb[2]);
 
}
 
 
 
$dane = file("plik.txt");
 
 
 
for( $w = 0; $w < count($dane); $w++ ) {
 
  rgb2hex(explode(', ', $dane[$w]));
 
}
[PHP] pobierz, plaintext 

albo tak :

[PHP] pobierz, plaintext 
function rgb2hex($rgb) {
 
  $rgb = explode(', ', $rgb);
  return sprintf("#%02X%02X%02X", $rgb[0], $rgb[1], $rgb[2]);
 
}
 
 
 
$dane = file("plik.txt");
 
 
 
for( $w = 0; $w < count($dane); $w++ ) {
 
  rgb2hex($dane[$w]);
 
}
 
[PHP] pobierz, plaintext 

Ten post edytował peter13135 10.03.2012, 13:47:58

[Kshyhoo]

Jestem amatorem i nigdy nie robiłem nic na plikach. Po za tym, mam zapalenie ścięgna i żre prochy- nie myślę logicznie. A za co "+"? Zobacz sam (IMG:style_emoticons/default/wink.gif)
Dzięki za pomoc. Kombinowałem już wczoraj z explode, ale mi nie szło.