mixing HTML en PHP syntax probleem

[Verwijderd]

code:

<?php if ($x_voorraad < "1") 
{echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=' . urlencode($x_oms) . '&nummer=' . urlencode($x_nummer) . '&partnr=' . urlencode($x_partnr)">
<img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>';}
elseif ($x_voorraad >= "1") {echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";} ?>

de bedoeling van het script is als de voorraad groot genoeg is ( een groen gifje verschijnt )
indien anders zou er één aanvraag icoontje moeten verschijnen ( die de variabelen doorgeeft aan één mailform, die dan achter mailadres van de klant vraagt , om zo de leveringstermijn aan deze klant te melden )

PROBLEEM.
ik krijg dit stukje code maar niet aan de praat
ik heb dit niet zelf geprogrammeerd, maar moet nu code stukjes aanpassen,
daar deze collega niet meer bij ons werkt.

Iemand die mij kan vertellen hoe dit euvel op te lossen
alvast bedankt, vermoed dat het te maken heeft met het parsen van ; " en ' maar ik zie het niet.

[TeeDee]

Definieer: 'ik krijg dit stukje code maar niet aan de praat'

PHP:

<?php if ($x_voorraad < "1") 
{echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=' . urlencode($x_oms) . '&nummer=' . urlencode($x_nummer) . '&partnr=' . urlencode($x_partnr)">
<img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>';}
elseif ($x_voorraad >= "1") {echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";} ?>

Ik ben geen held met PHP (lees: 0,0 ) maar de syntaxhighlighter van GoT haalt er in ieder geval al een tweetal foutjes uit.

[disjfa]

Waarom plant je al je code op zo min mogenlijk regels? Regel gewoon de code in meer regels. Dan is het een stuk overzichtenlijker. En makkelijker te debuggen.

Verder zou ik zelf nooit php en html bijelkaar zetten. Code vs template zeg maar.

[Glabbeek]

Het geheel wordt een stuk duidelijker met wat newlines en tabs in je code:

PHP:

<?php 
    if ($x_voorraad < "1") {
        echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=' . 
            urlencode($x_oms) . 
            '&nummer=' . 
            urlencode($x_nummer) . 
            '&partnr=' . 
            urlencode($x_partnr)"><img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>';
    } elseif ($x_voorraad >= "1") {
        echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";
    } 
?>

Tip: Een enkele quote is iets anders dan een dubbele quote...

[sariel]

Tsja, op de 2e regel heb je een stukje html code die je niet met een quote begint. daardoor gaat de rest ook fout.

[Reinier]

PHP:

< "1"

vind ik er ook wat vreemd uitzien

[LuCarD]

Je kan ook binnen een if statement uit php stappen.

PHP:

if (true) { ?>
<a href="blaaat"><?= $blaat ?></a>
<? } else { ?>
<a href="bar"><?= $foo2 ?></a
<? } ?>

Maar persoonlijk ben ik een voorstander van printf in dit soort gevallen.

PHP:

if ($x_voorraad < 1) 
{
  printf('<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=%s&nummer=%s&partnr=%s"><img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>'
  , urlencode($x_oms)
  , urlencode($x_nummer)
  , urlencode($x_partnr));
}
elseif ($x_voorraad >= 1) 
{ 
  echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";
}

[mcdronkz]

Tijd om eens op http://smarty.php.net te kijken

[Fuzzillogic]

mcdronkz schreef op maandag 11 september 2006 @ 11:23:
Tijd om eens op http://smarty.php.net te kijken

Of doe hip en modern, gebruik XSLT. Maar als je nog nooit een template gezien hebt, neem een kijkje bij XTemplate, hetgeen erg simpel is.

Maar idd: gebruik templates. Je slaat jezelf voor de kop dat je dat niet veel en veel eerder hebt gedaan

[Verwijderd]

alvast bedankt voor de tips,
maar Glabbeek zoals je ziet , is de "elseif" niet in groen ( dit is dus ook m'n probleem in dreamweaver )
als ik het parse krijg ik altijd

Parse error: syntax error, unexpected T_CONSTANT_ENCAPSED_STRING, expecting ',' or ';' in /extranet/artikeldb_viewer/ArtDBlist_weergave_02.php on line 372

die line 372
is lijn 8

PHP:

<?php  
    if ($x_voorraad < "1") { 
        echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=' .  
            urlencode($x_oms) .  
            '&nummer=' .  
            urlencode($x_nummer) .  
            '&partnr=' .  
            urlencode($x_partnr)"><img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>'; 
    } elseif ($x_voorraad >= "1") { 
        echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>"; 
    }  
?>

[Verwijderd]

BIG thanx LuCarD

It works .... :-)

ben nieuw op dit vlak ( maar alvast bedankt allemaal voor de goeie tips )

[Blacksnak]

nm, ff aan't nalezen en zit een en ander nog niet goed. FF checken...

[ Voor 95% gewijzigd door Blacksnak op 11-09-2006 11:35 . Reden: foutje ]

[moozzuzz]

code:

"><img src="...

Toch even alle posts lezen, dan zie je dat er verschillende opm komen ivm het mixen van HTML en PHP. Bovendien maakt de codeparser van GoT duidelijk waar je moet zoeken!

[Zyppora]

Weet niet wie dit zo geprogrammeerd heeft, maar hier krijg ik last van mijn ogen van

PHP:

<?php if ($x_voorraad < "1") 
{echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms=' . urlencode($x_oms) . '&nummer=' . urlencode($x_nummer) . '&partnr=' . urlencode($x_partnr)">
<img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>';}
elseif ($x_voorraad >= "1") {echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";} ?>

Laten we het eens met linebreaks proberen (zoals hierboven al aangegeven):

PHP:

<?php
if ($x_voorraad < "1") {
echo '<a href="../emailons/aanvraagleveringstermijn/contact.php?artikeloms='
. urlencode($x_oms)
. '&nummer=' . urlencode($x_nummer)
. '&partnr=' . urlencode($x_partnr)
"><img src="images/ed_help.gif" alt="Aanvraag Leveringstermijn"></a>';
}elseif ($x_voorraad >= "1") {
echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";
}
?>

Hier zien we al een stuk meer. Bijvoorbeeld, zoals ook al aangegeven, dat er single quotes en double quotes door elkaar zijn gebruikt. Het is heel belangrijk deze uit elkaar te houden. Persoonlijk gebruik ik double quotes om het verschil tussen PHP code en (HTML) output aan te geven (HTML tussen quotes) en single quotes om binnen de HTML code tags etc. te doen.

PHP:

<?php
if ($x_voorraad < 1) {
echo "<a href='../emailons/aanvraagleveringstermijn/contact.php?artikeloms="
. urlencode($x_oms)
. "&nummer=" . urlencode($x_nummer)
. "&partnr=" . urlencode($x_partnr)
. "><img src='images/ed_help.gif' alt='Aanvraag Leveringstermijn'></a>";
}elseif ($x_voorraad >= 1) { // Eigenlijk is het if() gedeelte hier niet nodig
echo "<img src='../../db_vormgeving/algemeen/leverbaarheid/lights_green.gif'>";
}
?>

Let ook op de punt in regel 7, daar miste je er een en dat geeft ook een syntax error. Numerieke waarden hoeven overigens binnen PHP niet per se binnen quotes te staan. Het commentaar over de elseif() structuur slaat eigenlijk op het feit dat als $x_voorraad niet kleiner is dan 1, dat de else dan automatisch wel uitgevoerd wordt. Er bestaan geen andere mogelijkheden dan dat $x_voorraad groter of gelijk is aan 1.

In deze code heb ik trouwens nog een foutje laten zitten. Die mag je zelf opzoeken (zo moeilijk istie namelijk niet). Probeer maar uit.

EDIT: Ben weer goed laat met mijn oplossing

[BCC]

moozzuzz schreef op maandag 11 september 2006 @ 11:31:

code:

"><img src="...

Toch even alle posts lezen, dan zie je dat er verschillende opm komen ivm het mixen van HTML en PHP. Bovendien maakt de codeparser van GoT duidelijk waar je moet zoeken!

Net als elke andere PHP ontwikkel omgeving...

[Reinier]

Zyppora schreef op maandag 11 september 2006 @ 11:35:

PHP:

<?php
. "&nummer=" . urlencode($x_nummer)
. "&partnr=" . urlencode($x_partnr)
?>

In deze code heb ik trouwens nog een foutje laten zitten. Die mag je zelf opzoeken (zo moeilijk istie namelijk niet). Probeer maar uit.

Je bedoelt vast iets anders, maar de ampersands moeten natuurlijk als & geschreven worden

[LuCarD]

Reinier schreef op maandag 11 september 2006 @ 11:57:
[...]

Je bedoelt vast iets anders, maar de ampersands moeten natuurlijk als & geschreven worden

hangt van de doctype af.

[Zyppora]

Reinier schreef op maandag 11 september 2006 @ 11:57:
[...]

Je bedoelt vast iets anders, maar de ampersands moeten natuurlijk als & geschreven worden

En heeft meer met de (X)HTML Recommendations te maken dan met de Programming rules Maar goed opgemerkt. (zo kun je overigens ook aangeven dat de laatste img tag geen alt= attribuut heeft, wat ook bij de Recommendations hoort).