$sort = date;
$s_back = ASC;
if (!empty($_GET['sort'])) {
if ($_GET['sort'] == 'name'){$sort = 'name';} else {if ($_GET['sort'] == 'email') {$sort = 'email';}
else {if ($_GET['sort'] == 'date') {$sort = 'date';}
}
}
}
Полная Версия: Громоздкий код
Даже не громоздкой, а плохо читаемый. Можно ли как то заменить иф чтобы код легче читался?
к стати если глобальная переменная гет не равна условию можно ли ее в последнем условии удалить через ансет?!
1. switch
2. Через unset ключ массива можно удалить всегда
_____________
There never was a struggle in the soul of a good man that was not hard
2. Через unset ключ массива можно удалить всегда
_____________
There never was a struggle in the soul of a good man that was not hard
| Цитата (h0ck @ 29.06.2013 - 13:46) |
| Можно ли как то заменить иф чтобы код легче читался? |
можно его, например, по-человечески написать
Отформатировать можно!
| Цитата (glock18 @ 29.06.2013 - 13:59) | ||
можно его, например, по-человечески написать |
Ну так подсказал бы сразу как ето зделать, а не пустословные советы давал... Я ждал если чесно не критики по моему говнокоду, а помощи!
| Цитата (h0ck @ 29.06.2013 - 19:13) |
| Я ждал если чесно не критики по моему говнокоду.... |
Самокритичный, но обидчивый ;)
Для начала просто отформатируй красиво, тогда логика становится намного лучше видна:
if (!empty($_GET['sort']))
{
if ($_GET['sort'] == 'name')
{$sort = 'name';}
else
{if ($_GET['sort'] == 'email')
{$sort = 'email';}
else
{
if ($_GET['sort'] == 'date')
{$sort = 'date';}
}
}
}
А дальше смотрим и переделываем:
$sort = ( !empty( $_GET['sort']) and in_array( $_GET['sort'], array( 'name', 'email', 'date') ) )? $_GET['sort'] : '';
PS.
$sort = date;И кстати. Тут что у тебя - константы или символьные строки? Если строки, то лучше их в кавычках напиши, иначе с таким подходом можешь получить "грабли".
$s_back = ASC;
_____________
* Хэлп по PHP
* Описалово по JavaScript
* Хэлп и СУБД для PostgreSQL
* Обучаю PHP, JS, вёрстке. Интерактивно и качественно. За разумные деньги.
* "накапливаю умение телепатии" (С) и "гуглю за ваш счет" (С)
Здесь расположена полная версия этой страницы.