Как оптимизировать эту часть кода?

Question

[Andrew Kolomiets]

Как можно это оптимизировать и какой источник информации посоветуете на эту тему?

<?if($result['login'] == 'admin'){
                    ?><input class="lk_input_main" autocomplete="off" readonly type="text" value="<?=$result['login']?>" name="login"><?
                } else{
                    ?><input class="lk_input_main" autocomplete="off" type="text" value="<?=$result['login']?>" name="login"><?
                }?>

Answer 1

[Алексей Скляров]

1. Используйте хотя бы такую конструкцию:

<?php if($result['login'] == 'admin'): ?>
    <input class="lk_input_main" autocomplete="off" readonly type="text" value="<?=$result['login']?>" name="login">
<?php else: ?>
    <input class="lk_input_main" autocomplete="off" type="text" value="<?=$result['login']?>" name="login">
<?php endif; ?>

2. Лучше использовать шаблонизатор для таких целей
3. Ваш код отличается лишь наличием readonly у инпута. Можно сократить так:

<input class="lk_input_main" autocomplete="off" <?php if($result['login'] == 'admin'): ?> readonly<?php endif; ?> type="text" value="<?=$result['login']?>" name="login">

Но лучше используйте шаблонизаторы.

Comments

[Beliyadm]

Плохая идея пихать условия php внутри html. Я бы сначала проверил условие, записал readonly в переменную и вывел её в инпуте, намного читабельней

<?php 
	$readonly = '';
	if($result['login'] == 'admin') { 
	    	$readonly = 'readonly';
	}
?>
<input class="lk_input_main" autocomplete="off" <?php echo $readonly; ?> type="text" value="<?=$result['login']?>" name="login">

[Алексей Скляров]

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

[FanatPHP]

Алексей Скляров, а как тут шаблонизатор принципиально сократит код-то?

[FanatPHP]

Beliyadm, это слишком смелое утверждение, про "плохая идея пихать условия php внутри html".
Не нужно так обобщать.
Как только readonly превратится в onclick="callMyFunc(this, 'параметр')", все эти благие намерения превратятся в тыкву.

Ну и если уж советовать сокращать код, то делать это по-человечески:

<?php $readonly = ($result['login'] === 'admin') ? ' readonly' : '' ?>
<input class="lk_input_main" autocomplete="off"<?=$readonly?> type="text" value="<?=$result['login']?>" name="login">