Wat gaat er verkeerd met dit scriptje? (1e script)
Hellaas geeft het script op dit moment telkens het plaatje aan wat weergegeven moet worden als het de juiste datum is. M.a.w. Als ik de datum op dit moment wijzig naar een datum die niet geldig is blijft het plaatje weergegeven worden. (Wijzig de datum op mijn eigen PC die ik nu gebruik als localhost)
Ideeen en oplossingen om dit makkelijker te doen zijn uiteraard van harte welkom. Ben compleet nieuw in php, dus wil graag leren en ontdekken.
Code (php)
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
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
<?php
setlocale (LC_ALL, "nl_NL");
$aDatumBegin = array( "15 September 2008", // De datum instellen wanneer de reclame moet verschijnen.
"29 September 2008",
"20 October 2008",
"5 January 2009",
"19 January 2009",
"25 February 2009");
$aDatumEind = array("24 September", // De datum instellen wanneer de reclame moet eindigen.
"08 October 2008",
"30 October 2008",
"15 January 2009",
"29 January 2009",
"11 March 2009");
foreach($aDatumBegin as $DatumBegin); // Instellen dat deze Array de werkelijke informatie bevat.
foreach($aDatumEind as $DatumEind); // Instellen dat deze Array de werkelijke informatie bevat.
// Hieronder stellen we in van welke datum tot de datum waarop het plaatje weergegeven mag worden.
if(time() >= strtotime("$DatumBegin 00:00:01AM") and time() <= strtotime("$DatumEind 11:59:59PM"))
{
echo "<img src=reclame/plaatje.jpg>"; // Het plaatje wat weergegeven wordt als het tussen de juiste data valt.
}
else
{
echo "<img src=reclame/anderplaatje.jpg>"; // Het plaatje dat wordt weergegeven wanneer het niet de juiste datum is.
}
?>
setlocale (LC_ALL, "nl_NL");
$aDatumBegin = array( "15 September 2008", // De datum instellen wanneer de reclame moet verschijnen.
"29 September 2008",
"20 October 2008",
"5 January 2009",
"19 January 2009",
"25 February 2009");
$aDatumEind = array("24 September", // De datum instellen wanneer de reclame moet eindigen.
"08 October 2008",
"30 October 2008",
"15 January 2009",
"29 January 2009",
"11 March 2009");
foreach($aDatumBegin as $DatumBegin); // Instellen dat deze Array de werkelijke informatie bevat.
foreach($aDatumEind as $DatumEind); // Instellen dat deze Array de werkelijke informatie bevat.
// Hieronder stellen we in van welke datum tot de datum waarop het plaatje weergegeven mag worden.
if(time() >= strtotime("$DatumBegin 00:00:01AM") and time() <= strtotime("$DatumEind 11:59:59PM"))
{
echo "<img src=reclame/plaatje.jpg>"; // Het plaatje wat weergegeven wordt als het tussen de juiste data valt.
}
else
{
echo "<img src=reclame/anderplaatje.jpg>"; // Het plaatje dat wordt weergegeven wanneer het niet de juiste datum is.
}
?>
Gewijzigd op 01/01/1970 01:00:00 door Marc van Gemeren
Code (php)
1
2
3
4
5
6
2
3
4
5
6
SELECT
plaatje
FROM
tabel
WHERE
CURRENT_DATE BETWEEN begin AND eind
plaatje
FROM
tabel
WHERE
CURRENT_DATE BETWEEN begin AND eind
Mocht deze query geen resultaten opleveren, weet je dus dat er geen enkel plaatje voldoet en dat je dus een standaard plaatje moet tonen.
Het voorgaande scriptje geeft aan dat ieder element in de array in $value staat, gedurende de loop (alles tussen { en }). Die accolades ({ en }) ontbreken bij jou. Overigens zou het beter zijn als je gebruik maakt van de index (automatische key van een value).
Dat doe je dan zo:
Je kunt nu $key gebruiken om bij de andere array de einddatum op te halen. (Via $array2[$key]). Nog beter zou zijn als je de start- en einddatum in één array zet. Dus:
Inderdaad kan ik me voorstellen dat zoiets vanuit een database nog makkelijker is te organiseren en benaderen. Maar laat ik maar rustig beginnen.
Ik ga eens aan het werk met de tip van Jezpur.
Je hoort en ziet wel weer van me terug.
Jezpur schreef op 28.10.2008 15:55:
Waarom niet? Ik vind een stukje SQL begrijpend lezen een stuk makkelijker voor een beginner dan dat met PHP te doen. Meteen goed leren, daar heb je later profijt van.Blanche, als je de titel leest, had je geweten dat dit Marcs eerste script is. Dan moet je nog niet met een database aan de gang gaan, dat wil je gewoon niet.
Heb nu het volgende scriptje :
Code (php)
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
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
<?php
$datumnu = date('d-m-Y');
$toondata("20-10-2008"=>"30-10-2008",
"05-01-2009"=>"15-01-2009");
foreach($toondata as $begin => $eind) {
if($datumnu > $begin && $datumnu < $eind) {
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
}
echo "$datumnu"; // data checken
echo "$begin"; // data checken
echo "$eind"; // data checken
echo "$toondata"; //data checken
?>
$datumnu = date('d-m-Y');
$toondata("20-10-2008"=>"30-10-2008",
"05-01-2009"=>"15-01-2009");
foreach($toondata as $begin => $eind) {
if($datumnu > $begin && $datumnu < $eind) {
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
}
echo "$datumnu"; // data checken
echo "$begin"; // data checken
echo "$eind"; // data checken
echo "$toondata"; //data checken
?>
De Onderste "echo's" heb ik even neergezet om te kijken wat hij weergeeft.
Maar ik krijg nu slechts de melding
unexpected T_DOUBLE_ARROW (op lijn 14)
Waar ga ik verkeerd? Ligt het aan de functie data? Nogmaals 1e script dus het is heel veel knippen en plakken om te proberen.
Alvast bedankt.
$toondata = array("20-10-20......
Maar je weet dat dit geen datums zijn maar strings met tekst.
Bovendien zet je variabelen niet binnen quotes. Die zet je daarbuiten:
Code (php)
$begin en $eind zullen na jouw foreach() altijd de laatst gebruikte gegevens uit de array bevatten. Ná de loop checken wat het inhoud is dus niet nodig, je kunt beter ín de loop kijken wat het is ;).
Trouwens, variables moet je buiten quotes houden:
Code (php)
// EDIT //
Jezpur, was me net voor met exact dezelde tips :P.
Trouwens:
Dat kan nog wel goed gerekend worden, maar is wat onzinnig...
Overzichtelijker en meer aan te raden is dan toch die laatste ."" weg te houden.
Gewijzigd op 01/01/1970 01:00:00 door Jelle Posthuma
Jezpur schreef op 30.10.2008 10:21:
De tweede
echo "dit is een ".$var."";
is ook niet goed. Je echoot daar 'lucht'. Die "" is zinloos.
Dus gewoon alleen
echo "dit is een ".$var;
Gewijzigd op 01/01/1970 01:00:00 door - SanThe -
En heel belangrijk even de datum veranderd naar 01-11-2008 want ik wil hem vandaag nog wel weergeven.
Wanneer ik nu het script laat lopen krijg ik het volgende geechoëd :
niet de juiste datum (maar de datum valt weltussen de gevraagde)
niet de juiste datum ( waarvoor wordt dit weergegeven?)
30-10-2008 ( dit geeft hij waarschijnlijk weer voor $datumnu)
05-01-2009 ( dit voor $begin)
15-01-2009 ( dit voor $eind)
Array (dit voor toondata)
Iets werkt dus nog niet naar behoren, het gaat natuurlijk om de eerste weergave van "niet de juiste datum" aangezien de datum van vandaag wel juist is.
De andere echo's haal ik toch weer weg maar wilde ook graag zien wat die weergaven.
Als je 01-11-2008 vergelijkt met 31-01-1999 dan is 31-01-1999 groter dan 01-11-2008 om de simpele reden dat er vanuit het meest linker character wordt gekeken. En 3 is groter dan 0.
Om met datums te rekenen gebruik je altijd het YYYY-mm-dd format
Dus het is logisch dat de 1e item in die toondata array niet gaat werken.
Die eerste check van datumnu > begin, dat is true, maar datumnu < eind, daar gaat ie fout.
Want $datumnu is niet kleiner dan 30-10-2008, maar gelijk aan...
De 2e gaat ook fout, daar heb je "$datumnu > $begin"
En ie begin datum van de 2e item in die toondata array, is groter dan datum nu, niet andersom.
Oftewel, is is volkomen logisch dat ze allebij naar de else toe gaan.
// EDIT //
En kijk naar SanThe's reactie hier boven.
Je telt hier niet met hele datums, maar met 3 blokjes integers.
Eerst jaar vergelijken, dan maand en dan pas dag.
Dus:
$datumnu = date('Y-m-d');
En de waarden in de array: JJJJ-MM_DD.
Gewijzigd op 01/01/1970 01:00:00 door Jelle Posthuma
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
Als ik het nu probeer krijg ik terug : "dejuistedatumnietdejuistedatum"
Wanneer ik de $datumnu verander in date(YYYY-mm-dd) krijg ik terug : "nietdejuistedatumnietdejuistedatum"
Nu begrijp ik niet waarom ik 2 keer een waarde terug krijg. Ik wil alleen een juiste waarde terug krijgen als de datum tussen $begin en $eind klopt. En een "else" als de datum niet klopt.
Het gaat uiteindelijk om plaatjes als een soort waarschuwing. Dus als de datum tussen $begin en $eind valt het plaatje weergeven anders een ander plaatje.
Ben waarschijnlijk erg lastig voor een simpel scriptje in jullie ogen, maar ik doe mijn best. Alvast bedankt alweer.
Gewijzigd op 01/01/1970 01:00:00 door Marc van Gemeren
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
$datumnu = date('Y-m-d');
$toondata = array ("2008-10-20"=>"2008-11-01",
"2009-01-05"=>"2009-01-15");
$okee = false;
foreach($toondata as $begin => $eind)
{
if($datumnu > $begin && $datumnu < $eind)
{
$okee = true;
}
}
if($okee)
{
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
?>
$datumnu = date('Y-m-d');
$toondata = array ("2008-10-20"=>"2008-11-01",
"2009-01-05"=>"2009-01-15");
$okee = false;
foreach($toondata as $begin => $eind)
{
if($datumnu > $begin && $datumnu < $eind)
{
$okee = true;
}
}
if($okee)
{
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
?>
In die $toondata array, staan 2 waarden (en 2 keys), die foreach loopt door elke waarde heen (2x dus), en aangezien jij echoëd binnen die foreach, zal die ook 2x die echo doen.
Zoals SanThe dat heeft, gaat ie eerst naar de 1e waarde kijken, vergelijken, en dan $okee op true of false zetten, dan kijkt ie naar de 2e waarden, dan gaat ie $okee overschijven met een true of false aan de hand van of die waarde wel in je if boompje past.
Wat me gelijk brengt op issue met het bovenstaande script:
Het maakt niet uit wat de 1e waarde in de array is, zolang de 2e waarde klopt, dan geeft ie sowieso een true terug. (Ook als de 1e dus false is)
Dan kan je dus beter zo iets doen:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
$datumnu = date('Y-m-d');
$toondata = array ("2008-10-20"=>"2008-11-01",
"2009-01-05"=>"2009-01-15");
$okee = 0;
foreach($toondata as $begin => $eind)
{
if($datumnu > $begin && $datumnu < $eind)
{
$okee++;
}
}
if($okee == count($toondata))
{
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
?>
$datumnu = date('Y-m-d');
$toondata = array ("2008-10-20"=>"2008-11-01",
"2009-01-05"=>"2009-01-15");
$okee = 0;
foreach($toondata as $begin => $eind)
{
if($datumnu > $begin && $datumnu < $eind)
{
$okee++;
}
}
if($okee == count($toondata))
{
echo "de juiste datum" ;// toon
}
else
{
echo "niet de juiste datum"; // niet tonen
}
?>
Jelle Posthuma schreef op 30.10.2008 12:08:
Het maakt niet uit wat de 1e waarde in de array is, zolang de 2e waarde klopt, dan geeft ie sowieso een true terug. (Ook als de 1e dus false is)
Bij mijn scriptje is altijd minimaal 1 array()-waarde waar. Het is mij ook niet echt duidelijk of er 1 of dat ze allemaal waar moeten zijn. Wel scherp opgemerkt trouwens.
SanThe schreef op 30.10.2008 12:19:
Bij mijn scriptje is altijd minimaal 1 array()-waarde waar. Het is mij ook niet echt duidelijk of er 1 of dat ze allemaal waar moeten zijn. Wel scherp opgemerkt trouwens.
Jelle Posthuma schreef op 30.10.2008 12:08:
Het maakt niet uit wat de 1e waarde in de array is, zolang de 2e waarde klopt, dan geeft ie sowieso een true terug. (Ook als de 1e dus false is)
Bij mijn scriptje is altijd minimaal 1 array()-waarde waar. Het is mij ook niet echt duidelijk of er 1 of dat ze allemaal waar moeten zijn. Wel scherp opgemerkt trouwens.
Geef ik je gelijk in.
Eerst stond het in een foreach waardoor je individueel de uitkomst kon zien.
Maarja, zolang de TS maar snapt hoe het in elkaar moet zitten aan de hand van je voorbeeld.
Voor de TS is het nu kwestie van uitbreiden zo hij het wilt, de basis staat er.
SanThe schreef op 30.10.2008 10:26:
De tweede
echo "dit is een ".$var."";
is ook niet goed. Je echoot daar 'lucht'. Die "" is zinloos.
Dus gewoon alleen
echo "dit is een ".$var;
Jezpur schreef op 30.10.2008 10:21:
De tweede
echo "dit is een ".$var."";
is ook niet goed. Je echoot daar 'lucht'. Die "" is zinloos.
Dus gewoon alleen
echo "dit is een ".$var;
Ik bedoelde eigenlijk dat je na een var nog wel door kan gaan met tekst. Had ik misschien beter zo kunnen doen:
Vele malen dank voor de hulp.