Jak se to nedělá

kyborek · 2012-03-15 18:20:21

Zdravím všechny, při své práci jsem narazil na kusy kódu které mě opravdu zaráží ale jeden mě tak vytočil že jsem se rozhodl poučit svět o tom jak se to nedělá. Vždy když na něco narazím tak to sem vložím pro vaše pobavení/poučení a pokud pochopíte že kusy kódu tady jsou totální nesmysl tak jste na dobré cestě.
Pokud sami najdete nesmyslný kód někoho cizího budu rád když sem přispějete

pro začátek kus html/php:

<h1><? echo"Objednávky"; ?></h1>

Lkopo · 2012-03-17 21:11:38

Nedávať na koniec PHP skriptu (resp. koniec súboru) znak:

?>

môže to spôsobovať zbytočné chyby ak náhodou bude pod daným znakom medzera.

Ďalej bežná PHP chyba

if($a = $b) ...

Martin · 2012-05-24 15:53:23

Sice už to zde asi pošlo, ale narazil jsem na lejnokód. Doporučuji...

Lkopo · 2012-05-27 11:30:55

bartama napsal:

Pro pobavení pěkný, jinak ještě k předchozímu topicu, přiznám se, PHP dělám cca 5let a i tak jsem včera při programování udělal if($x = $y) člověk prostě občas zapomene a pak nadává, kde je zase chyba.

viď moja prvá správa

Matej · 2012-06-17 22:24:35

Lkopo napsal:

Ďalej bežná PHP chyba
if($a = $b) ...

Toto vie niekedy aj na pár desiatok minút znepríjemneniť deň, ale v závislosti od kontextu to chyba nevyhnutne byť nemusí. Ja osobne priraďovanie hodnoty v podmienke používam pomerne často 8-)

kksmirice · 2012-07-16 20:54:30

Tak také přispěji výpisem z kódu, který mne pobavil:

<?php
...
if ($pocet == 0)
   {
   $zbozi1_1 = '<tr><td colspan="4" class="prazdny_kosik">Ve Vašem košíku ještě není žádné zboží. Zboží se přidává do košíku kliknutím na tlačítko "Přidat zboží do košíku".</td></tr>';
   }

?>
<?php echo $zbozi1_1 ?>
<?php echo $zbozi2_2 ?>
<?php echo $zbozi3_3 ?>
<?php echo $zbozi4_4 ?>
<?php echo $zbozi5_5 ?>
<?php echo $zbozi6_6 ?>
<?php echo $zbozi7_7 ?>
<?php echo $zbozi8_8 ?>
<?php echo $zbozi9_9 ?>
<?php echo $zbozi10_10 ?>
<tr>
<td height="10px" colspan="4"></td>
</tr><?php
if ($pocet != 0)
{
echo '<tr>';
echo '<td height="20px" width="100%" colspan="4" align="right" class="kosik_dokoncit"><a href="./index.php?s=login_1">Přejít k pokladně</a></td>';
echo '</tr>';
}
?>
<tr>
<td height="470px" colspan="4"></td>
</tr>
</table>

kyborek · 2012-07-29 10:20:16

Budu hádat, při veselém objednávání to člověku najednou napíše: "Je nám líto, ale z technických důvodů můžete mít v košíku nejvýše deset položek"

kksmirice · 2012-09-02 09:06:29

Jeden z dalších php kódů, tedy podle zápisu v souboru se spíše jedná o html
přehledný zápis...

...

?>

<h2><?php echo LANG_OPTIONS; ?></h2>

<ul id="tab-bar">
<li<?php if ($_GET["tab"]=="") { echo " class=\"active-item\""; } ?>><a href="./admin.php?function=options"><?php echo LANG_SETTINGS; ?></a></li>
<li<?php if ($_GET["tab"]=="access") { echo " class=\"active-item\""; } ?>><a href="./admin.php?function=options&tab=access"><?php echo LANG_ACCESS; ?></a></li>
<li<?php if ($_GET["tab"]=="meta_tags") { echo " class=\"active-item\""; } ?>><a href="./admin.php?function=options&tab=meta_tags"><?php echo LANG_META_TAGS; ?></a></li>
<li><span class="float-ending"><!-- --></span></li>
</ul>


<?php if ($_GET["tab"]==""): ?>
...

A včul mudruj

Martin · 2012-09-02 10:05:31

No já nevím, možná ještě spím, ale nevidím na tom nic špatného. Ba právě naopak...

Martin · 2012-12-23 14:55:23

Tento brutální dotaz jsem našel v jedné práci... Otázka za sto tisíc. Copak asi dělá?

/* !!! */
	public function updateSelect($where, $material, $pocet, $vykres, $pozice, $delka, $sirka, $poznamka, $limit) {
		return $this->dibi->query('UPDATE data
									SET %if', $material!=="",' material=%s', $material,' %end %if ',$material!==""&&($pocet!==""||$vykres!==""||$pozice!==""||$delka!==""||$sirka!==""||$poznamka!==""),',%end
										%if', $pocet!=="",' pocet=%i', $pocet,' %end %if ',$pocet!==""&&($vykres!==""||$pozice!==""||$delka!==""||$sirka!==""||$poznamka!==""),',%end
										%if', $vykres!=="",' vykres=%s', $vykres,' %end %if ',$vykres!==""&&($pozice!==""||$delka!==""||$sirka!==""||$poznamka!==""),',%end
										%if', $pozice!=="",' pozice=%s', $pozice,' %end %if ',$pozice!==""&&($delka!==""||$sirka!==""||$poznamka!==""),',%end
										%if', $delka!=="",' delka=%i', $delka,' %end %if ',$delka!==""&&($sirka!==""||$poznamka!==""),',%end
										%if', $sirka!=="",' sirka=%i', $sirka,' %end %if ',$sirka!==""&&($poznamka!==""),',%end
										%if', $poznamka!=="",' poznamka=%s', $poznamka,' %end
									WHERE %and', $where,'
									%if',isset($limit),'LIMIT %i',$limit,'%end');
	}

(ani se mi ho sem nedaří pořádně zapsat)
Vzhledem k tomu, že nikdo neuhádne co dělá, mohu pro zajímavost prozradit, že funguje...

#1 2012-03-15 18:20:21

Re: Jak se to nedělá

#2 2012-03-17 21:11:38

Re: Jak se to nedělá

#3 2012-05-24 15:53:23

Re: Jak se to nedělá

#4 2012-05-27 11:30:55

Re: Jak se to nedělá

#5 2012-06-17 22:24:35

Re: Jak se to nedělá

#6 2012-07-16 20:54:30

Re: Jak se to nedělá

#7 2012-07-29 10:20:16

Re: Jak se to nedělá

#8 2012-09-02 09:06:29

Re: Jak se to nedělá

#9 2012-09-02 10:05:31

Re: Jak se to nedělá

#10 2012-12-23 14:55:23

Re: Jak se to nedělá

Zápatí