Почему мне кажется, что это говнокод?

Question

[Константин Кавенский]

Вот, собственно, накодилось, пока отдыхал от дел, но чето у меня подозрения что это можно сделать... элегантнее, что ли?

<?php
$userDice=mt_rand(1,6);
$computerDice=mt_rand(1,6);
if ($userDice > $computerDice)
echo "У юзера выпало $userDice, а у компьютера $computerDice. Выигрыш юзера";
else
if ($userDice < $computerDice)
echo "У компьютера выпало $computerDice, а у юзера $userDice. Выигрыш компьютера";
else
if ($userDice == $computerDice)
echo ("У юзера выпало $userDice, а у компьютера $computerDice. Ничья!");
?>

Answer 1

[Сергей delphinpro]

Держи еще вариант ГК

$message = 'У юзера выпало %s, а у компьютера %s. %s';
$resultMessages = ['Выигрыш компьтера', 'Ничья', 'Выигрыш игрока'];

$userDice = mt_rand(1,6);
$compDice = mt_rand(1,6);

$diff = $userDice - $compDice;
$result = ($diff ? $diff / abs($diff) : $diff) + 1;

printf($message, $userDice, $compDice, $resultMessages[$result]);

Рефакторим на php7 (минус две строки)

$message = 'У юзера выпало %s, а у компьютера %s. %s';
$resultMessages = ['Выигрыш компьтера', 'Ничья', 'Выигрыш игрока'];

$userDice = mt_rand(1,6);
$compDice = mt_rand(1,6);

printf($message, $userDice, $compDice, $resultMessages[($userDice <=> $compDice) + 1]);

Comments

[Stalker_RED]

Это крутой троллинг, чтоб вынести нубу мозг. Если задуматься, то в плане читаемости это еще больший говнокод, чем оригинал. :)

[Сергей delphinpro]

Stalker_RED: а я люблю так писать =))

[Stalker_RED]

Сергей: Тогда уж сразу так, чтоб добить
sandbox.onlinephpfunctions.com/code/78b1f4c78b2556...

[Сергей delphinpro]

Stalker_RED:

[Константин Кавенский]

Stalker_RED: Это спецом под 21:9?

[Stalker_RED]

Константин Кавенский: Это "типа однострочник".

Answer 2

[Илья]

Потому что это говно код. Дело закрыто!

www.php-fig.org

А если по коду:

$userDice = mt_rand(1,6);
$computerDice = mt_rand(1,6);

// для PHP 7
switch ($userDice <=> $computerDice) {
	case -1:
		$message = "У компьютера выпало {$computerDice}, а у юзера {$userDice}. Выигрыш компьютера";
		break;
	case 0:
		$message = "У юзера выпало {$userDice}, а у компьютера {$computerDice}. Ничья!";
		break;
	case 1:
		$message = "У юзера выпало {$userDice}, а у компьютера {$computerDice}. Выигрыш юзера";
		break;
}

echo $message;

Comments

[Сергей delphinpro]

Ваш вариант тоже ничем не отличается от исходного =))

[Илья]

Сергей: говнокод больше подразумевает удобочитаемость нежели алгоритмическую сложность. Ну и к тому же вопрос: "Если нужно будет исправить сообщение, то сколько строк придется переписать в моем варианте и сколько в вашем?"

[Сергей delphinpro]

Илья: Ну я вижу, что у меня представление данных отделено от логики. Нужно будет заменить пару первых строк. Можно легко вывести их на инглише или китайском =)

[Илья]

Сергей: я на самом деле перепутал ваш вариант с вариантом Максим Тимофеев , но все равно. Если структура сообщения будет одинаковая, а если к примеру сообщение в принципе разные. Ну да ладно, это тот еще холивар, в другой ветки уже начали про стандарты говорить)))))

[Сергей delphinpro]

Илья: Да я вообще фронтэндер, про пхп краем уха слышал )

[Stalker_RED]

Илья: Откуда эта мода пошла, использовать фигурные скобки в строке в тех случаях, когда они не нужны? Годами они оставались относительно редко используемой фичей, и тут как прорвало!

[Илья]

