Из двух циклов сделать один?

Question

[Человек]

Есть два цикла, один цикл выводит значения по умолчанию,
а другой присваивает этим значениям по умолчанию три значения из GET запросов.

Как сделать один цикл, с этим же функционалом?
Это хорошая идея?
Или лучше пусть будут два отдельных цикла?
Просто не хочется писать много циклов.

$request = $_GET;
$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

foreach($request as $key => $items){
if(isset($request)){
$show[$key] = $items;
}}

foreach($show as $items){
echo $items."<br>";
}

<form method ="GET">   
Форма:                                                                                     
<div> Имя:     <input type="text" name="name1" />    </div>     
<div> Возраст: <input type= "number" name="age1" /> </div>  
<div> Вес:     <input type= "number" name="weight1" /> </div>  
<input type="submit" value="Отправить">
</form>

Answer 1

[AUser0]

$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

foreach($show as $key => $items){
    $show[$key] = (isset($_GET[$key]) AND strlen($_GET[$key])) ? $_GET[$key] : $items;
    echo "{$show[$key]}<br>";
}

Comments

[Человек]

Как эту тернарную конструкцию переписать при помощи if/else.
Я плохо понимаю синтаксис тернарных конструкций.
Мне будет гораздо легче понять суть кода при помощи if/else.

[Человек]

Правильно ли я переписал ваш код при помощи if else?

$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

foreach($show as $key => $items){
if(isset($_GET[$key]) && strlen($_GET[$key])){
$show[$key] = $_GET[$key];
}

else{
$show[$key] = $items;
}


echo "{$show[$key]}<br>";
}

[AUser0]

Ну если эта простейшая формулировка вас затрудняет, то... тяжело вам будет!

$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

foreach($show as $key => $items)
{
    if (isset($_GET[$key]) AND strlen($_GET[$key]))
    {
        $show[$key] = $_GET[$key];
    }
    echo "{$show[$key]}<br>";
}

[Человек]

AUser0, а куда вы дели
: $items;
из $show[$key] = (isset($_GET[$key]) AND strlen($_GET[$key])) ? $_GET[$key] : $items;

она же должна быть в

else{
$show[$key] = $items;
}

[AUser0]

Добрый Дельфин, переменная $show[$key] УЖЕ содержит значение из $items, это обусловлено выражением foreach($show as $key => $items). При использовании if это присвоение не нужно, потому что уже масло масленное.

[Человек]

AUser0, ну я интуитивно убрал else, когда
переписал ваш код при помощи if:

$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

foreach($show as $key => $items){
if(isset($_GET[$key]) && strlen($_GET[$key])){
$show[$key] = $_GET[$key];
}

else{
$show[$key] = $items;
}


echo "{$show[$key]}<br>";
}

Но согласно синтаксису тернарных операций,
двоеточие ":" это равносильно else

$show[$key] = (isset($_GET[$key]) AND strlen($_GET[$key])) ? $_GET[$key] : $items;

вот я и подумал, что : $items;
нужно поставить в else

else{
$show[$key] = $items;
}

[AUser0]

Добрый Дельфин, тернарный оператор без второй половины написать не получится (если без двух вопросов, не приучен), поэтому приходится присваивать элементу массива то значение, которое в нём УЖЕ хранится. В случае if можно обойтись без else, он и в тернарной то операции лишний, но без него никак. Зато тернарный короче.

[Человек]

AUser0, никогда не понимал желание использовать тернарники/стрелочные функции.
они так путают, убрали бы их из языка.

[ThunderCat]

Добрый Дельфин, глупо ругать инструмент только по тому что вы его не освоили, или не используете. Тернарник очень удобен, так как не исполняет блоки, а возвращает одно из значений, это короче и выразительней.

[Человек]

AUser0, а зачем здесь проверка на strlen?
что она дает нам?
в доках написано, что

Функция strlen() возвращает null при использовании на массивах, а также выводит ошибку уровня E_WARNING.

почему у вас при использовании strlen на массиве нет ошибки

[AUser0]

Добрый Дельфин, потому что есть разница между массивом целиком и отдельным элементом массива, который суть есть переменная (текстовая строка).
А зачем? А это вам домашнее задание на развитие догадливости и ААА-А-А-АА-вот-оно-зачем-ности. Не трудно же!

Подсказка - а вот зачем нам значения по умолчанию делать?

[Человек]

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

[AUser0]

Добрый Дельфин, эх-хе-хе! Эта проверка не даёт пройти пустым (буквально из 0 символов) строкам из полей ввода. Ведь если указано default value, тогда пустые строки вам не нужны совсем абсолютно?!

