По колено в коде — Говнокод.ру

PHP / Говнокод #18159

+146

1
2
3
4
5
6
7
8

if(isset($_GET['go']))
	$go = htmlspecialchars(strip_tags(stripslashes(trim(urldecode(mysql_escape_string($_GET['go']))))));
else
	$go = "main";

// ....

switch($go){

"Тыыыыыыы не пройдеееешь!". Зафильтровали (да ещё и криво) бедную переменную только для того, чтобы потом использовать её в switch-case блоке. Индусы-паникеры.

bSun0000, 13 Мая 2015

Комментарии (7)

JavaScript / Говнокод #18158

+142

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28

(function($){

  function pickMenuByHash(hash) {

    if(hash == undefined || hash == '' || hash == '#')
      hash = '#home'; // хеш по умолчанию

    // Выбираем ссылки с нужным хешем, чтро бы потом присвоить им класс "active"
    var A_tags = $('a[href="' + hash + '"]');
    
    // Ищем все ссылки в родительском теге UL относительно ссылок и удаляем у них классы "active"
    A_tags.parents('UL').find('a').removeClass('active');

    // Добавляем к ссылкам с нужным нам хешем класс "active"
    $('a[href="' + hash + '"]').addClass('active');
  }

  // Обновить состояние менюшек (сверху и справа) при загрузке страницы
  $(function(){
    pickMenuByHash(document.location.hash);
  });

  // При изменении хеша обновить состояния менюшек (сверху и справа)
  $(window).on('hashchange', function(){
    pickMenuByHash(document.location.hash);
  });

})(jQuery);

Вот тебе при изменении хеша на сайте верхнее и правое меню будет в актуальном для хеша состоянии.

littlefuntik, 13 Мая 2015

Комментарии (22)

Си / Говнокод #18157

+141

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22

const char *pS;
		char *pD;

		// clear out new string
		szNew[0] = '\0';

		pS = pszValue;
		pD = szNew;

		// step through the string, only copying back in characters that are printable
		while( *pS )
		{
			if( ((byte)*pS) < 32 || ((byte)*pS) > 255 )
			{
				pS++;
				continue;
			}
			*pD++ = *pS++;
		}

		// terminate the new string
		*pD = '\0';

Мало того, char в проекте подразумевается signed

mittorn, 12 Мая 2015

Комментарии (0)

PHP / Говнокод #18156

+143

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

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']));

Мдэээ...

Dev_18, 12 Мая 2015

Комментарии (13)

PHP / Говнокод #18155

+140

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24

<?php
/**
  * @desc Локализация сайта
  *
  **/
class System_Message{
  private function openLangFail($lang){
    $path = '/../message/'.$lang.'.php';
    if (file_exists($path)) {
      return 'File with languages not found';
    } else {
      return include $path; 
    }
  } 
  
  public function getLang($value, $lang='ru'){
    $lang = self::openLangFail($lang);
    if($lang[$value] != '') {
      return $lang[$value];
    } else {
	  return false;
	}
  }
}

Ну как?

Dev_18, 12 Мая 2015

Комментарии (9)

C# / Говнокод #18154

+143

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18

public enum WebViewImageType {
	Back = 0,
	Close = 1,
	Menu = 2
}		

WebViewImageType GetImageType(int jsType){ 
	switch (jsType) {
		case 0:
			return WebViewImageType.Back;
		case 1:
			return WebViewImageType.Close;
		case 2:
			return WebViewImageType.Menu;
		default:
			return WebViewImageType.Back;
	}
}

kjuby8709gsome, 12 Мая 2015

Комментарии (0)

C++ / Говнокод #18153

+48

1

auto filename = std::string{};

laMer007, 12 Мая 2015

Комментарии (157)

PHP / Говнокод #18152

+165

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17

// Whoever introduced xml to shipping companies should be flogged
$xml  = '<?xml version="1.0"?>';
$xml .= '<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ns1="http://fedex.com/ws/rate/v10">';
$xml .= '	<SOAP-ENV:Body>';
$xml .= '		<ns1:RateRequest>';
$xml .= '			<ns1:WebAuthenticationDetail>';

// почти сотня строк

$xml .= '</SOAP-ENV:Envelope>';
$curl = curl_init($url);
curl_setopt($curl, CURLOPT_POST, true);
curl_setopt($curl, CURLOPT_POSTFIELDS, $xml);
curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
curl_setopt($curl, CURLOPT_HEADER, false);
curl_setopt($curl, CURLOPT_TIMEOUT, 30);
curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, false);

Смотри-ка, оно еще и ругается

https://github.com/opencart/opencart/blob/58eeb544e2eddbe18b00224e10d2761259e8ce7b/upload/catalog/model/shipping/fedex.php#L48

Fike, 12 Мая 2015

Комментарии (77)

JavaScript / Говнокод #18151

+145

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45

var errorFirst = true;
		var errorSec   = true;
		
		if(errFirstV!==false && errFirstN!==false){
			errorFirst=true;
		}
		else if (errFirstV===false && errFirstN!==false) {
			
			errorFirst=false;
			
		}
		else if (errFirstV===false && errFirstN===false) {
			
			errorFirst=false;
			
		}
		else if (errFirstV!==false && errFirstN===false) {
			
			errorFirst=false;
			
		}
		
		
		if(errFirstT!==false && errFirstE!==false){
		
			errorSec=true;
		}
		else if(errFirstT===false && errFirstE!==false){
		
			errorSec=false;
		}
		else if(errFirstT===false && errFirstE===false){
		
			errorSec=false;
		}
		else if(errFirstT!==false && errFirstE===false){
		
			errorSec=false;
		}
		
		// check both lines
		
		if (errorFirst !==true && errorSec !== true) {
			error=false;
		}

ЭТО шЭдэвр!!!!

armengabriel, 12 Мая 2015