Scriptingvragen (PHP, JavaScript etc.)

[Salomon]

Oeps, foutje
Zoek:

Code: Selecteer alles

$pag = isset ( $_GET['pag'] ) : addslashes ( $_GET['pag'] ) : "home";

Vervang:

Code: Selecteer alles

$pag = isset ( $_GET['pag'] ) ? addslashes ( $_GET['pag'] ) : "home";

Ofwel, verander de eerste : met ?

En header.php en footer.php moeten wel bestaan

[Paul]

het is onzinnig om die get var te addslashen.

[svenn]

Ik zit even vast met het volgende, en ik zie niet meteen waar ik misloop.
ik moet een array includen in een bestandbrowser talen; Om dingen te includen heb ik een kleine functie;
namelijk :

Code: Selecteer alles

function add($path,$file,$nood,$text)
{
	if (!file_exists($file)) 
      	{	
		// check from mainmap
		$include = $path.$file;
	  	error("Bestand bestaat niet! $include<br/>$text","1");
	}
	else
	{
		echo "<i>".$path.$file."</i><br>";
		include($path.$file);
	}
}

nu pas ik deze toe, maar het vreemde is, dat de eerste 2 bestanden worden ge-included en de 3de niet. Wanneer ik gewoon met include(); werk, komt de array wel mee :s :

Code: Selecteer alles

<?php
error_reporting(E_ERROR | E_WARNING | E_PARSE | E_NOTICE);
// basic startup
include "./start.php";
$path = "";
// file adding

add($path,"classes/template.class.php","1","bewerk bestand ontbreekt");//<-- deze werkt
add($path,"classes/engine.class.php","1","engine bestand ontbreekt");//<-- deze werkt
add($path,"classes/talen.class.php","0","taal bestand ontbreekt");//<-- deze werkt niet :-(

//include "classes/talen.class.php"; //<-- deze werkt
// knip knip code ;)
?>

[Bas]

De include gebeurt daar in de scope van die functie, en de definitie van $talen ook. Zet dus, bovenin je talenarray-bestand, global $talen;

[Jan]

Ring.php Wanneer ik naar ?id=9&act=next ga, kom ik gewoon op de index van de site zelf terecht, en niet bij array element #1, http://www.janmarques.net

Wat doe ik fout?

[ElbertF]

$input == count($sites);

count() telt 10 vars in de array, geen 9.

[Jan]

Ja, fout geformuleerd, maar toch komt ie er bij 9 niet door, bij 10 gaat ie inderdaad netjes weer naar de eerste.

[ElbertF]

Hij probeert naar 10 te gaan, maar $sites[10] bestaat dus niet.

[Jan]

Ik geef toch op dat ie bij de uiterste waarde naar het 2e element moet gaan, wat die uiterste waarde ook mag zijn?

Hij doet het daarmee niet...

[Salomon]

Als de actie random is en id niet is ingesteld, werkt de if nog niet helemaal, maar dit werkt iig wel:

Spoiler: bekijk

Code: Selecteer alles

<?php
# [RING.PHP] - Script + English Guide
# ?? Jan Marques, 2007 (http://dev.janmarques.net/)
#
// Global
//     - Which site?
$input = $_GET['id'];
$action = $_GET['action'];

if ( empty ( $action ) || ( !in_array ($action, array("random", "previous", "next" ) ) ) || ( empty ( $input ) && $action != "random" ) || ( !is_numeric ( $input ) ) )
{
    die("ID/ACTION was not well defined!");
}

//     - Define sites in ring
$sites = array(
   1 => "http://www.janmarques.net",
        "http://mods.janmarques.net/",
        "http://www.therebelfactory.com/",
        "http://www.gta-antwerp.com", 
        "http://www.wmstart.nl", 
        "http://www.phpbb.nl", 
        "http://www.cuteptr.com", 
        "http://www.slforums.be", 
        "http://www.google.be"
);

// Random link
$random = array_diff($sites, array($sites[$input])); 
$random = array_rand($random);
$random = $sites[$random];

// Previous link
if ($input == 1)
{
    $previous = array_pop($sites);
}
else
{
    $number = $input - 1;
    $previous = $sites[$number];
}

