1. PHP / Говнокод #27634

    0

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    switch ($group) {
        case 'Root':
            break;
        case 'Admin':
            break;
        case 'Accountant':
            break;
        case 'Manager':
            break;
        }

return $group;

    Вот так можно проверить что ничего не надо делать

    Запостил: zoorg, 01 Сентября 2021

    Комментарии (10) RSS

    • ava CEHT9I6PbCKuu_nemyx 01.09.2021 11:17 # +2

      $group может быть экземпляром класса с магическим методом __toString, производящим побочные эффекты.
      Ответить

    • ava Amayak_Akopyan 01.09.2021 11:42 # +2

      А теперь я вам покажу немного уличной магии: 
      <?php

function kukarek($group) {
    switch ($group) {
        case 'Root':
            break;
        case 'Admin':
            break;
        case 'Accountant':
            break;
        case 'Manager':
            break;
        }

    return $group;
}

class Pituh {
    public $count = 0;
    private $name;
    public function __construct($initial) {
        $this->name = $initial;
    }
    public function __toString() {
        ++$this->count;
        return $this->name;
    }
}

$group = new Pituh('Admin');

kukarek($group);

echo "Меня посчитали $group->count раз(а).\n";


      https://ideone.com/TbzIwf

      В принципе, можно было ещё склонялку слов добавить, но для демонстрации пойдёт.

      Именно поэтому «PHP» — рахат-лукум моей души.
      Ответить

      • ava guest6 01.09.2021 11:57 # 0

        Менеджера посчитали больше. Он важнее.
        https://ideone.com/LvQsJQ

        Туштринг на каждом кейсе без кэширования? Какой багор )))
        Ответить

        • ava guest6 01.09.2021 12:05 # +1

          Напесал класс, который уворачивается от сравнения:
          https://ideone.com/bfYN4z
          Ответить

          • ava Amayak_Akopyan 01.09.2021 12:09 # +1

            Какой скилл )))
            Ответить

            • ava guest6 02.09.2021 13:07 # +2 

              <?php

class SchrodingersPituh {
    public $count = 0;
    public function __construct() {
    }
    public function __toString() {
    	$this->count ^= 1;
    	return $this->count ? '' : '1';
    }
}

function kukarek($pituh) {
	if (!"$pituh") {
		echo "Ne pituh\n";
	}
	if ("$pituh") {
		echo "Pituh\n";
	}
}

kukarek(new SchrodingersPituh());


              https://ideone.com/FsmdvI
              Ответить

        • ava Amayak_Akopyan 01.09.2021 12:07 # 0

          Угу. «PHP» не знает, с чем будут сравнивать значение, поэтому на каждом кейсе производит преобразование типов индивидуально. Хотя и так ясно, что в ветках «case» будут либо строки, либо числа, либо булины, а сравнение объектов с последними (в смысле с числами и с булинами) можно произвести только через «туштринг», так что результат вызова «туштринг» можно было бы и закэшировать.

          Хорошо хоть, что выражение из заголовка switch() вычисляется один раз.

          Именно поэтому «PHP» — щербет моего сердца.
          Ответить

        • ava Amayak_Akopyan 01.09.2021 12:36 # 0

          А в «JavaScript» похоже, что switch...case производит строгое сравнение (через «===», а не через «==»), поэтому toString() не вызывается:
          https://ideone.com/NET1vN
          Ответить

      • ava guest6 01.09.2021 12:09 # 0

        > склонялку слов добавить
        odin raz, odip rba, odiq rbe, odir rbi, odis rbo, odit rbu, odiv rca, odiw rce, odix rci, odiy rco, odiz rcu, odob rda, odoc rde, odod rdi, odof rdo, odog rdu, odoh reb, odoj rec, odok red
        Ответить

    • ava guest6 01.09.2021 17:15 # 0

      проверяю отпраку коментов в говнокодах :)
      Ответить

    Добавить комментарий