login  Naam:   Wachtwoord: 
Registreer je!
 Forum

Slechtste code ooit

Offline marten - 24/09/2008 12:19
Avatar van martenBeheerder Post hier voorbeelden van code waarvan jij vindt dat hij een plekje verdient in dit topic 

Dus slechte opmaak, vreselijke fouten noem maar op 

24 antwoorden

Gesponsorde links
Offline Stijn - 24/09/2008 12:43 (laatste wijziging 24/09/2008 12:46)
Avatar van Stijn PHP expert Mag het ook sitemasters code zijn? Hier is alvast de eerste code die ik op sitemasters poste, sindsdien is het beter gegaan.

  1. <?
  2. if($_GET['actie'] == "score")
  3. {
  4. $id = $_GET[id];
  5. //de score updaten
  6. if($_POST['score'] == "nuttig")
  7. {
  8. $update_score = mysql_query("UPDATE guides SET voted=voted+1 WHERE id='$id' AND actief='1'") or die(mysql_error());
  9. if(!$update_score)
  10. ...
  11. ?>
Offline Ultimatum - 24/09/2008 12:51
Avatar van Ultimatum PHP expert Dit block code vond ik net in een oud topic van mij .

  1. <?php
  2. include ("include/config.php");
  3.  
  4. $users = mysql_query("SELECT * FROM users");
  5. $members = mysql_num_rows($users);
  6. $ranks = mysql_query("SELECT * FROM ranks");
  7. $rank = mysql_num_rows($ranks);
  8.  
  9. if($members == "0"){
  10. $members = "We have no members!";
  11. }
  12. if($members == "1"){
  13. $members = "We have 1 member!";
  14. }
  15. else{
  16. $members = "We have ". $aantal ." members!";
  17. }
  18. ?>
  19. <p align="center"><?php echo ("$members"); ?></p>
  20. <table width="100%" border="1" cellpadding="0" cellspacing="0" bordercolor="#000000">
  21. <tr bgcolor="#CCCCCC">
  22. <td width="5%"><div align="center">id</div></td>
  23. <td width="25%"><div align="center">Username</div></td>
  24. <td width="25%"><div align="center">Email address</div></td>
  25. <td width="25%"><div align="center">Homepage</div></td>
  26. <td width="25%"><div align="center">Rank</div></td>
  27. </tr>
  28. <?php
  29. while($member_list = mysql_fetch_object($users)){
  30. $username = $member_list->username;
  31. $id = $member_list->id;
  32. $mail = $member_list->mail;
  33. $homepage = $member_list->homepage;
  34. while($rank_list = mysql_fetch_object($ranks)) {
  35. $rank = $rank_list->rank;
  36.  
  37. if(empty($homepage)){
  38. $homepage = "No homepage";
  39. }
  40. else{
  41. $homepage = "<a href=\"" . $homepage . "\" target=\"_blank\">$homepage</a>";
  42. }
  43. echo ("");
  44. echo ("<tr>");
  45. echo ("<td><a href=\"profiel.php?id=" . $id . "\" target=\"_blank\">$id</a></td>");
  46. echo ("<td>$username</td>");
  47. echo ("<td>$mail</td>");
  48. echo ("<td>$homepage</td>");
  49. echo ("<td>$rank</td>");
  50. echo ("</tr>");
  51. echo ("");
  52. }
  53. }
  54. ?>
  55. </table>


Schrik   
Offline Kr4nKz1n - 24/09/2008 12:51 (laatste wijziging 24/09/2008 12:58)
Avatar van Kr4nKz1n Onbekend
  1. <?php
  2. include("config.php");
  3. lay1();
  4. tabel1("Crew");
  5. echo "<table width=\"100%\" cellpadding=\"1\" cellspacing=\"0\" border=\"0\">
  6. <tr>
  7. <td colspan=\"4\" class=\"titel\"><b>Webmasters</b></td>
  8. </tr>";
  9. $paklid = mysql_query("SELECT * FROM leden WHERE level=6 AND activeer='' ORDER BY gebruikersnaam");
  10. $aantal = mysql_num_rows($paklid);
  11. if(!empty($aantal)){
  12. echo " <tr>
  13. <td><b>Gebruikersnaam</b></td>
  14. <td><b>Aangemeld op</b></td>
  15. <td><b>Geslacht</b></td>
  16. <td><b>Foto</b></td>
  17. </tr>";
  18. }
  19. else{
  20. echo " <tr>
  21. <td colspan=\"4\">Er zijn geen Webmasters.</td>
  22. </tr>";
  23. }
  24. while($gebruiker = mysql_fetch_object($paklid)){
  25. $aangemeld = $gebruiker->aangemeld;
  26. $gebruikersnaam = level("".$gebruiker->gebruikersnaam."");
  27. $geslacht = ifgeslacht($gebruiker->geslacht);
  28. $foto = iffoto($gebruiker->foto);
  29. echo " <tr>
  30. <td>".$gebruikersnaam."</td>
  31. <td>".$aangemeld."</td>
  32. <td>".$geslacht."</td>
  33. <td>".$foto."</td>
  34. </tr>";
  35. }
  36. echo " <tr>
  37. <td colspan=\"4\" class=\"titel\"><b>Moderators</b></td>
  38. </tr>";
  39.  
  40. $paklid = mysql_query("SELECT * FROM leden WHERE level=5 AND activeer='' ORDER BY gebruikersnaam");
  41. $aantal = mysql_num_rows($paklid);
  42. if(!empty($aantal)){
  43. echo " <tr>
  44. <td><b>Gebruikersnaam</b></td>
  45. <td><b>Aangemeld op</b></td>
  46. <td><b>Geslacht</b></td>
  47. <td><b>Foto</b></td>
  48. </tr>";
  49. }
  50. else{
  51. echo " <tr>
  52. <td colspan=\"4\">Er zijn geen Moderators.</td>
  53. </tr>";
  54. }
  55. while($gebruiker = mysql_fetch_object($paklid)){
  56. $aangemeld = $gebruiker->aangemeld;
  57. $gebruikersnaam = level("".$gebruiker->gebruikersnaam."");
  58. $geslacht = ifgeslacht($gebruiker->geslacht);
  59. $foto = iffoto($gebruiker->foto);
  60. echo " <tr>
  61. <td>".$gebruikersnaam."</td>
  62. <td>".$aangemeld."</td>
  63. <td>".$geslacht."</td>
  64. <td>".$foto."</td>
  65. </tr>";
  66. }
  67.  
  68. echo "</table>";
  69. tabel2();
  70. lay2();
  71. ?>


Crew-overzicht 

Plaatscode: 29124
Even een toelichting, dit is de admin pagina voor de gehele site 
Bannen van leden, nieuws berichten, poll`s, forum etc. 


Op zich best knap dat ik dat ooit zo heb gemaakt haha.
Offline Stijn - 24/09/2008 12:56
Avatar van Stijn PHP expert @ultimatum: lol, fetch object 
@kr4nkz1n: de spatie is die grote balk 
Offline Ontani - 24/09/2008 13:43
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
Deze week nog een stored procedure tegengekomen na het debuggen van de sql server omdat er iemand klaagde dat het traag ging.

Deze had 3 maanden gelopen en 13875750 Mb aan logfiles weggeschreven in diezelfde periode.

Weer een mooi staaltje scripting van een extern software pakket.
Offline thomasje - 24/09/2008 13:47
Avatar van thomasje MySQL beginner nou ik denk dat iedereen incl. ik zelf wel zo'n code zou kunnen posten van ons zelf. Is wel een leuk idee
Offline Kr4nKz1n - 24/09/2008 13:59
Avatar van Kr4nKz1n Onbekend Nou post dan 
Offline Sitebase - 25/09/2008 09:55
Avatar van Sitebase PHP expert  lol Ontani
Offline Berten - 25/09/2008 10:43 (laatste wijziging 25/09/2008 10:43)
Avatar van Berten PHP beginner Doorheen de applicatie waar ik nu aan werk wordt er op deze manier naar een String gecast:


Citaat:
String accoFound = "" + request.getAttribute("AccoFound");
Offline Ontani - 25/09/2008 10:52
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
Het laatst in een applicatie waar ik aan meewerk (vb.net)

Option Strict On

Bovenaan bijgetypt, dit zorgde ervoor dat mijn error venstertje in vb.net ineens de melding kreeg: Maximum number of errors has been exceeded.

Dus ik de code van m'n collega gaan doorspitten en kwam ik op dergelijke formules:

  1. Public Function BerekenOppervlakte(Byval strLengte as String, Byval strBreedte As String) As Decimal
  2. Dim strOppervlakte as String = 0
  3. strOppervlakte = strLengte * strBreedte
  4. return strOppervlakte
  5. End Function


Je kan al raden waarom dit ineens een aantal errors zou verroorzaken 

Offline Stijn - 25/09/2008 11:44
Avatar van Stijn PHP expert lol 
Offline Kr4nKz1n - 25/09/2008 12:04
Avatar van Kr4nKz1n Onbekend Er mag wel meer code gepost worden mensen 
Offline roberini - 25/09/2008 17:02
Avatar van roberini HTML beginner
  1. <?php /* ------------------------- */
  2.  
  3. $page = $_GET['p'];
  4. if($page == "doneet" || $page == "deleet" || $page == "reset")
  5. include("admin-beesik.php");
  6. else if($page == "masmessutjs" || $page == "adminmessutjs")
  7. include("admin-messutjs.php");
  8. else if($page == "multie" || $page == "seertsj")
  9. include("admin-seertsj.php");
  10. else if($page == "stets")
  11. include("admin-euserinfo.php");
  12. else if($page == "rechtbank")
  13. include("admin-rechtbank.php");
  14. else if($page == "pol")
  15. include("admin-pol.php");
  16. else if($page == "ectif" || $page == "betaald" || $page == "berichten")
  17. include("adminstets.php");
  18.  
  19. /* ------------------------- */ ?>


Stukje door iemand gemaakt met dyslectie. Ik heb hem geholpen met zijn code netjes te maken
Offline vinTage - 25/09/2008 19:43
Avatar van vinTage Nieuw lid Hoppa, mn eerste html meuk online 

Ik was daarvoor al wel met flash bezig, maar nog niet echt met actionscript (behalve de stopjes en gotoAnd...'s natuurlijk)
Offline Ontani - 25/09/2008 19:50
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
Haha lol, is zelfs zo lang geleden dat je op die foto bij de pagina "genodigden" er zelfs normaal uitzag 
Offline vinTage - 25/09/2008 19:54
Avatar van vinTage Nieuw lid Toen was ik nog geen alcoholist 
Offline Kr4nKz1n - 26/09/2008 08:53
Avatar van Kr4nKz1n Onbekend vinTage? Ik zie de </body> nog een table 

@roberini: Afgezien van het taal-gebruik, valt het best mee.
Offline xSc - 26/09/2008 10:50
Avatar van xSc Onbekend Lol, as String = 0 

Je wil een decimal returnen, maar je returnt een string.
Offline Ontani - 26/09/2008 10:55
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
Is niet het enige probleem.

strLengte en strBreedte zijn beide strings (Parameters).
op die strings word een wiskundige bewerking uitgevoerd en het resultaat terug in een string gestoken.

en dan wat jij zegt: een decimal return maar een string meegeven.
Offline Chilion - 26/09/2008 11:04
Avatar van Chilion MySQL interesse Gefeliciteerd met je trouwen VinTage 
Offline nemesiskoen - 26/09/2008 11:58 (laatste wijziging 26/09/2008 11:59)
Avatar van nemesiskoen Gouden medaille

PHP expert
  1. function evaluate($post, $get, $s){ //process answers for every question on the page
  2. $surveyMan = new SurveyManager();
  3. $m = '';
  4. $questionlist = $s->getQuestionList($post['curpage']);
  5. if($questionlist != -1){
  6. foreach($questionlist as $curQuestion){
  7. if($_GET['prev'] != 'true'){
  8. $subm = $curQuestion->processAnswers($post, $get, $s->getMode(), $s);
  9. }
  10. if($subm != ''){
  11. $m .= $subm.'<br />';
  12. }
  13. }
  14. }else{
  15. echo $languageMan->getMessageString($s, 'sur_echo');
  16. // yields error because languageman is not defined inside this function
  17. }
  18. return $m;
  19. }

$post en $get zijn gelijk aan $_POST en $_GET, onaangepast:-)
wordt dus zo aangeroepen:
  1. evaluate($_POST, $_GET, $nogeenvariabele);


onleesbaar dus voor iemand die niet eerst alle files volledig heeft doorgespit... Allas voor mij moest ik hier dus een hoop aanpassingen in maken, en moest ik dus daadwerkelijk honderden klassen gaan overlopen voordat ik ook maar enigzins wist welke klasse welke verantwoordelijkheid had:s

Die onderste commentaar heb ik er bijgeschreven omdat dit dus onzin was en dat de eigenaar van dit script dit kon zien en hier iets aan doen:-)
Offline marten - 21/10/2008 15:21
Avatar van marten Beheerder Is er zo weinig slechte code?
Offline Ontani - 21/10/2008 15:25
Avatar van Ontani Gouden medailleGouden medailleGouden medailleGouden medaille

-1
  1. Control.CheckForIllegalCrossThreadCalls = False


Prachtig lijntje, zorgt er gewoon voor dat controls uit verschillende threads kunnen communiceren met andere threads zonder dat er fouten gegenereerd worden.

Offline marten - 23/10/2008 11:43 (laatste wijziging 23/10/2008 11:51)
Avatar van marten Beheerder
  1. if(($_SERVER['REQUEST_METHOD'] == "POST") && isset($_POST['done']) && ($_POST['done'] == 'true'))
  2. { //-- part 3
  3. if(isset($_GET['naam']) && $_GET['naam'] != ''){ $_SESSION['naam'] = $_GET['naam']; }
  4. {}}


Deze kom ik net tegen niet mijn eigen code 
Gesponsorde links
Dit onderwerp is gesloten.
Actieve forumberichten
© 2002-2024 Sitemasters.be - Regels - Laadtijd: 0.303s