// Next link
if ($input ==  ( count($sites) + 1 ) )  
{
    $next = $sites[1];
}
else
{
    $number = $input + 1;
    $next = $sites[$number];
}

switch($action)
{
    case "random":
        die("Location: " . $random);
    break;
        
    case "previous":
        die("Location: " . $previous);
    break;
        
    case "next":
        die("Location: " . $next);
    break;
}

?>

[Jan]

Lol, absoluut niet.

Hij heeft nu weer:
ring.php?action=next&id=8
Location: http://www.google.be
ring.php?action=next&id=9
Location:
ring.php?action=next&id=10
Location: http://www.janmarques.net

[Salomon]

Ow wacht, vervang die door header

Wijziging: Uhm, werkt het nu?

Spoiler: bekijk

Code: Selecteer alles

<?php
# [RING.PHP] - Script + English Guide
# ?? Jan Marques, 2007 (http://dev.janmarques.net/)
#
// Global
//     - Which site?
$input = $_GET['id'];
$action = $_GET['action'];

if ( empty ( $action ) || ( !in_array ($action, array("random", "previous", "next" ) ) ) || ( empty ( $input ) && $action != "random" ) || ( !is_numeric ( $input ) ) )
{
    die("ID/ACTION was not well defined!");
}

//     - Define sites in ring
$sites = array(
   1 => "http://www.janmarques.net",
        "http://mods.janmarques.net/",
        "http://www.therebelfactory.com/",
        "http://www.gta-antwerp.com", 
        "http://www.wmstart.nl", 
        "http://www.phpbb.nl", 
        "http://www.cuteptr.com", 
        "http://www.slforums.be", 
        "http://www.google.be"
);

// Random link
$random = array_diff($sites, array($sites[$input])); 
$random = array_rand($random);
$random = $sites[$random];

// Previous link
if ($input == 1)
{
    $previous = array_pop($sites);
}
else
{
    $number = $input - 1;
    $previous = $sites[$number];
}

// Next link
if ($input ==  ( count($sites) + 1 ) )  
{
    $next = $sites[1];
}
else
{
    $number = ( $input == count( $sites ) ) ? 1 : $input + 1;
    $next = $sites[$number];
}

switch($action)
{
    case "random":
        header("Location: " . $random);
    break;
        
    case "previous":
        header("Location: " . $previous);
    break;
        
    case "next":
        header("Location: " . $next);
    break;
}

?>

[Jan]

Ja, maar nu worden zowel id10 als id9 doorverwezen naar http://www.janmarques.net, terwijl 10 een error zou moeten geven

[Salomon]

Dan heeft 10 pech toch

[Jan]

Nee, jij hebt pech, want nu moet je verder naar de fout zoeken

[Salomon]

Zet na $sites = array(......); eens:

Code: Selecteer alles

if ($input == count($sites) {
    die("Oops!");
} 

[Jan]

Werkt niet, want bij 11, 12 gaf ie wel al error

[Salomon]

Code: Selecteer alles

if ($input == count($sites) - 1 {
    die("Oops!");
}  

[Paul]

Verander:

Code: Selecteer alles

$random = $sites[array_rand(array_diff($sites, array($sites[$input], $sites["0"])))];

In

Code: Selecteer alles

$random = $sites[rand(0, sizeof($sites) - 1)];

En

Code: Selecteer alles

// Previous link
if ($input == "1")
{
$previous = array_pop($sites);
}
else
{
$number = $input - 1;
$previous = $sites[$number];
}

// Next link
if ($input == count($sites))
{
$next = $sites["1"];
}
else
{
$number = $input + 1;
$next = $sites[$number];
}

in

Code: Selecteer alles

// Previous link
$prev = $input - 1;
$next = $input + 1;
if (!isset($sites[$prev]))
{
$prev = 1;
}
$previous = $sites[$prev];

if (!isset($sites[$next]))
{
$next = sizeof($sites) - 1;
}
$next = $sites[$next];
}

[Jan]

Bedankt!
Alleen: die $random, nu neemt ie toch ook de site zelf mee, niet?