1. Список говнокодов пользователя v_anonym

    Всего: 4

  2. PHP / Говнокод #13521

    +158

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    <?php
class Digits {
  //Convert digits to negative
  public static function toNegative($num) {
    return '-' . $num;
  }
}

    CRM система, очень дорогая
    ПХП хоть само по себе стремное, но я бы даже до такого не додумался
    Основная функция системы: анализ и рассчеты
    Они еще жалуются что оно плохо считает
    Интересно а что же будет если ему передать -10 например ? --10 :)
    Чем не устраивает:
    function toNegative($num) {
    return ($num > 0) ? -1 * $num : $num;
    }

    v_anonym, 31 Июля 2013

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

  3. PHP / Говнокод #12659

    +146

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    class Step {
  public function xxx($arg1, $arg2 = NULL) {

  };
}

class FirstStep extends Step {
  public function xxx($arg3, $arg1 = NULL, $arg2 = NULL) {

  };
}

    В очередной раз подтверждает что пхп - гамно!
    А автору кода вообще линейкой по пальцам, куда, как, зачем менять сигнатуру да и еще не правильно!!!
    И это в проэкте каторый стоит 15000Евро

    v_anonym, 27 Февраля 2013

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

  4. PHP / Говнокод #12319

    +54

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    function validEmail($email)
{
	$isValid = true;
	$atIndex = strrpos($email, "@");
	if (is_bool($atIndex) && !$atIndex) {
		$isValid = false;
	} else {
		$domain = substr($email, $atIndex + 1);
		$local = substr($email, 0, $atIndex);
		$localLen = strlen($local);
		$domainLen = strlen($domain);
		if ($localLen < 1 || $localLen > 64) {
			// local part length exceeded
			$isValid = false;
		} else if ($domainLen < 1 || $domainLen > 255) {
			// domain part length exceeded
			$isValid = false;
		} else if (!strrpos($domain, ".")) {
			// domain part does not have .
			$isValid = false;
		} else if ($domain[0] == '.' || $domain[$domainLen - 1] == '.') {
			// domain part starts or ends with '.'
			$isValid = false;
		} else if ($local[0] == '.' || $local[$localLen - 1] == '.') {
			// local part starts or ends with '.'
			$isValid = false;
		} else if (preg_match('/\\.\\./', $local)) {
			// local part has two consecutive dots
			$isValid = false;
		} else if (!preg_match('/^[A-Za-z0-9\\-\\.]+$/', $domain)) {
			// character not valid in domain part
			$isValid = false;
		} else if (preg_match('/\\.\\./', $domain)) {
			// domain part has two consecutive dots
			$isValid = false;
		} else if (!preg_match('/^(\\\\.|[A-Za-z0-9!#%&`_=\\/$\'*+?^{}|~.-])+$/', str_replace("\\\\", "", $local))) {
			// character not valid in local part unless 
			// local part is quoted
			if (!preg_match('/^"(\\\\"|[^"])+"$/', str_replace("\\\\", "", $local))) {
				$isValid = false;
			}
		}
		/* if ($isValid && !(checkdnsrr($domain,"MX") || checkdnsrr($domain,"A"))) {
		  // domain not found in DNS
		  $isValid = false;
		  } */
	}
	return $isValid;
}

    Интересно а автор когдата слишал про filter_var?
    Это все можна заменить на filter_var($input, FILTER_VALIDATE_EMAIL);
    Товаришь программист! Не умеешь срать - не мучай жопу!

    v_anonym, 19 Декабря 2012

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

  5. PHP / Говнокод #12318

    +39

    1. 1
    2. 2
    3. 3
    4. 4
    function getTextLabel($labelName)
{
	return $labelName;
}

    Зачем плодить такие фейки?

    v_anonym, 19 Декабря 2012

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