- 1
- 2
- 3
- 4
- 5
if ($ipaddr == $ip_reverse) {
return true;
} else {
return false;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+144
if ($ipaddr == $ip_reverse) {
return true;
} else {
return false;
}
https://github.com/pfsense/pfsense/blob/master/etc/inc/util.inc#L625-L629
Давно хотел спросить: это как-нибудь здравым смыслом оправдывается? Часто встречаю похожее, когда чужой код смотрю. Может, это профи делают для… читаемости какой-то, я не знаю. Или я ищу хорошее там, где его нет?
+143
if (is_array($search_ct_properties->getEngines()))
{
$where_engines = array();
foreach ($search_ct_properties->getEngines() as $engine)
{
if ($engine == 5)
{
$where_engines[] = 'ct_modifications.fuel_id != 6';
}
else
{
$where_engines[] = 'ct_modifications.fuel_id = 6';
}
}
$where[] = '(' . implode(' OR ', $where_engines) . ')';
}
если 5, то это точно не 6!
+141
$page = strtoupper($_SERVER['REQUEST_URI']);
if (strpos($page, "%D0%A2%D0%BE%D0%BF10")){
echo "<li class=\"active\"><a href=\"/%D0%A2%D0%BE%D0%BF10\">Топ-10</a></li>";
} else {
echo "<li><a href=\"/%D0%A2%D0%BE%D0%BF10\">Топ-10</a></li>";
}
if (strpos($page, "%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA")){
echo "<li class=\"active\"><a href=\"/%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA\">Трафик</a></li>";
} else {
echo "<li><a href=\"/%D0%A2%D1%80%D0%B0%D1%84%D0%B8%D0%BA\">Трафик</a></li>";
}
if (strpos($page, "%D0%9B%D0%B8%D0%B4%D0%B5%D1%80")) {
echo "<li class=\"active\"><a href=\"/%D0%9B%D0%B8%D0%B4%D0%B5%D1%80\">Лидер</a></li>";
} else {
echo "<li><a href=\"/%D0%9B%D0%B8%D0%B4%D0%B5%D1%80\">Лидер</a></li>";
}
...а кроме того, в данном случае strtoupper бесполезен
+145
<?
for ($i = 1; $i <= 100; $i++) {
$t1 = $i / 3;
$t2 = $i / 5;
if (preg_match("/\./", "$t1") == false && preg_match("/\./", "$t2") == false) {
echo "FizzBuzz<br>";
} else {
if (preg_match("/\./", "$t1") == false) {
echo "Fizz<br>";
} else {
if (preg_match("/\./", "$t2") == false) {
echo "Buzz<br>";
} else {
echo "$i<br>";
}
}
}
}
?>
ZF...
+141
$phsms = '';
for($p=0;$p<strlen($phonesms);$p++)
if(intval($phonesms[$p]) >= 0) $phsms = $phsms.intval($phonesms[$p]);
preg_match('/.*(9[0-9]{2})([0-9]{7})/', $phsms, $match2);
$phsms = $match2[1].$match2[2];
Валидируем телефоны.
+143
function buildUpdateTimestampTrigger($tableName) {
return
sprintf(
file_get_contents(SQL_FILES_PATH . 'updateTimestamp_PlPg.sql'),
strtolower($tableName) . '_updated'
) .
sprintf(
file_get_contents(SQL_FILES_PATH . 'updateTimestampTrigger.sql'),
$tableName
);
}
+144
$orig_id = isset($_POST["orig_id"]) ? (int) $_POST["orig_id"] : (int) $_GET["orig_id"];
https://github.com/uisky/notabenoid/blob/6edfee9f33b8466be9164cd3ba6be90b8f3a5770/protected/controllers/MyCommentsController.php#L123
+142
<?php
class ModelCheckoutOrder extends Model {
public function addOrder($data) {
$this->event->trigger('pre.order.add', $data);
$this->db->query("INSERT INTO `" . DB_PREFIX . "order` SET invoice_prefix = '" . $this->db->escape($data['invoice_prefix']) . "', store_id = '" . (int)$data['store_id'] . "', store_name = '" . $this->db->escape($data['store_name']) . "', store_url = '" . $this->db->escape($data['store_url']) . "', customer_id = '" . (int)$data['customer_id'] . "', customer_group_id = '" . (int)$data['customer_group_id'] . "', firstname = '" . $this->db->escape($data['firstname']) . "', lastname = '" . $this->db->escape($data['lastname']) . "', email = '" . $this->db->escape($data['email']) . "', telephone = '" . $this->db->escape($data['telephone']) . "', fax = '" . $this->db->escape($data['fax']) . "', custom_field = '" . $this->db->escape(isset($data['custom_field']) ? serialize($data['custom_field']) : '') . "', payment_firstname = '" . $this->db->escape($data['payment_firstname']) . "', payment_lastname = '" . $this->db->escape($data['payment_lastname']) . "', payment_company = '" . $this->db->escape($data['payment_company']) . "', payment_address_1 = '" . $this->db->escape($data['payment_address_1']) . "', payment_address_2 = '" . $this->db->escape($data['payment_address_2']) . "', payment_city = '" . $this->db->escape($data['payment_city']) . "', payment_postcode = '" . $this->db->escape($data['payment_postcode']) . "', payment_country = '" . $this->db->escape($data['payment_country']) . "', payment_country_id = '" . (int)$data['payment_country_id'] . "', payment_zone = '" . $this->db->escape($data['payment_zone']) . "', payment_zone_id = '" . (int)$data['payment_zone_id'] . "', payment_address_format = '" . $this->db->escape($data['payment_address_format']) . "', payment_custom_field = '" . $this->db->escape(isset($data['payment_custom_field']) ? serialize($data['payment_custom_field']) : '') . "', payment_method = '" . $this->db->escape($data['payment_method']) . "', payment_code = '" . $this->db->escape($data['payment_code']) . "', shipping_firstname = '" . $this->db->escape($data['shipping_firstname']) . "', shipping_lastname = '" . $this->db->escape($data['shipping_lastname']) . "', shipping_company = '" . $this->db->escape($data['shipping_company']) . "', shipping_address_1 = '" . $this->db->escape($data['shipping_address_1']) . "', shipping_address_2 = '" . $this->db->escape($data['shipping_address_2']) . "', shipping_city = '" . $this->db->escape($data['shipping_city']) . "', shipping_postcode = '" . $this->db->escape($data['shipping_postcode']) . "', shipping_country = '" . $this->db->escape($data['shipping_country']) . "', shipping_country_id = '" . (int)$data['shipping_country_id'] . "', shipping_zone = '" . $this->db->escape($data['shipping_zone']) . "', shipping_zone_id = '" . (int)$data['shipping_zone_id'] . "', shipping_address_format = '" . $this->db->escape($data['shipping_address_format']) . "', shipping_custom_field = '" . $this->db->escape(isset($data['shipping_custom_field']) ? serialize($data['shipping_custom_field']) : '') . "', shipping_method = '" . $this->db->escape($data['shipping_method']) . "', shipping_code = '" . $this->db->escape($data['shipping_code']) . "', comment = '" . $this->db->escape($data['comment']) . "', total = '" . (float)$data['total'] . "', affiliate_id = '" . (int)$data['affiliate_id'] . "', commission = '" . (float)$data['commission'] . "', marketing_id = '" . (int)$data['marketing_id'] . "', tracking = '" . $this->db->escape($data['tracking']) . "', language_id = '" . (int)$data['language_id'] . "', currency_id = '" . (int)$data['currency_id'] . "', currency_code = '" . $this->db->escape($data['currency_code']) . "', currency_value = '" . (float)$data['currency_value'] . "', ip = '" . $this->db->escape($data['ip']) . "', forwarded_ip = '" . $this->db->escape($data['forwarded_ip']) . "', user_agent = '" . $this->db->escape($data['user_agent']) . "', accept_language = '" . $this->db->escape($data['accept_language']) . "', date_added = NOW(), date_modified = NOW()");
Я вас еще немного помучаю, хорошо?
3978 символов, считая отступы
https://github.com/opencart/opencart/blob/d5f66b0d75a5adf35815e333e45598bbc38750d0/upload/catalog/model/checkout/order.php#L6
+146
if(isset($_GET['go']))
$go = htmlspecialchars(strip_tags(stripslashes(trim(urldecode(mysql_escape_string($_GET['go']))))));
else
$go = "main";
// ....
switch($go){
"Тыыыыыыы не пройдеееешь!". Зафильтровали (да ещё и криво) бедную переменную только для того, чтобы потом использовать её в switch-case блоке. Индусы-паникеры.
+143
if(!$user['email']) {
$err = $this->system_message->getLang('empty_email');
} elseif(strlen($user['email']) < self::MIN_USER_NAME) {
$err = $this->system_message->getLang('small_email');
} elseif(strlen($user['email']) > self::MAX_USER_NAME) {
$err = $this->system_message->getLang('long_email');
} elseif(!filter_var($user['email'], FILTER_VALIDATE_EMAIL)) {
$err = $this->system_message->getLang('incorrect_name');
}
if(!$user['passw']) {
$err = $this->system_message->getLang('empty_passw');
} elseif(strlen($user['passw']) < self::MIN_PASSWORD_NAME) {
$err = $this->system_message->getLang('small_passw');
} elseif(strlen($user['passw']) > self::MAX_PASSWORD_NAME) {
$err = $this->system_message->getLang('long_passw');
}
if($err) {
//!TODO add a exeption
} else {
$find_email = $this->db_users->prepare("SELECT * FROM `users` WHERE `email` = :email");
$find_email->execute(array(':email' => $user['email']));
Мдэээ...