[PHP/(My)SQL] voor dummies - Deel 53

Index

» digital corner

actieve topics nieuwe topics
abonnement Unibet Coolblue
pi_55676419
quote:
Op maandag 17 december 2007 16:06 schreef Mordreth het volgende:

[..]

Deze is simpel aan te passen :
[ code verwijderd ]
Ik kom er nog steeds niet compleet uit....

Wat ik zoek is een functie die mij de timestamp van de eerste dag van weeknummer X in jaar Y geeft. Ik heb er net weer mee zitten te stoeien, maar ik krijg geen algoritme uit mijn pols geschud die dit voor elkaar krijgt op een manier dat het ook met de jaarwisseling goed gaat.
pi_55676534
Ik heb het volgende probleem:

Ik ben bezig een Google Sitemap XML bestand te genereren. Lokaal (Windows XP, IIS) werkt dit allemaal, en als ik het bestand in m'n browser open krijg ik netjes een XML tree te zien. Nu upload ik het naar mijn server (Linux, Apache) en daar krijg ik een blanco bestand.

Ik ben er achter gekomen dat dit veroorzaakt wordt door de eerste regel van het XML bestand, namelijk:
1<?xml version='1.0' encoding='UTF-8'?>


Als ik deze regel weghaal krijg ik wel output te zien, maar is het (natuurlijk) geen XML meer...
Ik heb werkelijk waar géén idee waarom dit is of wat ik er aan zou moeten doen
pi_55677000
quote:
Op maandag 31 december 2007 11:34 schreef Xcalibur het volgende:
Ik heb het volgende probleem:

Ik ben bezig een Google Sitemap XML bestand te genereren. Lokaal (Windows XP, IIS) werkt dit allemaal, en als ik het bestand in m'n browser open krijg ik netjes een XML tree te zien. Nu upload ik het naar mijn server (Linux, Apache) en daar krijg ik een blanco bestand.

Ik ben er achter gekomen dat dit veroorzaakt wordt door de eerste regel van het XML bestand, namelijk:
[ code verwijderd ]

Als ik deze regel weghaal krijg ik wel output te zien, maar is het (natuurlijk) geen XML meer...
Ik heb werkelijk waar géén idee waarom dit is of wat ik er aan zou moeten doen
En Apache parsed de "<?" niet als PHP?
pi_55677004
Ja, ik heb er een
1header('Content-type: text/xml; charset=UTF-8');


in staan.... lijkt me goed toch?
pi_55677032
quote:
Op maandag 31 december 2007 11:49 schreef Geqxon het volgende:
En Apache parsed de "<?" niet als PHP?
Scherp.... dat zou het wel eens kunnen zijn.
Lokaal staat short open tags uit, online staat het aan.... eens kijken of ik die kan aanpassen

Edit: uit de PHP documentatie bij short_open_tag
quote:
Tells whether the short form (<? ?> ) of PHP's open tag should be allowed. If you want to use PHP in combination with XML, you can disable this option in order to use <?xml ?> inline. Otherwise, you can print it with PHP, for example: <?php echo '<?xml version="1.0"'; ?> . Also if disabled, you must use the long form of the PHP open tag (<?php ?> ).


[ Bericht 34% gewijzigd door Xcalibur op 31-12-2007 11:55:32 ]
pi_55677169
quote:
Op maandag 31 december 2007 11:50 schreef Xcalibur het volgende:

[..]

Scherp.... dat zou het wel eens kunnen zijn.
Lokaal staat short open tags uit, online staat het aan.... eens kijken of ik die kan aanpassen :)
Mocht je het niet aan kunnen passen:

1
2
3		<?php
echo "<?xml ............ ":
?>


Het kost je wat overhead, maar is dat voor een Google sitemap belangrijk?
pi_55678294
quote:
Op maandag 31 december 2007 11:30 schreef Geqxon het volgende:

[..]

Ik kom er nog steeds niet compleet uit....

Wat ik zoek is een functie die mij de timestamp van de eerste dag van weeknummer X in jaar Y geeft. Ik heb er net weer mee zitten te stoeien, maar ik krijg geen algoritme uit mijn pols geschud die dit voor elkaar krijgt op een manier dat het ook met de jaarwisseling goed gaat.
En dat lukt mij inmiddels. Het blijft echter gestoei met de weeknummers. Het is op het moment Week 1 van 2007, en dat is waar het ook gelijk goed fout gaat in het script. Ik ga maar weer even worstelen.
pi_55678299
ja, ik zag dat dat kon inderdaad.... maar dat is natuurlijk wel een ranzige oplossing
pi_55679806
Even snel een klein HTTaccess vraagje.