Было бы правильнее указать не только default value, но и шаблоны проверки строк. Например для телефонного номера - пропускать только цифры, тире и круглые скобки. Удобно сделать такую фильтрацию через RegExp. Или указать название функции, которая должна фильтровать/проверять значение.

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

[Человек]

AUser0, как сделать фильтр я не понимаю =(
мне пока strlen() хватит для начала
где можно посмотреть примеры фильтров?
и что такое RegExp?

[Человек]

AUser0, вот такой кривой фильтр у меня получился:
убирает все из html формы кроме цифр.

foreach($_GET as &$items){
$items = filter_var($items, FILTER_SANITIZE_NUMBER_FLOAT);
}

echo $equals = $_GET["num1"] *  $_GET["num2"];

Answer 2

[Дмитрий]

$result = [];
foreach($request as $key => $items){
    $res = (!array_key_exists($key, $show)) ? $show[$key] : $items;
    echo $res;
    $result[$key] = $res;
    }
}

Comments

[Человек]

Ошибка Warning: Undefined variable $item in

$request = $_GET;
$show =["name1" => "по умолчанию", "age1" => "по умолчанию", "weight1" => "по умолчанию"];

$result = [];
foreach($request as $key => $items){
    $res = (!array_key_exists($key, $show)) ? $show[$key] : $item;
    echo $res;
    $result[$key] = $res;
    }

[Дмитрий]

Обновите пример где есть форма, чтоб можно было тест сделать

[Дмитрий]

Добрый Дельфин, обновил
Возможно там условие надо в другую сторону

$res = (!array_key_exists($key, $request)) ? $show[$key] : $items;

Answer 3

[Сергей Соколов]

array_merge() наложит на дефолтные значения то, что передали из формы.

$args = [
    'name1' => FILTER_SANITIZE_STRING,
    'age1' => FILTER_SANITIZE_NUMBER_INT,
    'weight1' => FILTER_SANITIZE_NUMBER_FLOAT,
];

$defaults = [
    "name1" => "по умолчанию",
    "age1" => "по умолчанию",
    "weight1" => "по умолчанию",
];

$results = array_merge($defaults, filter_input_array(INPUT_GET, $args));

// вывод
foreach ($results as $name => $value) {
    printf('<p>%s: %s</p>', $name, $value);
}

Comments

[ThunderCat]

Народ, вы че, угораете? Там тупо ставятся дефолтные хидден поля в форме и все, вопрос решен.

[Человек]

ThunderCat, так придется дублировать, если будет много кнопок.

[ThunderCat]

Добрый Дельфин,

так придется дублировать, если будет много кнопок.

Каких кнопок? Ты что, их рукам добавляешь?

[Сергей Соколов]

ThunderCat, вот щас для каждого поля будем ещё одно скрытое добавлять.
А бэк-то и не догадается, то ли ему от юзера пришло, то ли дефолтное подставилось. Имхо логичнее на бэке уже разбираться, что пользователь вписал, а что пропустил и чем заменить.

[ThunderCat]

Сергей Соколов,

вот щас для каждого поля будем ещё одно скрытое добавлять.

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

А бэк-то и не догадается, то ли ему от юзера пришло, то ли дефолтное подставилось.

Это... а какая разница?

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

Ну да, зачем править верстку, тем более что она по стандартному шаблону выводится и вставить еще одно поле не проблема. Надо залезть в код, и перечислить все возможные поля, конечно же.

[Сергей Соколов]

ThunderCat, «там, откуда я родом..» другие стандартные фичи и другая нормальность:
форме соответствует объект документа, класс определяет значения по умолчанию, в аннотациях прописаны права доступа, правила валидации. Запрос целиком «накладывается» на объект документа. Нет перечисления полей. Здесь это не воспроизвести короче, чем в ответе.

[ThunderCat]

Сергей Соколов, а, ну да, графья обычным хтмл не пользуются, и никогда не писали на мерзком процедурном... Так надо было в ответе и написать - создаете объект класса, прогоняете через валидатор, пришедшие поля сетите в объект и вызываете репозиторий->сохранить(). Че придумывать какие-то массивы, циклы... ))

[Сергей Соколов]

ThunderCat, мой ответ — мои правила. )

[Человек]

ThunderCat, а мои код написан при помощи процедурного стиля?

[ThunderCat]

Добрый Дельфин, твой твои все коды пока в процедурном стиле.

[Человек]

ThunderCat, а чем плох процедурный стиль?

[ThunderCat]

Добрый Дельфин, А где я сказал что он плох? Он подходит для небольших скриптов, но большие проекты с высокой сложностью на нем писать не удобно, долго, сложно и не поддерживаемо. Для более-менее крупных приложений используют подход ООП.

