Mistři v programování na #hovnokod

Bezpecnosti neni nikdy dost

if (isset($this->input['variant']) && $this->input['variant'] == "1") {
    if (!in_array($this->input['variant'], [0, 1])) {
        throw new SecurityBrokenException;
    }
}

Sleva asi nebude...

    public function getDiscount()
    {
        return 0;
    }

Jak elegantne ziskat unix timestamp. Pravidelne se objevuje v projektu co ma 250k radku.

$celedatum = Date("Y-m-d H:i:s");
$celedatum1 = strtotime($celedatum);

nalezeno v jednom CMS. Nejen že to slavné "řazení", řešené dvojitou smyčkou, lze udělat už v SQL dotazu, ale hlavně, pokud $data obsahují záznamy s neexistujícím ID, tak se to hezky zacyklí :)

/**
 * select pictures src by ids
 * @param array $data
 * @return multitype:|multitype:unknown
 */
private function modelSelectPicturesSrcByIds(array $data) {
    if (empty($data)) {
        return array();
    } else {
        foreach ($data as &$tmp) {
            $tmp = (int) $tmp;
        }

        $ids = implode(',', $data);

        $result = $this->env->db->selectAssocList("SELECT * FROM {$this->table_file_p} WHERE id in ({$ids})");

        //order by id
        $return = array();
        $data = array_reverse($data);
        $currentId = array_pop($data);
        if ($result) {
            while ($currentId != null) {
                foreach ($result as $row) {
                    if ($row['id'] == $currentId) {
                        $return[] = $row;
                        $currentId = array_pop($data);
                        break;
                    }
                }
            }
        }

        return $return;
    }
}

najdi rozdíl

    if(element.data("max")) {
            helps = $(this).closest(element).find("input").val();
            newVal = Number(helps)+1;

            $(this).closest(element).find("input").val(newVal).trigger('change');

    }else {
            helps = $(this).closest(element).find("input").val();
            newVal = Number(helps)+1;
            $(this).closest(element).find("input").val(newVal).trigger('change');
    }