[PHP]Funkcja ignoruje warunek if Opcje

[Niedowiarek]

Witam, mam taki oto skrypt na zmianę wielkości obrazka:

[PHP] pobierz, plaintext 
    function avatar_resizer($avatar_link, $avatardimensions="80|80")
    {
		$rozmiar = getsizeimage($avatar_link);
 
		if($avatar_link[0] > 80 && $avatar_link[1] > 80) {
 
 
                    $avatar_link = htmlspecialchars_uni($avatar_link);
                    $avatar_dimensions = explode("|", $avatardimensions);
                    if($avatar_dimensions[0] && $avatar_dimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatar_dimensions[0]}\" height=\"{$avatar_dimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
 
            return $avatar;
		}                                               
    }
[PHP] pobierz, plaintext 

Działał, dopóki nie dodałem instrukcji if, która sprawdza czy obrazek ma już 80x80, a jeżeli ma to przejść do końca funkcji.

Teraz wywala błąd: Parse error: syntax error, unexpected '}' in /virtual/w/o/workdomain.ugu.pl/global.php on line 547

Linijka 547 to ostatnia klamra zamykająca.

[rocktech.pl]

Witam.

Jakiego IDE używasz?

Usuń ostatnią klamrę nie będę złośliwy (IMG:style_emoticons/default/smile.gif)

[Niedowiarek]

Na to to ja też wpadłem (IMG:style_emoticons/default/haha.gif) Ale wtesy znowóż wyskakuje:

Fatal error: Call to undefined function getsizeimage() in /virtual/w/o/workdomain.ugu.pl/global.php on line 526

[mortus]

Klamer jest za dużo, ale i usunięcie tej ostatniej nie pomoże. W instrukcji warunkowej if posługujesz się zmienną $avatar_link, a powinieneś chyba użyć zmiennej $rozmiar.

A funkcja nazywa się getimagesize().

EDIT: Poza tym wydaje mi się, że obraz powinien być skalowany proporcjonalnie, bo inaczej będzie zniekształcony.

Ten post edytował mortus 5.01.2012, 16:32:11

[Niedowiarek]

Okej, zmieniłem na getimagesize() oraz $avatar_link na $rozmiar, lecz teraz wyskoczyło ostrzeżenie:

Warning [2] getimagesize(images/avatars/php.gif?dateline=1325777488) [function.getimagesize]: failed to open stream: No such file or directory - Line: 526 - File: global.php PHP 5.2.17-ugu1 (Linux)

Wyniakłoby z niego że nie ma takiego obrazku, jaki chcę mieć w avatarze, ale sprawdziałem, i jest.

Tutaj obecna funkcja:

[PHP] pobierz, plaintext 
function avatar_resizer($avatar_link, $avatardimensions="80|80")
    {
		$rozmiar = getimagesize($avatar_link);
 
		if($rozmiar[0] > 80 && $rozmiar[1] > 80) {
 
 
                    $avatar_link = htmlspecialchars_uni($avatar_link);
                    $avatar_dimensions = explode("|", $avatardimensions);
                    if($avatar_dimensions[0] && $avatar_dimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatar_dimensions[0]}\" height=\"{$avatar_dimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
 
            return $avatar;
		}                                               
 
 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

[mortus]

Funkcja jako argument $avatar_link przyjmuje adres obrazka, a nie adres pliku PHP, który ten obrazek generuje (chyba).

[Niedowiarek]

No ale w takim razie funkcja powinna wygenerować to ostrzeżenie również jak nie było żadnej modyfikacji (instrukcja if sprawdzająca wielkość obrazka), a działała. ;/

[mortus]

Ja nie mówię, że funkcja nie działa, tylko że dostaje nieprawidłowe dane na wejście. Sprawdź, co kryje zmienna $mybb->user['avatar'], powinien to być adres obrazka z odpowiednim rozszerzeniem.

Ten post edytował mortus 5.01.2012, 16:42:37

[Niedowiarek]

Okej, pozmieniałem teraz co nie co, i funkcja wygląda tak:

[PHP] pobierz, plaintext 
<?php
function avatar_resizer($avatar_link)
    {
	$rozmiar = getimagesize($avatar_link);
 
	if($rozmiar[0] > 80 || $rozmiar[1] > 80) {
 
	if($rozmiar[0] > 80) {
 
	$avatardimensions[0] = 80;
	$avatardimensions[1] = (80*$avatardimensions[1])/$avatardimensions[0];
 
	return $avatardimensions;
 
	} else if($rozmiar[1] > 80) {
 
	$avatardimensions[1] = 80;
	$avatardimensions[0] = (80*$avatardimensions[0])/$avatardimensions[1];
 
	return $avatardimensions;
	}
 
            if($avatar_link)
            {
                    $avatar_link = htmlspecialchars_uni($avatar_link);
 
                    if($avatardimensions[0] && $avatardimensions[1])
                    {
                            $avatar_width_height = "width=\"{$avatardimensions[0]}\" height=\"{$avatardimensions[1]}\"";
                    }
                    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" $avatar_width_height />";
            }
            else
            {
                    $avatar = '';
            }
 
            return $avatar;
			}
 
    }
 
 
$avatar = avatar_resizer($mybb->user['avatar']);
?>
[PHP] pobierz, plaintext 

I teraz obrazek się wyświetla, ale tak samo kiedy nie było skali, więc jeżeli bok ma mniej niż 80 to go rozciąga.

Ten post edytował Niedowiarek 5.01.2012, 23:25:00

[wookieb]

Swojego syna też nazwiesz "Błąd" bo nie rozumiesz po co nazywa się pewne rzeczy?
Temat: Jak poprawnie zatytulowac watek

[Niedowiarek]

@Refresh

[mortus]

Ale przecież w tym żadnych cudów nie ma:

[PHP] pobierz, plaintext 
function avatar_resizer($avatar_link) {
  $defaultWidth = 80;
  $defaultHeight = 80;
  list($avatarWidth, $avatarHeight) = getimagesize($avatar_link);
  if($avatarWidth > $avatarHeight) {
    $newWidth = $defaultWidth;
    $newHeight = (int)($defaultWidth * $avatarHeight / $avatarWidth);
    if($newHeight > $defaultHeight) {
      $newWidth = (int)($defaultHeight * $avatarWidth / $avatarHeight);
      $newHeight = $defaultHeight;      
    }
  } elseif($avatarHeight > $avatarWidth) {
    $newWidth = (int)($defaultHeight * $avatarWidth / $avatarHeight);
    $newHeight = $defaultHeight;
    if($newWidth > $defaultWidth) {
        $newWidth = $defaultWidth;
        $newHeight = (int)($defaultWidth * $avatarHeight / $avatarWidth);
    }
  } else {
    $newWidth = $defaultWidth;
    $newHeight = $defaultHeight;
  }
  if($avatar_link) {
    $avatar_link = htmlspecialchars_uni($avatar_link);
    $avatar = "<img src=\"{$avatar_link}\" alt=\"\" width=\"{$newWidth}\" height=\"{$newHeight}\" />";
  } else {
    $avatar = '';
  }
  return $avatar;
}
[PHP] pobierz, plaintext 

Kodu na proporcjonalne przeliczenie wysokości i szerokości obrazka w sieci jest całe mnóstwo. Wystarczy dobrze poszukać.

EDIT: Jeśli coś dalej nie będzie działać, to sprawdź, co zwraca funkcja getimagesize() - var_dump($avatarWidth, $avatarHeight) w moim kodzie lub var_dump($rozmiar) w Twoim kodzie.

Ten post edytował mortus 6.01.2012, 14:13:47

[Niedowiarek]

Hmm.. Jest tak jak powiedziałeś. Getimagesize pobiera obrazek z linku, a ja mam na dysku. Jest jakaś inna funkcja sprawdzająca wielkość obrazka?

Edit: Taki warning wyskakuje: Warning [2] getimagesize(images/avatars/php.gif?dateline=1325855477) [function.getimagesize]: failed to open stream: No such file or directory - Line: 527 - File: global.php PHP 5.2.17-ugu1 (Linux)

Ten post edytował Niedowiarek 6.01.2012, 14:21:59

[mortus]

Problemem jest przekazywanie niewłaściwej wartości do funkcji avatar_resizer(), a nie nieprawidłowe działanie funkcji getimagesize(). Coś jest skopane w innym miejscu aplikacji, tzn. w miejscu w którym pobierasz adresy url obrazków.

[Niedowiarek]

[PHP] pobierz, plaintext 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

Tutaj pobieram obrazekdo funkcji.

[mortus]

Pisałem już, abyś sprawdził co zawiera zmienna $mybb->user['avatar']: var_dump($mybb->user['avatar']);. Jeśli zawiera adres pliku PHP, to poszukaj tego pliku i sprawdź, co on takiego robi (być może odpowiada za przeskalowanie obrazka).

[Niedowiarek]

string(44) "images/avatars/mysql.png?dateline=1325856623"


To mi wyszło, znalazłem też w usercp.php taką linijkę: "avatar" => $avatar['avatar'].'?dateline='.TIME_NOW

@refresh

[mortus]

Trzeba się pozbyć tej końcówki:

[PHP] pobierz, plaintext 
$avatar = avatar_resizer(substr($mybb->user['avatar'], 0, strpos($mybb->user['avatar'], '?') - 1);
[PHP] pobierz, plaintext 

[Niedowiarek]

Tzn. zamienić to:

[PHP] pobierz, plaintext 
$avatar = avatar_resizer($mybb->user['avatar']);
[PHP] pobierz, plaintext 

na to co podałeś?

Nie mogę tego usunąć całkowicie bo do tej zmiennej się odnoszę w szablonie MyBB.