2010-12-09 4 views
0

Chaque fois que je tente d'afficher les valeurs de données en PHP je reçois l'erreur suivante:Erreur lors de l'affichage des valeurs de données en PHP

Fatal error: Uncaught exception 'Exception' with message 'DateTime::__construct() [datetime.--construct]: Failed to parse time string (this wednesday G:i A) at position 16 (:): Unexpected character' in C:\xampp\htdocs\keypad\end.php:14 Stack trace: #0 C:\xampp\htdocs\keypad\end.php(14): DateTime->__construct('this wednesday ...') #1 {main} thrown in C:\xampp\htdocs\keypad\end.php on line 14

Voici le code que je utilise:

<?php require_once('Connections/rent.php'); ?> 
<?php 

// post 

$id = $_GET['id']; 
$user = $_GET['user']; 
$week = $_GET['week']; 
$pRent = $_GET['pRent']; 
$status = $_GET['status']; 
$rentPaid = $_GET['rentPaid']; 
$result = $_GET['result']; 
$pDate = date("d/m/Y G:i A"); 
$lDate = new DateTime('this wednesday G:i A'); 
$cDate = new DateTime('next tuesday G:i A'); 

// update 

mysql_query("UPDATE rent SET dNo = '$dNo', pdate = '$pDate', pRent = '$result' WHERE rent.id = $id"); 


// if pRent = 0 then set status to clear, colour to #3C0 
if ($pRent == 0) { 
    mysql_query("UPDATE rent SET status = 'clear', colour = '#3C0' WHERE rent.id = $id"); 
} 

// if pRent =! 0 but just paid then set status to paid/not clear, colour to #FF0 
elseif ($pRent =! 0) { 
    mysql_query("UPDATE rent SET status = 'paid', colour = '#3C0' WHERE rent.id = $id"); 
} 

// if status = on holiday, then set status to clear, colour to #3C0 
elseif ($status == 'awaiting') { 
    mysql_query("UPDATE rent SET status = 'awaiting', colour = '#09F' WHERE rent.id = $id"); 
} 

// check all drivers who have/have not paid rent in the past week 
elseif ($lDate < $pDate && $cDate > $pDate) { 
    // date is within desired range 
    mysql_query("UPDATE rent SET status = 'paid', colour = '#3C0'"); 
} else { 
    // date is not within desired range 
    mysql_query("UPDATE rent SET status = 'awaiting', colour = '#09F' WHERE rent.status =! 'not working'"); 
} 
?> 

Toutes les suggestions ?

+1

Je pense que le message d'erreur est assez clair. '' ce mercredi G: i A'' n'est pas quelque chose que 'DateTime()' comprend. – deceze

+0

Je vais juste ajouter les avertissements standard que votre code est extrêmement vulnérable à l'injection SQL; S'il vous plaît consulter les déclarations préparées par PDO (http://au.php.net/manual/en/pdo.prepared-statements.php) comme un moyen plus sûr d'exécuter votre SQL. –

Répondre

3

Vous avez un problème sur la ligne 14, $lDate = new DateTime('this wednesday G:i A') est invalide, jetez un oeil à Relative DateTime formats pour une liste des valides. Ce que vous devez faire est:

$lDate = new DateTime('this wednesday') 
$lDate->format('G:i A'); 

Juste une suggestion, deux premières lignes devraient être

<?php 
    require_once 'Connections/rent.php'; 

Et s'il vous plaît envisager d'utiliser PDO ou d'une autre bibliothèque similaire, ou au moins vos désinfectez requêtes SQL.

Questions connexes