Problem mit php-Gallerie

Seite 1 von 1 - Forum: Coding Stuff auf overclockers.at

URL: https://www.overclockers.at/coding-stuff/problem_mit_php-gallerie_48720/page_1 - zur Vollversion wechseln!

grubjo schrieb am 18.09.2002 um 10:34

Hi, ich habe mit PHP eine Gallerie erstellt, doch leider gehts verdammt langsam und wenn ich mehr als 20 Bilder in die Gallerie gebe, kommt die Meldung "Fatal error: Maximum execution time of 30 seconds exceeded in /home/ima00/grubjo/public_html/homepage/gallerie.php on line 42".
Es kann sichs jeder selbst unter http://dmt.fh-joanneum.at/~grubjo/homepage anschaun.

Ich habe die beiden Files angehängt, vielleicht möchte sichs mal jemand anschaun - würde mich freuen :-)

Ciao, Hannes

Jehul schrieb am 18.09.2002 um 10:40

Code:

$image = imageCreateFromJpeg($photo);
$imagebreite = imagesX($image);
$imagehoehe = imagesY($image);

für die breite und höhe gibts a eigene funktion, die weit schneller ist!
get_image_size (oder so)

EDIT: got it http://www.php.net/manual/en/function.getimagesize.php

watchout schrieb am 18.09.2002 um 17:45

rofl - das switch in der resizeimage.php is' ein bisserl confus :rolleyes:
generell: wenn ein string nicht geparsed werden muss (keine variablen oder \n drinnen) dann verwende single-quotes (') statt double-quotes (")

edit: ups, vertan was verwendest du auch so komische 'pseudo-objektorientierte' funktionen....

watchout schrieb am 18.09.2002 um 18:19

also:
1. wenn du möchtest dass noch mal jemand code von dir liest/verbessert/dir hilft dann solltest du diesen in eine angebrachte form bringen! dh. keinen infernalischen mix aus tabs und blanks machen, sondern bei einer form bleiben (idealerweise sind das tabs weil die kann man einstellen)
2. verwendest du anscheinend register_globals, was nicht nur auf den meisten webservern deaktiviert ist, sondern auch unlesbar ist, und somit unter punkt 1 fällt

grubjo schrieb am 18.09.2002 um 18:49

hi, den mix aus tabs und blanks macht glaube ich Dreamweaver, da ich eigentlich nur mit 2 blanks einrücke.
Was du unter 2. meinst weis ich nicht genau, bin eher ein php-Anfänger bzw. jsp-Umsteiger.

Ciao, Hannes

funka schrieb am 18.09.2002 um 18:50

...

bilder resizen ist eine langsame gschicht
du kannst optimieren
aber im endeffekt zahlt sichs mehr aus wennst die set_time_limit() verwendest und die max exec time hoeher setzt....

ich wuerd an deiner stelle alle bilder mal resizen
und dann die resizten speichern...
kannst auch so machen dast im resizeimage.php checkst obs das resizte bild schon gibt - falls ja
header(..);
fpassthru ( $bild );
falls nein
resizen
speichern

grubjo schrieb am 18.09.2002 um 19:51

----
für die breite und höhe gibts a eigene funktion, die weit schneller ist!
get_image_size (oder so)
----

DANKE FÜR DEN TIP!!!! Jetzt gehts viel schneller und auch die Fehlermeldung ist weg :-))

Jehul schrieb am 19.09.2002 um 13:34

Zitat von grubjo
----
für die breite und höhe gibts a eigene funktion, die weit schneller ist!
get_image_size (oder so)
----

DANKE FÜR DEN TIP!!!! Jetzt gehts viel schneller und auch die Fehlermeldung ist weg :-))

sag ich ja.

watchout schrieb am 19.09.2002 um 19:01

Zitat von grubjo
hi, den mix aus tabs und blanks macht glaube ich Dreamweaver, da ich eigentlich nur mit 2 blanks einrücke.
Was du unter 2. meinst weis ich nicht genau, bin eher ein php-Anfänger bzw. jsp-Umsteiger.

Ciao, Hannes

dreamweaver/frontpage/usw. sind für php-programmierung denkbar ungeeignet, verwend' da lieber nen text-only editor, ausserdem gibts sicher ne einstellung

das andere is' ganz einfach:
mit register_globals werden per post bzw. get übergebene variablen automatisch als globale variable definiert
zb: index.php?coke=1 - dann gibts ne variable namens $coke mit inhalt 1
es trägt sehr zur übersichtlichkeit bei, wenn man stattdessen $_GET['coke'] schreibt, das hat keinen performanceunterschied, eher gehts schneller - dafür steigt die lesbarkeit um 1000%, weil man weiss wo ne variable herkommt

grubjo schrieb am 21.09.2002 um 14:57

aha, guat, da hab ich wieda was dazuglernt - danke!

flaimo schrieb am 27.09.2002 um 06:07

Zitat von funka
ich wuerd an deiner stelle alle bilder mal resizen
und dann die resizten speichern...
kannst auch so machen dast im resizeimage.php checkst obs das resizte bild schon gibt - falls ja
header(..);
fpassthru ( $bild );
falls nein
resizen
speichern

also i und filesystem san scho immer auf kriegsfuß gewesen... gibts da irgendwo scho was fertiges zum downloaden, weil i hab nix gefunden, aber i kann ma ned vorstellen dass sowas ned scho mal ausprogrammiert worden is.

spunz schrieb am 27.09.2002 um 10:40

http://sourceforge.net/projects/gallery/

dafür gibts sogar nen klickbunti client

funka schrieb am 27.09.2002 um 11:37

jo gallery is supi

flaimo schrieb am 27.09.2002 um 15:16

ich brauch keine ganze rießen gallery applikation, nur ein script wie ich ein erzeugtes bild cachen kann, bzw. gechached bild gleich weiterreichen kann. im prinzip sollte man einstellen können wieviel sekunden es maximal gechached werden soll bevor es neu erstellt wird, aber auf jeden fall auch erneuert werden, sollte sich was an der original ausgangsdatei was geändert haben.