Wat is er mis met dit script?

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

G P

G P

14/05/2005 20:40:00
Quote Anchor link
Ik heb een probleem: ga eens eventjes naar http://www.avosvzw.be/gallerij.php

Daar zie je een fotoalbum. Maar ik krijg altijd het volgende te zien: "Notice: Undefined offset: 1 in /var/www/html/gallerij.php on line 60"

Wat doe ik verkeerd?


Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
<?php
error_reporting(E_ALL);
// change
$map = '.';    // dir for images
$col = 3;        // collumns per page
$row = 3;    // rows per page
$img = array('jpg'); // possible extentions
$tab = array(); // empty
if(isset($_GET['actie'])){
    $_GET['actie'] = $_GET['actie'];
}

else{
    $_GET['actie'] = '';
}

if(isset($_GET['dir'])){
    $_GET['dir'] = $_GET['dir'];
}

else{
    $_GET['dir'] = '';
}

$dir = $_GET['dir'];
if(!isset($_GET['p'])){
    $p = 1;
}

else{
    $p = $_GET['p'];
}
function
img_resize($a, $b = '', $c = ''){
    if(file_exists($a)){
        if($b == '' || !is_numeric($b)){
            $b = 100;
        }

        if($c == '' || !is_numeric($c)){
            $c = 100;
        }

        $f = getimagesize($a);
        $g = array();
        if($f[1] > $c || $f[0] > $b){
            if($f[0] > $c){
                $d = $f[0] - $b;
                $e = $d / $f[0];
                $g[0] = $b;
                $g[1] = $f[1] - ($f[1] * $e);
            }

            elseif($f[1] > $c){
                $d = $f[1] - $c;
                $e = $d / $f[1];
                $g[0] = $f[0] - ($f[0] * $e);
                $g[1] = $c;
            }

            else{
                $g[1] = $c;
                $g[0] = $b;
            }
        }

                else{
            $g[0] = $f[0];
            $g[2] = $f[1];
                }

        return 'src="'.$a.'" width="'.$g[0].'" height="'.$g[1].'"';
    }

    else{
        return false;
    }
}
function
dir_array($a, $b){
    $g = array();
    if($c = opendir($a)){
        while(false !== ($d = readdir($c))){
            if($d != "." && $d != ".."){
                $e = explode('.', $d);
                $f = $e[count($e)-1];
                if(is_dir($a."/".$d)){
                    $g = array_merge($g, dir_array($a."/".$d, $b));
                }

                elseif(in_array(strtolower($f), $b)){
                    $g[] = $a."/".$d;
                }
            }
        }

        closedir($c);        
    }

    return $g;
}
function
map_array($a){
    $g = array();
    if($c = opendir($a)){
        while(false !== ($d = readdir($c))){
            if($d != "." && $d != ".."){
                if(is_dir($a."/".$d)){
                    $g[$d] = $a."/".$d;
                }
            }
        }

        closedir($c);        
    }

    return $g;
}
function
sort_table($a, $b, $e){
    global $p;
    $f = $b*$e;
    $f = count($a)/$f;
    $f = ceil($f);
    if(isset($_GET['actie'])){
        $actie = '&actie='.$_GET['actie'];
    }

    else{
        $actie = '';
    }

    if($_GET['dir'] != ''){
        $actie .= '&dir='.$_GET['dir'];
    }

    if($b > count($a)){
        $b = count($a);
    }

    if($p < 1 || $p > $f){
        $p = 1;
    }

    $c = '<table border="1" width="'.(150*$b).'" style="border-collapse: collapse">';
    $d = 1;
    for($i = $b*$e*($p-1); $i < ($b*$e*$p); $i++){
        if($d == 1){
            $c .= '<tr align="center" valign="middle">';
        }

        $c .= '<td width="150" height="150">';
        if(isset($a[$i])){
            $c .= $a[$i];
        }

//        else{
//            $c .= '&nbsp;';
//        }

        $c .= '</td>';
        if($d == $b){
            $d = 1;
            $c .= '</tr>';
        }

        else{
            $d++;
        }
    }

    $c .= '<tr align="center" valign="middle"><td colspan="'.$b.'">';
    if($p > 1){
        $c .= '<a href="?p='.($p-1).$actie.'">< Vorige</a>';
    }

    else{
        $c .= '< Vorige';
    }

    $c .= ' | ';
    for($j = 1; $j <= $f; $j++){
        if($p != $j){
            $c .= '<a href="?p='.$j.$actie.'">'.$j.'</a>';
        }

        else{
            $c .= '<b>'.$j.'</b>';
        }

        $c .= ' | ';
    }

    if($p < $f){
        $c .= '<a href="?p='.($p+1).$actie.'">Volgende ></a>';
    }

    else{
        $c .= 'Volgende >';
    }

    if($_GET['actie'] == 2){
        $c .= '<br /><a href="?actie=1&dir='.$_GET['dir'].'">Overzicht</a>';
    }

    elseif($_GET['actie'] == 1){
        $c .= '<br /><a href="?dir=">Overzicht</a>';
    }

    $c .= '</td></tr>';
    $c .= '</table>';
    return $c;
}

?>

<html>
<head>
<title>Foto's Cm 2004 Schwarszee</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
</head>
<body bgcolor="#000000" text="#FFFFFF" link="#FFFFFF" vlink="#FFFFFF" alink="#FFFFFF">
<div align="center">
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
<?php
$i
= 1;
foreach(dir_array($dir, $img) as $a){
    if(file_exists($a) == true){
        if($_GET['actie'] == 1){
            $tab[] = '<a href="?p='.$i.'&actie=2&dir='.$_GET['dir'].'"><img '.img_resize($a).' border="0" /></a>';
        }

        else{
            $tab[] = '<a href="'.$a.'"><img '.img_resize($a, 700, 700).' border="0" /></a>';
        }

        $i++;
    }
}

