PHPhulp.nl PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
k

Hoe maak ik dit leesbaar?

Door koen willemen op 01-02-2013 09:36
845 views
2 vragen eigenlijk..

- Zijn er overbodige dingen tussen die ik kan weglaten, verkorten?
- Ik script in ZEND, tips om overzichtelijk te scripten?

Heb namelijk pagina's met een minimum van 1-2.000 regels

Klein stukje code

echo "
<table>
<td class='subtitle'/>MaffiaWorlds - Misdaden - Steel light
<tr>
<td class='maintxt'>
<table>
<tr>
<td width='20' valign='top'>
<img src='/Afbeeldingen/Algemeen/Steel-Light/Opmerking.png' height='16' width='16' alt='Opmerking'></td><td>
<span class='Opmerking_Groen'>De overval is gelukt!</span>
<br>
Terwijl de junkie's lagen te slapen, besteelde jij ze!
<br>
Je verdiende € ".number_format($ammount)."
</td></table>
<tr><td class='subtitle'>&nbsp;</td></tr>
<noscript>
";

Ik heb het gevoel dat er een aantal dingen kunnen worden weggelaten. De layout van dit bericht wil ik zo houden.
- SanThe -
01-02-2013 09:39
Html verwacht dubbele quotes en geen enkele.
k
koen willemen
01-02-2013 09:46
Sorry vergeten bij te zeggen ik zit in PHP.
Had misschien beter dit ineens laten zien:

$msg = $message[$msgnum];
if ($kjunkie == 1) {


echo "
<table>
<td class='subtitle'/>MaffiaWorlds - Misdaden - Steel light
<tr>
<td class='maintxt'>
<table>
<tr>
<td width='20' valign='top'>
<img src='/Afbeeldingen/Algemeen/Steel-Light/Opmerking.png' height='16' width='16' alt='Opmerking'></td><td>
<span class='Opmerking_Groen'>De overval is gelukt!</span>
<br>
Terwijl de junkie's lagen te slapen, besteelde jij ze!
<br>
Je verdiende € ".number_format($ammount)."
</td></table>
<tr><td class='subtitle'>&nbsp;</td></tr>
<noscript>
";
E
Erwin H
01-02-2013 09:47 gewijzigd op 01-02-2013 09:47
Een hoop zou al schelen als je al die tables zou weghalen....

<table>
  <tr>
    <td width='20' valign='top'>
      <img src='/Afbeeldingen/Algemeen/Steel-Light/Opmerking.png' height='16' width='16' alt='Opmerking'>     
    </td>
    <td>
      <span class='Opmerking_Groen'>De overval is gelukt!</span>
      <br>
      Terwijl de junkie's lagen te slapen, besteelde jij ze!
      <br>
      Je verdiende € ".number_format($ammount)."
    </td>
</table>

Deze table bijvoorbeeld is nergens voor nodig. Doe je het in divs dan ben je al van een heleboel code af.
(En merk ook op dat je een /tr tag mist....)

[size=xsmall]Toevoeging op 01/02/2013 09:48:59:[/size]

Koen willemen op 01/02/2013 09:46:26

Sorry vergeten bij te zeggen ik zit in PHP.

Daarmee verandert de opmerking van Santhe niet. Gebruik enkele quotes in php en dubbele in html.

k
koen willemen
01-02-2013 09:55
Oke, maar het gebeurt raar of zelden dat ik in html kom, en als ik dat dan deed was het inderdaad met enkele quote's, zal erop letten bedankt!

Maar als ik die tables weghaal, valt heel de structuur weg?
E
Erwin H
01-02-2013 09:57
Als je ze niet vervangt door div elementen wel ja. Maar opmaak met tables is van de vorige eeuw, niet van vandaag. Normale pagina opmaak doe je met div elementen. Scheelt een hoop code, is veel leesbaarder en veel flexibeler (en er voor bedoeld).
k
koen willemen
01-02-2013 10:01
Dus ipv <table> gewoon <div> erin zetten, klein voorbeeldje misschien om me op weg te helpen?
k
koen willemen
01-02-2013 10:12
Ja ik werk al denk ik 2-3jaar met tables.
Maar betekent het dan ook dat ik <td> en dergelijke moet gaan vervangen?
Want nu al m'n pagina gaan beginnen herscripten in iets wat ik niet ken lijkt me wel hopeloos nu :P

En waarom word het afgeraden om me tables te werken?
B
Bart V B
01-02-2013 10:20
Tables zijn voor tab gescheiden data.
Denk bijvoorbeeld een status, of een rekening overzicht.
(Zoiets als wat excel doet.)

En dat is niet wat jij wil, dus daarom is het af te raden.
E
Erwin H
01-02-2013 10:20
Tables moet je gebruiken waarvoor ze bedoelt zijn (net als alle andere elementen) en dat is bij tables voor het tonen van tabulaire data. Dus niet voor het opmaken van pagina's. Daarnaast gelden de redenen die ik hierboven al uiteen gezet heb. Wil je er meer over weten, google eens op 'html why not use tables'. Geeft 675 miljoen resultaten...

In tables ken je geen rows en cells, dus al die thead, tbody, col, th, tr en td tags mag je dan weggooien (minder code dus, zoals ik al zei).

En wat het meer werk betreft, dat is nu zo inderdaad. Ja het zal je wat tijd kosten om dit te leren gebruiken. Dat is eenmalig en hoe eerder je het doet, hoe langer je er plezier aan kunt hebben. Anders zal het je weer problemen op gaan leveren bij je volgende project.

Reageren

Inloggen om te reageren