[Человек]

ThunderCat,
А большими проектами можно назвать интернет магазин или маркетплейс?
Какие еще бывают большие проекты?

[Сергей Соколов]

Добрый Дельфин, ThunderCat по что мой ответ комментами за..ли зафладили?

[ThunderCat]

Сергей Соколов, щито паделать... раз пошло обсуждение, не выносить же его из вопроса, все в контексте уже...

[Человек]

Сергей Соколов, ну что ты такой сердитый человек-то.
ну будьте людьми вы ребят, ну чего, ну все же мы люди.

Answer 4

[ThunderCat]

<form method ="GET">   
Форма:                                                                                     
   <div> Имя:     
      <input type="hidden" name="name1" value="по умолчанию"/>
      <input type="text" name="name1" />    
   </div>     
...
   <input type="submit" value="Отправить">
</form>

Comments

[Человек]

что это не понял?

[ThunderCat]

Добрый Дельфин, Что не понятно? Это стандартный метод передать дефолтные значения из формы.

[Человек]

ThunderCat, ну и как это вывести на экран?

[ThunderCat]

Добрый Дельфин, обычным foreach(), нужен один цикл и никаких дополнительных массивов и прочей ерунды.

[Человек]

ThunderCat, а как взять данные по умолчанию из html формы и вывести в foreach?
можете показать пример?

[ThunderCat]

Добрый Дельфин,

if(!empty($_GET)){
   foreach($_GET as $key => $val){
      echo "$key : $val <br>";
   }
}

[Человек]

ThunderCat, надо чтобы при входе на страницу, пока ничего не нажал выходило три значения:
по умолчанию
по умолчанию
по умолчанию
Ваш код не выводит значения по умолчанию

[ThunderCat]

Добрый Дельфин, нет, не выводит. Хотя зачем - не понятно. Можно те же значения просто вбить в форму по умолчанию, зачем выводить результат, если ты ничего не делал?

[Человек]

ThunderCat, а как сделать, что он выводил?

[ThunderCat]

Добрый Дельфин, У тебя задача какая? Попрактиковаться/побаловаться или это реальное что-то?
Если реальное - то что конкретно из функционала работает на таком кривом коде?

[Человек]

ThunderCat,

на таком кривом коде?

вы говорите про свой код?

Попрактиковаться, как вывести значения по умолчанию на экран вашим методом.

[Человек]

ThunderCat,
А большими проектами можно назвать интернет магазин или маркетплейс?
Какие еще бывают большие проекты?

[ThunderCat]

Добрый Дельфин,

А большими проектами можно назвать интернет магазин или маркетплейс?

Да, там где есть много сущностей, много связей между ними и достаточно сложные взаимодействия. Например тот же магазин имеет с пару десятков сущностей, с достаточно сложными отношениями - пользователи, роли, товары, свойства товаров, заказы, статусы обработки, фидбэк, статистика... и еще много чего.

Какие еще бывают большие проекты?

Любые, от локальной CRM до соцсетей.

[ThunderCat]

Добрый Дельфин,

Попрактиковаться, как вывести значения по умолчанию на экран вашим методом.

Если задача чисто учебная, то смысл спрашивать как вывести абсолютно абстрактные данные в абсолютно абстрактном виде теряет смысл. Есть 5-10 способов, каждый из которых хорошо подойдет в конкретной ситуации, и плохо подойдет в других. Зависит от того откуда будут браться данные, куда уходить, как их будут использовать и еще миллион нюансов. Все равно что спрашивать какие цветы лучше всех? На похороны или девушке дарить? Или сажать на клумбе? Или...

[Человек]

ThunderCat, что такое локальное CRM?
и что такое роли в магазине?

[Человек]

ThunderCat, значит тут нужно использовать js, чтобы вывести на экран вашим методом, верно?
покажите хотя бы 1 способ на php, мне крайне интересно.

[ThunderCat]

Добрый Дельфин, нет, думаю через жс не лучший способ. Мой вариант рассчитан именно на вывод отправленной формы, просто значения без формы он выводить не будет. Подразумевается что список будет показан если была отправлена форма (что обычно именно так и делается).

Еще раз - данные для формы не берутся из воздуха, они могут храниться в бд, в файле, или как в случае вашего кода - забиты гвоздями в код (так называемый хардкод). Вот из источника данных и берите значения для списка перед отправкой формы. При отправке проверяете пришли ли данные из формы, или это просто заход на страницу, и выводите нужные значения.

что такое локальное CRM?

Гуглите CRM

и что такое роли в магазине?

Гуглите RBAC