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

К примеру есть $_POST['name'] как мне из него сделать $name, т.е в массиве много ключей и всех мне надо перевести в обычный вид. 

но так лучше не делать

зачем такая задача?

Это не совсем подходит. Я же говорю надо перевести элемент массива в строковую переменную... а форич просчитывает  массив как элемент - ключ. А требуется из ключа сделать переменную.
из $_POST['name'] в  $name
из $_POST['city']  в $city
и т.д


А надо это чтобы потом проще было работать с переменными, и пропускать через функции. (Во всяком случае для меня так проще)

xarakiry, 
Либо я ничего не понял, либо Вы не поняли приведенного кода.

Судя по сумбурным описаниям, скорее второе. Поэтому перечитайте и перетестируйте образец кода.

xarakiry, через функции пропускать удобнее, как раз, массив. В цикле.
Давай ты покажешь нам эти функции, а мы покажем тебе компактный код в три строчки, который с ними работает.

Ну, вот моя функция 



это я ещё убрал из неё трим...

Знаю знаю выглядит ужасно и не компактно... 

xarakiry, примечание: htmlspecialchars - не для обработки перед вставкой в БД. а для обработки непосредственно перед выводом пользователю

Это все связанные между собой свойства?
Если да, то создай объект/модель/массив
и передавай в функцию именно его

// FIELD_NAME - это всегда имя из списка имен разрешенных для получения извне данных.
// FIELD_VALUE - сами данные соответствующие этому имени.
// как их лучше профильтровать см. один из приемов организации фильтрации входящих данных


Это сообщение отредактировал(а) Fortop - 15.7.2008, 21:06

Это POST
а транзакция ему не нужна

Отличный пример ! Вот только последнее не понял... что за функция mysql_start_transaction, я её даже на php.net не нашёл.... ???

Вопросик:


не понял что за "разрешенный тип или еще лучше имя callback функции для фильтрации и валидации данных" это как ?

xarakiry, это слишком сложно для человека, который впервые узнал про циклы
тебе надо понять две вещи.
1. что любой массив удобнее обрабатывать в цикле. чем городить из него сто переменных и писать каждую отдельной строчкой. Надеюсь, ты это уже понял
2. Что твои данные надо обрабатывать в цикле. Ты уже написал этот цикл? Можно посмотреть?

После того, как напишешь обработку в цикле, можно будет приступить к автоматическому составлению запроса. В цикле, разумеется.

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


Это как раз самое простое 
тривиальная обертка над твоим




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

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

Код

$allowed = array('user_id' => 'int',                  'name'    => 'string',                  'male'     => 'bool');

тогда фильтрация пришедших данных будет выглядеть примерно так

Код