if($_GET['actie'] == '' || $_GET['dir'] == '' || !is_dir($_GET['dir'])){
    echo '<b>Fotoalbums:</b><br />';
    if(count(map_array($map)) != 0){
        foreach(map_array($map) as $a => $b){
            echo '- <a href="?actie=1&dir='.$b.'">'.$a.'</a><br />';
        }
    }

    else{
        echo '- <a href="?actie=1&dir='.$map.'">'.$map.'</a><br />';
    }

    echo '- <a href="?actie=1&dir='.$map.'">Allemaal</a><br />';
}

elseif($_GET['actie'] == 1){
    echo sort_table($tab, $col, $row);
}

elseif($_GET['actie'] == 2){
    echo sort_table($tab, 1, 1);
}

?>

</div>
</body>
</html>
 
PHP hulp

PHP hulp

24/11/2024 23:23:42
 
Jelmer -

Jelmer -

14/05/2005 20:47:00
Quote Anchor link
(@willem-Jan, ik zei toch dat lijnnummers handig waren)

Welke regel is regel 60 eigenlijk?
Een undefined offset komt meestal (altijd) doordat een script een niet bestaande key uit een array opvraagt. Is vaak te verhelpen met isset(), om even te kijken of hij wel bestaat.
 
Willem Jan Z

Willem Jan Z

14/05/2005 20:48:00
Quote Anchor link
Niks...
Haal error_reporting(E_ALL); bovenin maar weg... Dan geeft hij de notice niet meer.... Maar wat de notice veroorzaakt snap ik niet...
 
Willem Jan Z

Willem Jan Z

14/05/2005 20:54:00
Quote Anchor link
Ik start gewoon ff me HTML-editor (als hij niet al open staat) en plaats het script erin... Heel makkelijk...

Lijn 60: return 'src="'.$a.'" width="'.$g[0].'" height="'.$g[1].'"';

maar ja, kan ook een lijn erboven zijn... Misschien staat < ? niet op de eerste regel bij hem...

Omgeving:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
<?
       else{
            $g[0] = $f[0];
            $g[2] = $f[1];
                }

       /* Line 60 */ return 'src="'.$a.'" width="'.$g[0].'" height="'.$g[1].'"';
    }

    else{
        return false;
?>


Edit:
waarschijnlijk is het de $a, die staat nergens gedefinieerd
Gewijzigd op 14/05/2005 20:55:00 door Willem Jan Z
 
MoehMan x

MoehMan x

14/05/2005 20:55:00
Quote Anchor link
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
<?php
return 'src="'.$a.'" width="'.$g[0].'" height="'.$g[1].'"';
?>

=regel 60
deze wordt aangeroepen na als else van de if:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
<?php
if($f[1] > $c || $f[0] > $b){
?>



zit de fout niet in de else daarvan?
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
<?php
     else{
            $g[0] = $f[0];
            $g[2] = $f[1];
}

?>

en moet $g[2] niet $g[1] zijn? (blinde gok)

EDIT: ik ben te traag met typen :/
EDIT: $a is een parameter en is dus wel gedefinieerd lijkt me
Gewijzigd op 14/05/2005 20:57:00 door MoehMan x
 
G P

G P

14/05/2005 21:07:00
Quote Anchor link
Willem-Jan:
Niks...
Haal error_reporting(E_ALL); bovenin maar weg... Dan geeft hij de notice niet meer.... Maar wat de notice veroorzaakt snap ik niet...


Bedankt, dat heeft dus geholpen!!!

Nu heb ik wel nog het probleem met de foto's: je ziet een thumb-overzicht, maar als je op de thumbnail klikt krijg je de afbeelding dus niet te zien. En als je het te zien krijgt is ie platgedrukt.

P.S. Ik gebruik de mappen gallerij1 en gallerij2 als voorbeeld
 
Willem Jan Z

Willem Jan Z

14/05/2005 21:13:00
Quote Anchor link
In FF wel hoor is het wel goed te zien trouwens...

Quote:
Uit broncode:

<img src="./gallerij1/1_12.jpg" width="200" height="" border="0" />


Er staat geen hoogte ingesteld... Als je height helemaal weglaat zal hij wel goed zijn... Of gewoon de goede hoogte neerzetten
 
MoehMan x

MoehMan x

14/05/2005 21:17:00
Quote Anchor link
In Opera werkt hij ook goed...
 
Jelmer -

Jelmer -

14/05/2005 21:29:00
Quote Anchor link
Undefined indez, zoals ik al zei, is fout met key uit een array

height is leeg:
<img src="./gallerij1/1_12.jpg" width="200" height="" border="0" />

undefined index 1
height="'.$g[1].'"

jaja, daar zit een fout. Los die op, en alles werkt zoals het hoort.
 
G P

G P

14/05/2005 23:22:00
Quote Anchor link
Zo'n fout oplossen. Als ik eens wist hoe...

Willem-Jan:
In FF wel hoor is het wel goed te zien trouwens...

Quote:
Uit broncode:

<img src="./gallerij1/1_12.jpg" width="200" height="" border="0" />


Dat staat op die pagina zelf, maar het script werkt maar met één pagina (zie hierboven), en daar staat deze code heel anders vermeld.
 
Willem Jan Z

Willem Jan Z

14/05/2005 23:33:00
Quote Anchor link
Ja, dat snapte ik ook nog wel... Maar dat is wel het probleem, er is geen hoogte gedefinierd, zoals Jelmer ook al aangaf...
 
G P

G P

15/05/2005 10:40:00
Quote Anchor link
Ik kan dus geen hoogte instellen
 



Overzicht Reageren

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.