([a-zA-Z0-9] valideert echter ook "(niets) en ik wil hier een minimum aan vast knopen van bv 4 karakters! en geen maximum? anyone?
Just say hi!
pi_55680020
Zo uit mijn hoofd:

1(a-zA-Z0-9]{4,})
quote:
Using extended regular expressions, you can specify arbitrary pattern occurrence counts using a more verbose syntax than the question-mark, plus-sign, and asterisk quantifiers. The curly-braces ("{" and "}") can surround a precise count of how many occurrences you are looking for.

The most general form of the curly-brace quantification uses two range arguments (the first must be no larger than the second, and both must be non-negative integers). The occurrence count is specified this way to fall between the minimum and maximum indicated (inclusive). As shorthand, either argument may be left empty: if so the minimum/maximum is specified as zero/infinity, respectively. If only one argument is used (with no comma in there), exactly that number of occurrences are matched.

http://gnosis.cx/publish/programming/regular_expressions.html
pi_55682507
iemand een idee of ik de $_SERVER['SERVER_ADMIN'] kan benaderen met php in CLI?
of is dat echt helemaal onmogelijk en zal ik het moeten hardcoden?
pi_55682587
quote:
Op maandag 31 december 2007 13:22 schreef Geqxon het volgende:
Zo uit mijn hoofd:
[ code verwijderd ]
Tnx, ik had zelf al geprobeerd maar blijkbaar ging het door de * fout
Just say hi!
pi_55682689
quote:
Op maandag 31 december 2007 12:30 schreef Geqxon het volgende:

[..]

En dat lukt mij inmiddels. Het blijft echter gestoei met de weeknummers. Het is op het moment Week 1 van 2007, en dat is waar het ook gelijk goed fout gaat in het script. Ik ga maar weer even worstelen.
Het is week 1 van 2008. En met wat code om de eerste dag van die week te bepalen kom ik op 31 december 2007.

Edit - de code:
1
2
3
4
5
6
7
8
9
10
11
12
13
14		<?php
function firstDayOfWeek($week, $year) {
    $jan1st = mktime(0,0,0,1,1,$year);
    if(date('W', $jan1st) == 1) {
        //Correctie toepassen, alleen voor als 1 januari in week 1 zit
        // En 1 januari kan soms ook bij week 53 of 52 worden gerekend.
        $week--;
    }
    $weekday = date('N', $jan1st) - 1; //Dag 1 is maandag
    return strtotime('+'.$week.' weeks -'.$weekday.' days', $jan1st);
}

echo date('Y-m-d', firstDayOfWeek(1,2008)); //Levert 2007-12-31
?>


[ Bericht 76% gewijzigd door Light op 31-12-2007 14:57:17 ]
fokstok.nl
pi_55683103
quote:
Op maandag 31 december 2007 14:48 schreef Light het volgende:

[..]

Het is week 1 van 2008. En met wat code om de eerste dag van die week te bepalen kom ik op 31 december 2007.

Edit - de code:
[ code verwijderd ]
Dat klopt, en als "date('W-Y')" keurig 01-2008 zou tonen dan zou mijn pagina perfect werken, want ook het algoritme dat ik gebruik geeft dan 31 december weer. Helaas, hij geeft nu 01-2007 weer.

Het is jammer dat de date & time functies van PHP wat rommelig zijn, want "strftime("%W-%Y")" geeft 53-2007 weer, aangezien die weer geen ISO weeknummer geeft.
pi_55683661
quote:
Op maandag 31 december 2007 15:04 schreef Geqxon het volgende:

[..]

Dat klopt, en als "date('W-Y')" keurig 01-2008 zou tonen dan zou mijn pagina perfect werken, want ook het algoritme dat ik gebruik geeft dan 31 december weer. Helaas, hij geeft nu 01-2007 weer.

Het is jammer dat de date & time functies van PHP wat rommelig zijn, want "strftime("%W-%Y")" geeft 53-2007 weer, aangezien die weer geen ISO weeknummer geeft.
Met date("W o") krijg je wel het ISO weeknummer en het ISO jaar. Nadeel is dat je PHP 5.1.0 of nieuwer nodig hebt.
fokstok.nl
pi_55683836
quote:
Op maandag 31 december 2007 10:32 schreef Tarabass het volgende:
Fix-List:

Variable (volgens mij) nu global gemaakt.
Escape bij declaratie weggehaald en in de query pas meegegeven
If-statements verkort naar 1 if-statement
ID-integer uit de query gesloopt, ook geen idee waarom ik hem er bij had gezet (en dan ook nog als string).
exit; comment weggehaald.

Waarom moet ik de variable $ip escapen in de query? Dit is een integer, en geen string.
$date is een datetime. Waarom vraag je dat?

A List Apart wordt doorgenomen, dank voor de link.

Code na aanpassingen:
[ code verwijderd ]
Een IP is wel een string, bestaat uit getallen met puntjes.

Kijk voor de grap ook eens naar: http://nl3.php.net/manual/en/function.ip2long.php
persoonlijke pagina
weblog
pi_55683949
Eigenlijk is een IP-adres een array van characters.
pi_55687910
quote:
Op maandag 31 december 2007 15:29 schreef colourAgga het volgende:

[..]

Een IP is wel een string, bestaat uit getallen met puntjes.

Kijk voor de grap ook eens naar: http://nl3.php.net/manual/en/function.ip2long.php
Kijk voor de grap eens goed naar zijn code
Wil iedereen die in telekinese gelooft nu mijn hand op steken?
| Foto's van toen en nu | Icons | Whatpulse keyboard | .NET developer? |
pi_55688265
quote:
Op maandag 31 december 2007 15:29 schreef colourAgga het volgende:

[..]

Een IP is wel een string, bestaat uit getallen met puntjes.

Kijk voor de grap ook eens naar: http://nl3.php.net/manual/en/function.ip2long.php
Dat gebruik ik ook in m'n code. Ik haal hem ook op met long2ip

En hoe ziet de veiligheid van de code er nu uit? Zijn er nog dingen die ik toe moet voegen, en waar zou het mis op kunnen gaan? Testen kan hier..

wees een beetje zuinig op m'n database ajb
pi_55689079
quote:
Op maandag 31 december 2007 14:43 schreef mschol het volgende:
iemand een idee of ik de $_SERVER['SERVER_ADMIN'] kan benaderen met php in CLI?
of is dat echt helemaal onmogelijk en zal ik het moeten hardcoden?
Die server_admin variabele bevat de waarde uit de apache config ServerAdmin. CLI gebruikt niet de SAPI en die variabele hoort gewoon leeg te zijn in CLI.
En ServerAdmin is nou niet iets waar je blindelings op moet vertrouwen.
Bij phpinfo() (of php -i) zie je welke variabelen je tot je beschikking hebt, misschien heb je daar wat aan?
pi_55689476
quote:
Op maandag 31 december 2007 10:32 schreef Tarabass het volgende:
Code na aanpassingen:
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		<?php
   function CheckPostData() {
        $ip   = $_SERVER['REMOTE_ADDR'];
        $ip   = ip2long($ip);
      $data = $_POST['data'];
      $date = date('y-m-j H:i:s');
        Global $mysql_tbn;

        if(($data != "Hier je bericht....." && $data != "") && (is_string($data))) {
            InsertQuery($ip, $data, $date);
        }
   }
    function InsertQuery($ip, $data, $date) {
        Global $mysql_tbn;

      $sql_insert = "INSERT INTO " . $mysql_tbn . " SET ";
      $sql_insert .= "ip = " . $ip;
      $sql_insert .= ", data = '" . mysql_real_escape_string($data) . "'";
      $sql_insert .= ", date = '" . $date . "'";

        $result = mysql_query($sql_insert);
        header("Location: " . $_SERVER['PHP_SELF']);
        exit;
    }
?>
Wat opmerkingen:
* Global $mysql_tbn; heb je niet nodig in CheckPostData
* is_string geeft ook true als de invoer ($data) leeg is en dus overbodig.
* De opbouw van $sql_insert vind ik vervelend lezen, gebruik OF een prepared statement of iets als sprintf:
1
2
3		<?php
$sql_insert = sprintf("INSERT INTO %s SET date='%s',ip=%d, data=' %s' ",$mysql_tbn,$date,$ip,mysql_real_escape_string($data));
?>

