login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Crewpagina

Offline Simax - 21/11/2006 09:18 (laatste wijziging 21/11/2006 09:31)
Avatar van SimaxOnbekend Beste mensen,

Een tijd geleden ben ik me gaan verdiepen in php.
Nu heb ik een script gemaakt, maar denk zelf dat er een aantal dingen niet kloppen. Aan jullie de vraag om eventuele opmerkingen te geven.

http://www.plaatscode.be/3170

Alvast bedankt .

MvG,
Generix

13 antwoorden

Gesponsorde links
Offline Ultimatum - 21/11/2006 09:27
Avatar van Ultimatum PHP expert 1. persoonlijk geef ik de voorkeur om mysql functies (FROM, SELECT) met hoofdletter te schrijven omdat dat duidelijker is.
2. Je geeft een class op de p atribuut maar de class is leeg (<p class=''>
3. Je geeft je td een naam wat volgens mij ook onzinnig is
Offline Simax - 21/11/2006 09:30 (laatste wijziging 21/11/2006 09:45)
Avatar van Simax Onbekend Ok.

1. Daar geef ik je gelijk in.
2. Heb de class verwijderd.
3. Naam is makkelijk voor het terug zoeken.
Je weet wat bij elkaar hoort.

Daarnaast heb ik ontdekt dat ik de while niet heb afgesloten.


En zijn er verder nog opmerkingen?

MvG,
Generix
Offline Ultimatum - 21/11/2006 09:45
Avatar van Ultimatum PHP expert Ja, je zet je tabellen in php zonder te echoën of wat dan ook. Ik zou je aanraden om het gewoon buiten php te houden dus php te sluiten voor html etc.. 
Offline Simax - 21/11/2006 09:47
Avatar van Simax Onbekend Het is ok mijn bedoeling de tabellen zoveel mogelijk in html te schrijven. Mijn vraag is wat er fout gaat? .

MvG,
Generix
Offline Ultimatum - 21/11/2006 09:53
Avatar van Ultimatum PHP expert je moet als je tabellen begint dat je php dan afsluit

dus
  1. <?php
  2. while(iets...)
  3. {
  4. ?>
  5. tablellen hier
  6. <?php
  7. }
  8. ?>
Offline Simax - 21/11/2006 09:55 (laatste wijziging 21/11/2006 12:14)
Avatar van Simax Onbekend Ok.

Bedankt .

Deze manier zou dan beter moeten zijn?

  1. <?php
  2. Mysql_connect('localhost', 'gebruikersnaam', 'wachtwoord');
  3.  
  4. $res= Mysql_query('Select land, naam From crew')
  5. ?>
  6.  
  7. <table cellpadding='1' cellspacing='1'>
  8. <?php
  9. while($arr= Mysql_fetch_assoc($res))
  10. {
  11. ?>
  12. <tr>
  13. <th name='land'>
  14. </th>
  15. <th name='naam'>
  16. <p class=''>Naam</p>
  17. </th>
  18. </tr>
  19. <tr>
  20. <td name='land'>
  21. <img src='<?php $arr['land'] ?>' class='land'/>
  22. </td>
  23. <td name='naam'>
  24. <p><?php $arr['naam'] ?></p>
  25. </td>
  26. </tr>
  27. <?php
  28. }
  29. ?>
  30. </table>
  31.  
  32. <?php
  33.  
  34. ?>


Eventuele opmerkingen graag? 

MvG,
Generix
Offline Ultimatum - 21/11/2006 10:12
Avatar van Ultimatum PHP expert Je zou zelfs dit kunnen doen (als shorttags aanstaan)

  1. <td name='land'>
  2. <img src='<?= $arr['land'] ?>' class='land' />
  3. </td>
Offline Simax - 21/11/2006 10:24
Avatar van Simax Onbekend Ok.

Ik heb mijn script aangepast en
gebruik gemaakt van jou advies.

Zijn er nu nog eventuele opmerkingen?

Alvast Bedankt .

MvG,
Generix
Offline nielsvdwal - 21/11/2006 11:33
Avatar van nielsvdwal PHP gevorderde die <table> en </table> buiten de while zetten .. want nu doet ik elke keer een nieuwe tabel maken.. en als je die buiten de while zet.. dan maakt hij gewoon elke keer alleen een nieuwe rij :-)
Offline Simax - 21/11/2006 12:23
Avatar van Simax Onbekend Het zou op deze manier moeten werken zoals ik het begrijp?

Btw..
Het is toch nuttig om php te sluiten bij het openen van de while, en opnieuw te openen voor het sluiten van de while, of heb ik het mis?

MvG,
Generix
Offline nielsvdwal - 21/11/2006 13:03 (laatste wijziging 21/11/2006 13:05)
Avatar van nielsvdwal PHP gevorderde als je html in de while wilt gebruiken is het nuttig.. als je php wilt gebruiken hoef je het niet aftesluiten nee..

btw..
  1. <th name='land'>
  2. </th>
  3. <th name='naam'>
  4. <p class=''>Naam</p>
  5. </th>

<th> heb ik persoonlijk nog nooit van gehoord.. <td> van maken misschien?  

edit @ hieronder: okeej, ik kende het niet .. maar dan zal het wel goed zijn dus.. waar is <th> dan voor?
Offline Gerard - 21/11/2006 13:04
Avatar van Gerard Ouwe rakker <th> bestaat wel degelijk. Dat mag dus gewoon.
Offline Simax - 21/11/2006 13:22
Avatar van Simax Onbekend Het bestaat iid wel degelijk.
Neem hier maar eens een kijkje..
http://www.site...1&id=2

MvG,
Generix
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.203s