Stalker_RED: мне так виднее. И часто я использую вывод массивов ("text {$row['id']} text")

[Максим Тимофеев]

А в каком месте автор сказал про php7 ? Потому как в более ранних версиях "<=>" нету. И то что Вы привели не говно код, это не рабочий код для php < 7

[Stalker_RED]

Илья: Серьезно, виднее? Настроить подсветку в редакторе сложнее, чем писать скобочки каждый раз? Ну ок.

[Григорий Есин]

Неее, это тоже говнокод. Надо испольновать паттерн "Visitor" :))

Answer 3

[Максим Тимофеев]

Если не устраивать из этого ООП, то в голову приходит только это:

<?php
$userDice = mt_rand(1,6);
$computerDice = mt_rand(1,6);
$text = 'У юзера выпало ' . $userDice . ', а у компьютера ' . $computerDice . '.'; 
if ($userDice > $computerDice){
  $text .= ' Выигрыш юзера';
} else if ($userDice < $computerDice){
  $text .= ' Выигрыш компьютера';
} else {
  $text .= ' Ничья';
}
echo $text;

Comments

[Илья]

Вы уверены что это лучше чем было?

[Максим Тимофеев]

Илья: Я да, есть конкретные предложения или замечания - озвучивайте.

[Илья]

Максим Тимофеев: что мешает написать так: "У юзера выпало {$userDice}, а у компьютера {$computerDice}." ?

[Максим Тимофеев]

Илья: Такой тип конкатенации как у меня не является говнокодом, мало того это стандарт. Хотя более быстрое выполнение обеспечит такой вариант:

$text = 'У юзера выпало ';
$text .= $userDice;
$text .= ', а у компьютера ';
$text .= $computerDice;

А в ваше варианте не вижу вообще смысла. Фигурные скобки используют для экранирования внутри php, например: $this->owner->{$someVar}

[Максим Тимофеев]

Илья: Если подразумевать что этот код является частью чего-то большего, то вообще надо использовать функции. А в идеале ООП, создавать объект, запускать игру и т.д.

[Виталий Хоменко]

Максим Тимофеев: И внутри строк такие скобки тоже используют, для выделения переменных.

[Максим Тимофеев]

Виталий IIIFX Хоменко: Есть стандарты php psr, в которых описан мой метод конкатенации и не описан Ваш. О чем спор?

[Илья]

Максим Тимофеев: где простите описан ваш способ конкатинации? Точное место положении скажите пожалуйста (PSR 1-2, строка, абзац?).

[Илья]

Максим Тимофеев: почему все понимаю "говнокод" только со стороны какой-то оптимизации? Немаловажную роль занимает внешний вид кода (понятность). Если для вас конечно все равно как выглядит код, то у нас с вами очень разные понятия говнокода.

[Виталий Хоменко]

Максим Тимофеев: Я не спорю с вами. Я лишь сказал, что подобный способ тоже используется. Если вы хотите слепо следовать PSR, при этом отвергая все остальное, то это ваше право.

[neol]

Максим Тимофеев: Говнокодом ваш тип конкатенации конечно не является, но для PHP 7 это более медленный вариант https://habrahabr.ru/company/mailru/blog/318008/ "Оптимизация с помощью encapsed-строк"

[Максим Тимофеев]

neol: откуда PHP 7 взялось? Автор писал о версии? Если нет, то почему бы не привести в пример 5.4 или 5.6? А с точки зрения js это вообще работать не будет.

[Максим Тимофеев]

Илья: строка, абзац. Сразу хотел привести, но не могу найти. Но там есть точно. Если есть опровержения - покажите абзац и строку.

[Илья]

Максим Тимофеев: про PSR - не могу привести опровержение того, чего в стандарте просто не описано. Про PHP7 - если вы пишите все еще на PHP > 7 - то нам тем более с вами говорить не о чем. Если бы вы были .NET разработчиком, то скорее бы всего писали на .NET 3.0 и не страшно что есть уже 4.6.

[Илья]

Максим Тимофеев: WAT?! Причем тут вообще JS? Вы пишите код чтобы он был совместим с JS, или чтобы он все таки отлично работал на PHP?