* Je doet niks met $result in InsertQuery en als de query.
* Waarom geen else clause bij if ($data blabla) ?
pi_55690335
quote:
Op maandag 31 december 2007 14:48 schreef Light het volgende:

[..]

Het is week 1 van 2008. En met wat code om de eerste dag van die week te bepalen kom ik op 31 december 2007.

Edit - de code:
[ code verwijderd ]
Dat kan mooier :P

1
2
3
4
5
6		function firstDayOfWeek($week, $year) 
{
   $jan4th = mktime(0, 0, 0, 1, 4, $year);
   $weekday = date('N', $jan4th) + 6; 
   return strtotime("+$week weeks -$weekday days", $jan4th);
}


(4 januari valt altijd in week 1)
Ik heb niet helemaal gecontroleerd of ie goed is, alleen dat ie dezelfde resultaten geeft als die van Light.
Wil iedereen die in telekinese gelooft nu mijn hand op steken?
| Foto's van toen en nu | Icons | Whatpulse keyboard | .NET developer? |
pi_55690743
En voor wat ik gebruik:

1
2
3
4
5
6
7
8
9		<?php
function StartOfWeek($year, $week) 
{ 
    $Jan1 = mktime(1,1,1,1,1,$year); 
    $MondayOffset = (11-date('w',$Jan1))%7-3; 
    $desiredMonday = strtotime(($week-1) . ' weeks '.$MondayOffset.' days', $Jan1); 
    return $desiredMonday; 
} 
?>


http://www.phpbuilder.com/board/showpost.php?s=c26b8729c4b94eadb55d319c3c29bc4a&p=10274032&postcount=5

pi_55691606
quote:
Op maandag 31 december 2007 18:55 schreef smesjz het volgende:
* Global $mysql_tbn; heb je niet nodig in CheckPostData
True, rommelig van me maar die had ik laten staan na aanpassingen van de vorige keer (gaf hem toen mee naar de insertquery)
quote:
* is_string geeft ook true als de invoer ($data) leeg is en dus overbodig.
Dit begrijp ik niet

Zou je dat uit kunnen leggen? Het is toch een 'and' en de string moet dan toch voldoen aan alle drie?
quote:
* De opbouw van $sql_insert vind ik vervelend lezen, gebruik OF een prepared statement of iets als sprintf:
Eens. Is ook niet de manier die ik nu veel gebruik, inderdaad omdat het zo vervelend lezen is.

Nu doe ik meestal dit:
1
2
3		<?php
$sql_insert = "INSERT INTO " . $mysql_tbn . " (ip, data, date) VALUES ('$ip', '" . mysql_real_escape_string($data) . "', '$date')";
?>
quote:
* Waarom geen else clause bij if ($data blabla) ?
Is wel netter, maar had het even snel gebouwd, om de veilig te leren en te testen. Beetje zonde van de tijd dus.

Fix-List:

* Global $mysql_tbn; weggehaald in CheckPostData
* Query herschreven naar een nettere en leesbaarder manier
* Code opgeschoond

Code na aanpassingen:
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22		<?php
    function CheckPostData() {
        $ip   = $_SERVER['REMOTE_ADDR'];
        $ip   = ip2long($ip);
        $data = $_POST['data'];
        $data = trim($data);
        $date = date('y-m-j H:i:s');

        if(($data != "Hier je bericht....." && $data != "") && (is_string($data))) {
            InsertQuery($ip, $data, $date);
        }
    }
    function InsertQuery($ip, $data, $date) {
        Global $mysql_tbn;

        $sql_insert = "INSERT INTO " . $mysql_tbn . " (ip, data, date) VALUES ('$ip', '" . mysql_real_escape_string($data) . "', '$date')";
        $sql_insert = mysql_query($sql_insert); 

        header("Location: " . $_SERVER['PHP_SELF']);
        exit;
    }
?>

@SuperRembo
Waarom moet ik de variable $ip escapen in de query? Dit is een integer, en geen string.
$date is een datetime. Waarom vraag je dat? Jaja, gezien. Dubbele spatie Hoe kan ik die spaties mooier opschonen?

@All
Hoe is de veiligheid van het script nu?

[ Bericht 3% gewijzigd door Tarabass op 31-12-2007 20:52:36 ]
actieve topics nieuwe topics
abonnement Unibet Coolblue
[PHP/(My)SQL] voor dummies - Deel 53
Index

» digital corner

Forum Opties
Forumhop:
Hop naar:
(afkorting, bv 'KLB')