foreach ($_POST as $key => $val) {     if (array_key_exists($key, $allowed)) {         switch ($allowed[$key]) {             case 'int':                 $result[$key] = (int)$val;             break;             case 'string':                 // строка остается как есть.                  $result[$key] = $val;                 // Но если нужно можем и с ней что-то сделать (например привести в верхний регистр)                 // $result[$key] = strtoupper($val);             break;             case 'bool':                 // скобки нужны лишь для лучшего понимания - т.е. необязательны.                 $result[$key] = ($val == 'true');             break;             default:             break;         }     } } //теперь у нас в $result только разрешенные нами переменные  и только тех типов которые мы ожидаем

Но, Feldmarschall, прав - лучше все по порядку. Так удобнее это осваивать.

Это сообщение отредактировал(а) Fortop - 16.7.2008, 00:59

та я в курсе. 
как ты правильно заметил, я хотел по порядку.

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



Это сообщение отредактировал(а) Feldmarschall - 16.7.2008, 08:31

Fortop Да, дейст-но с массивами проще, код уменьшается.
Feldmarschall Да , я слушаю но не раскрыв рот. Если бы я считал себя специалистом я не писал бы в этом разделе, а наверное задавал бы вопросы в разделе профи. Я не считаю что задать вопрос и получить ответ чем то постыдным.  захожу сюда потому что тут люди для разных задач находят интересные решения, что очень радует.
Спасибо большое всем за помощь!   

не понял,  чему была эта тирада.

Если речь идет про таблицы InnoDB, то действительно, один запрос и так всегда будет выполнятся в одной транзакции, а обновление одной строки вообще можно считать атомарной операцией (не может быть обновлена только половина строки).
А если про MyISAM, то тут по вообще транзакции почти не имеют смысла.

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

Эт так. Лирическое отступление   

xarakiry, ты бы, вместо отступлений, лучше написал здесь, что у тебя получилось в итоге.

Просмотрел, всё что написано... начал писать... к примеру 



не подходит потому что форма multipart/form-data и помимо обычных значений загружаются и файлы, которые предварительно сохраняются и переименовываются. Кроме того если брать полный просчёт массива пост то вместе со всеми переменными также идёт и сабмит, который ясное дело не нужен в базе. Что касается транзакций, то да... я почистил функции свои убрал транзакции..для однострокового запроса они дейст-но ни к чему.
единственное что более менее подошло это array_walk($_POST,'trim');
так что вопрос преобразования элементов массива в переменную остаётся...

Das is fantastisch...

То есть, для чего нужен массив $allowed, ты не понял.
попробуй перечитать ответы ещё раз. И пример, который тебе дали.
И перестань говорить "не подходит". замени это выражение на более корректное "я опять ничего не понял".

Feldmarschall, 
говоришь, он слушает? ну-ну 

Да, действительно. Он вообще ничего не слушает.

xarakiry. Читай, пожалуйста, внимательнее.
Не путай обработку формы в цикле и составление запроса.
Для обработки (trim, escape_string) цикл прекрасно подходит.
Для составления запроса, действительно, просто автоматом обрабатывать пост не получится. Но тебе это никто и не предлагает.
Тебе предлагают сделать массив с именами полей в базе. И по этому массиву составлять запрос.
в функцию, которая составляет запрос, ты передаешь ДВА массива. свою дату, и $allowed, про который тебе писали. В самом простом варианте $allowed - это просто массив имен полей.

В результате вся запись в базу сведется к двум строчкам:
$allowed=array("dk_nomre","sehra_qeydi" и так далее
$query="UPDATE museum_base SET ".create_set_query($_POST,$allowed);
mysql_query($query);

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

Да, не внимательно прочитал... вот что у меня получилось...



крутил вот эту строку 


хотел добавить в неё  picture1='$picture1' ,picture2='$picture2' ,picture3='$picture3'
атак же условие WHERE unical_number='$data[unical_number]' ... до 3-х провозился...

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

и к массиву POST обращаться тоже - только ОДИН раз! точно так же, как ты ОДИН РАЗ пишешь функцию mysql_real_escape_string. А не сто тысяч раз, как trim.

и функция updatedbitems тебе тоже не нужна.
а нужна универсальная функция для составления запросов. в которую передавать не массив $data, $_POST! Потому что в $_POST у тебя ледат все данные!

а имена картинок можно дописать в $_POST. и в $allowed

Добавлено через 5 минут и 12 секунд
Fortop, еще одна трудноуловимая проблема. Он не понимает, что значит $allowed. Для нас стобой имя этой переменной не нуждается в комментариях. а ему надо объяснять, что это массив, содержащий разрешенные имена переменных.

Ребята ...не надо на меня злится...я щас сижу и разбираю.... я знаю я вас замучил... Я сам хочу приучится писать циклами ...

Не надо учиться писать циклами.
Надо учиться ЧИТАТЬ.

я тебе написал готовый, практически, код.
показал, что должно быть в массиве $allowed
показал, что в функцию надо передавать массив $_POST. который и является массивом с данными, то есть, $data
и массив $allowed, в котором перечислены разрешенные поля.
и перебирая в цикле $allowed, подставлять соответствующие данные из $_POST в строку запроса, попутно обрабатывая их.

я тебе 5 раз написал, что функция updatedbitems не нужна. но она у тебя все равно на первом месте.

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

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

Давай все-таки по шагам, как предлагал Feldmarschall вначале.

1. У тебя есть данные из $_POST
2. тебе нужно каждый элемент обработать trim и другими функциями.

Ты можешь это сделать в цикле?

Прошу прощения что долго не отвечал...просто работой завалили....

Да... могу конструкцией 

  foreach ($_POST as $key => $value){
       $value=trim($value);
  }

я думаю автор просил что-то вроде функции serialize и unserialize...    ??? Это так???

xarakiry, посмотрите вот на эту функцию array_keys( ) она возвращает массив, содержащий все ключи исходного массива, переданного в качестве параметра.