/ КОМПЬЮТЕРЫ И ПО / Языки программирования / PHP

Выпуск № 738
от 14.08.2007, 13:05

Вопрос № 97.716

Что в этом коде не так?? Выводит ошибку You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'by, comment, ip, stamp) values ('396', 'wormik', 'dsfadsf', '127 Приложение: if(!isset($_POST['text'], $_POST['id_pic'])) die('нет комментария'); $_POST['text'] = htmlspecialchars($_POST['text'], ENT_QUOTES); $_POST['text'] = trim($_POST['text']); if($_POST['text']==""){ header("location: image.php?c=$_POST[cat]&id=$_POST[id_pic]&empty"); die(); } if(!eregi("^[0-9]+$",$_POST['id_pic'])) die ('неверный ID'); if(!isuser()) $by = "anonym"; else $by = $_COOKIE['user']; $ip = $_SERVER['REMOTE_ADDR']; $query = "INSERT INTO comments (id_pic, by, comment, ip, stamp) values ('$_POST[id_pic]', '$by', '$_POST[text]', '$ip', NOW())"; mysql_query($query); print mysql_error();

Отправлен: 08.08.2007, 13:12 Вопрос задал: Бобровский Сергей Сергеевич (статус: 2-ой класс) Всего ответов: 6 Мини-форум вопроса >>> (сообщений: 0)

Отвечает: Селев Валерий Здравствуйте, Бобровский Сергей Сергеевич! Поскольку Вы заключаете значение $query в двойные кавычки, эскейпить одинарные не надо, попробуйте так: $query = "INSERT INTO comments (id_pic, by, comment, ip, stamp) values ('$_POST[id_pic]', '$by', '$_POST[text]', '$ip', NOW())";

Ответ отправил: Селев Валерий (статус: 5-ый класс) Ответ отправлен: 08.08.2007, 13:17 Оценка за ответ: 4 Комментарий оценки: Уже нашел, в чем проблема. Удивительным образом заключил поля и название таблици в косые кавычки ` и всё заработало. ХМм. Задумался.

Отвечает: Powoditch konstantin Здравствуйте, Бобровский Сергей Сергеевич! Дело в том что слово by является ключевым словом для SQL. Его надо екранировать вот так: INSERT INTO comments (id_pic, `by`, comment, ip, stamp) values ('$_POST[id_pic]', '$by', '$_POST[text]', '$ip', NOW())

Ответ отправил: Powoditch konstantin (статус: 4-ый класс) Ответ отправлен: 08.08.2007, 13:19 Оценка за ответ: 5 Комментарий оценки: Благодарю. ТЕперь я понял, почему ошибка исчезла, когда я заключил поля и таблицу в косые кавыки!

Отвечает: PVS Здравствуйте, Бобровский Сергей Сергеевич! Что-то у Вас слишком много всего экранировано слэшами и индексы массивов криво написаны. SQL должен выглядеть так: $query = "INSERT INTO comments (id_pic, by, comment, ip, stamp) values ( '".$_POST['id_pic']."', '$by', '".$_POST['text']."', '$ip', NOW())";

Ответ отправил: PVS (статус: Специалист) Ответ отправлен: 08.08.2007, 13:20

Отвечает: Oleg S. Mezhov Здравствуйте, Бобровский Сергей Сергеевич! 1. Не нужно экранировать одинарные кавычки, если вы создаете строку в двойных. 2. Интерполяция (подстановка значений переменных в строку) не всегда работает правильно, чаще используют следующий код или функцию создания форматированной строки sprintf. Приложение: $query = "INSERT INTO comments (id_pic, by, comment, ip, stamp) values ('" . $_POST[id_pic] . "', '" .$by . "', '" .$_POST[text] . "', '" . $ip . "', NOW())";

Ответ отправил: Oleg S. Mezhov (статус: 3-ий класс) Ответ отправлен: 08.08.2007, 13:31

Отвечает: Киряков Александр Иванович Здравствуйте, Бобровский Сергей Сергеевич! $_POST['text'] уберите обратный слеш -

Ответ отправил: Киряков Александр Иванович (статус: 1-ый класс) Ответ отправлен: 08.08.2007, 15:34

Отвечает: Gines Здравствуйте, Бобровский Сергей Сергеевич! by - это ключевое слово и лучше не использовать его в качестве имени поля. Я думаю что ошибка в этом.

Ответ отправил: Gines (статус: 3-ий класс) Ответ отправлен: 08.08.2007, 17:19

Вопрос № 97.792

Необходимо отпраить письмо функцией mail() и прикрепить к нему текстовый файл. как лучше можно это сделать?

Отправлен: 09.08.2007, 01:17 Вопрос задал: Sol_ (статус: Посетитель) Всего ответов: 1 Мини-форум вопроса >>> (сообщений: 0)

Отвечает: Powoditch konstantin Здравствуйте, Sol_! Вам следует прочитать о стандарте MIME. В приложении обяснение этого стандарта и пример посылки письма с приклеплением двух файлов к нему. Информацию нашел только на английском. Вас ведь это не остановит? Весь пример не влез в приложение, если есть интерес - напишите мне я вышлю на емейл полный текст. Приложение: MIME E-Mail Multipurpose Internet Mail Extensions (MIME), is a specification for enhancing the capabilities of standard e-mail. It provides a simple but a standardized way to represent and encode a wide variety of media types for transmission via e-mail. In all, you can send more than just plain text in a MIME e-mail. <?php // my_mime_mail_class.php include("./my_mail_class.php"); class My_Mime_Mail extends My_Mail { Properties The Content-Type header field value for textual data is set to "text/plain; charset=us-ascii" by default: var $type = 'text/plain'; var $charset = 'us-ascii'; The Content-Transfer-Encoding header is also set to its default value in its absence: var $encoding = '7bit'; We need a flag indicating whether the message has attachment(s): var $has_attach = 0; Files to be attached go into the $files array as elements. $files is a two-dimensional array with each element holding an associative array containing information on each file. For example, the following code snippet extracts information on the first file in the array: $file1 = $files[0]["file"]; // path of the file $filename1 = $files[0]["filename"]; // name of the file $filesize1 = $files[0]["filesize"]; // size of the file $filetype1 = $files[0]["filetype"]; // type of the file With this array, we can attach as many files as needed to an e-mail message: var $files = array(); The default content type for attached files is application/octet-stream. You may create an array holding extensions of well known file types and set the content type of a file according to its extension: var $mime_type = 'application/octet-stream'; The MIME-Version header field value and a warning for non-conformant e-mail clients are defined as the following: var $mime_version = "MIME-Version: 1.0"; var $mime_msg = "This is a multi-part message in MIME format."; We'll be using the "X-Mailer" custom header field: var $mailer = 'My Mime Mailer 1.0'; The boundary delimiter placeholder: var $boundary = ''; And another custom error message: var $ERR_CANNOT_OPEN_FILE = 'Cannot open the specified file!'; buildMimeHeaders() This method builds MIME headers in addition to the regular ones. If the message has attachment(s), a boundary delimiter is created to distinctively identify each body part, and the Content-Type header field is set to "multipart/mixed": function buildMimeHeaders() { $this->headers[] = "X-Mailer: " . $this->mailer; $this->headers[] = $this->mime_version; if ($this->has_attach) { $this->boundary = md5(uniqid(time())); If the message has attachments, we need a unique identifier. We use a multiple function call to ensure that a unique boundary delimiter is created: $this->boundary = md5(uniqid(time())); The time() function returns the current time in a UNIX timestamp format, which is used as a seed for the uniqid() function to produce a unique identifier. In turn, the outermost function, md5(), generates a 32-character boundary delimiter to ensure that no repeated IDs will be generated. The md5() function is based on an algorithm that takes a string of arbitrary length and uses it to generate a scrambled key. Since there are 2 raised to 128 possible key values, it is almost impossible to produce two keys with the same value. Using md5(), we therefore ensure that the possibility of generating duplicate IDs is almost nil. This is important because it's possible that someone might want to attach a copy of an e-mail message to another e-mail, and if the MIME boundaries were the same, the MIME separation at the receiving end would get confused: $this->headers[] = "Content-Type: multipart/mixed; boundary=" $this->boundary" "; $this->headers[] = $this->mime_msg . " "; $this->headers[] = "--" . $this->boundary; } $this->headers[] = "Content-Type: $this->type ; charset= $this->charset"; Set $this->encoding to 8bit if you want to use multi-byte characters in the text body part: $this->headers[] = "Content-Transfer-Encoding: $this->encoding"; } buildBodyParts() This is the very method that puts body parts together. If the message has no attachment, it silently returns to the calling function. function buildBodyParts() { if (!$this->has_attach) return true; The textual body part holds the first place in the array. Note the double CR/LF pairs used to separate it from additional body parts: $body_parts[0] .= $this->body . " "; Throw in a loop to add as many files to the message as the number of elements in the body part array indicates: for ($i=0; $i < count($this->files); $i++) { if (!($fp = @fopen($this->files[$i]["file"], "r"))) { $this->ERROR_MSG = $this->ERR_CANNOT_OPEN_FILE . " " . $this->files[$i]["file"]; return false; } Read in the entire content of the specified file: $file_body = fread($fp, filesize($this->files[$i]["file"])); Encode the file contents as a single long string, $file_body, and split it up into sets of 76 characters, which you'll recall is the maximum length allowed for lines of encoded data: $file_body = chunk_split(base64_encode($file_body)); PHP provides two functions that can be used together to achieve this goal: base64_encode(string data) This takes data in the form of a string, and returns the same data encoded in the base64 encoding type. chunk_split(string data, int length, string delimiter) This splits a given string of data into smaller chunks by inserting the string delimiter after every length characters. These parameters default to 76 and respectively if the last two parameters are omitted. Each body part is identified by the boundary delimiter we made earlier: $body_parts[$i+1] = "--" . $this->boundary . " "; Set the file type to that of the attached file or to the default value in its absence (application/octet-stream): if (!empty($this->files[$i]["filetype"])) $this->mime_type = $this->files[$i]["filetype"]; $body_parts[$i+1] .= "Content-Type: " . $this->mime_type . ";name=" . basename($this->files[$i]["filename"]) . " "; We use the base64 encoding method to encode non-textual data into 7-bit ASCII characters: $body_parts[$i+1] .= "Content-Transfer-Encoding: base64 "; $body_parts[$i+1] .= $file_body . " "; } The loop is done. Specify the end of the body parts by adding two trailing hyphens to the boundary delimiter: $body_parts[$i+1] .= "--" . $this->boundary . "--"; All body part headers and encoded data go into the message body block. We implode the $body_parts array into the body block: $this->body = implode("", $body_parts); return true; } viewMsg() The viewMsg() method now returns additional MIME headers: function viewMsg() { if (count($this->files) > 0) $this->has_attach = true; if (!$this->checkFields()) return false; $this->headers = array(); $this->buildHeaders(); $this->headers[] = "From: $this->from"; $this->headers[] = "To: $this->to"; $this->headers[] = "Subject: $this->subject"; $this->buildMimeHeaders(); if (!$this->buildBodyParts()) return false; $msg = implode(" ", $this->headers); $msg .= " "; $msg .= $this->body; return $msg; } send() Again, we overide the send() method defined in the My_Mail class to add MIME headers and build body parts: function send() { if (count($this->files) > 0) $this->has_attach = true; if (!$this->checkFields()) return false; $this->subject = stripslashes(trim($this->subject)); $this->body = stripslashes($this->body); $this->buildHeaders(); $this->buildMimeHeaders(); if (!$this->buildBodyParts()) return false; if (mail($this->to, $this->subject, $this->body, implode(" ", $this->headers))) return true; else { $this->ERROR_MSG = $this->ERR_SEND_MAIL_FAILURE; return false; } } } ?> Testing the My_Mime_Mail Class Here's a sample script using the class: <?php // my_mime_mail_class_test.php include("./my_mime_mail_class.php"); $mail = new My_Mime_Mail(); $mail->to = 'wankyu@whatever.com'; $mail->from = 'yonsuk@whoelse.com'; $mail->subject = "My picture!"; $mail->body = "Here goes my picture! Send me yours!"; $mail->files[0]["file"] = '/home/yonsuk/yonsuk.gif'; $mail->files[0]["filename"] = 'yonsuk.gif'; $mail->files[0]["filetype"] = 'image/gif'; if ($mail->send()) { echo("Successfully sent an email titled '$mail->subject'!"); } else { echo($mail->errorMsg()); } echo("<br>"); echo(str_replace(" ", "<br>", $mail->viewMsg())); ?> If the user uploads a file using PHP's file upload feature, the following global variables are automatically known to your e-mail script: $userfile, $userfile_name, $userfile_size, and $userfile_type. $userfile_type may not be available if the user's browser doesn't provide it. With these variables, you can easily build a body part containing the file that has been uploaded: $mail->files[0]["file"] = $userfile; $mail ->files[0]["filename"] = $userfile_name; $mail ->files[0]["filesize"] = $userfile_size; $mail ->files[0]["filetype"] = $userfile_type; Creating My_Smtp_Mime_Mail Class If you extend the My_Smtp_Mail class instead of My_Mail, you can send MIME e-mail messages using a remote SMTP server. You need to override the send() method differently, though: // my_smtp_mime_mail_class.php include("./my_smtp_mail_class.php"); class My_Smtp_Mime_Mail extends My_Smtp_Mail { function send() { if (count($this->files) > 0) $this->has_attach = true; if (!$this->checkFields()) return false; $this->subject = stripslashes(trim($this->subject)); $this->body = stripslashes($this->body); $this->buildHeaders(); The To and Subject headers are added before MIME headers: if (!empty($this->to)) $this->headers[] = "To: $this->to"; if (!empty($this->subject)) $this->headers[] = "Subject: $this->subject"; $this->buildMimeHeaders(); if (!$this->buildBodyParts()) return false; if (!$this->connect()) return false; if (!$this->talk("HELO", $GLOBALS["SERVER_NAME"])) return false; if (!$this->talk("MAIL", $this->from)) return false; if (!$this->talk("RCPT", $this->to)) return false; if (!$this->talk("DATA", implode(" ", $this->headers) . " " . $this->body)) return false; if (!$this->talk("QUIT")) return false; fclose($this->socket); return true; } }

Ответ отправил: Powoditch konstantin (статус: 4-ый класс) Ответ отправлен: 09.08.2007, 11:40

Вопрос № 97.831

есть простая форма с двумя полями, файл который обрабатывает данные с формы должен за счет условия if и просмотра переменной $_POST на наличие пустой переменой используя isset должна либо принять данные (если все поля заполнены) или выдать сообщение "Не заполнены все поля" но почему то скрипт все время выводит одно и тоже в не зависимости от того чтоя вводил в поле формы!!! Помогите пожалуйста а то уже 8 час мучаюсь с этой проблемой как только не изменял и форму и скрипт уже в каждой строчке ошибки мерещатся .. ппц Приложение: #форма <form action="script.php" method="$_POST"> Имя<input type=text name="name"> Фамилия<input type=text name="surname"> <input type=submit value="Отправить"> </form> #файл обрабатывающий данные с формы <?php if(isset($_POST["name"])){ echo "Не заполнено поле"; } else{ //// здесь далее обрабатывааем данные с формы print "$name"; } if(isset($_POST["surname"])){ echo "Не заполнено поле"; } else{ //// здесь далее обрабатывааем данные с формы print "$name"; } ?>

Отправлен: 09.08.2007, 12:57 Вопрос задал: Вацура Кирилл Андреевич (статус: Посетитель) Всего ответов: 7 Мини-форум вопроса >>> (сообщений: 2)

Отвечает: PVS Здравствуйте, Вацура Кирилл Андреевич! Не method="$_POST" а method="POST"

Ответ отправил: PVS (статус: Специалист) Ответ отправлен: 09.08.2007, 13:03

Отвечает: Селев Валерий Здравствуйте, Вацура Кирилл Андреевич! Попробуйте заменить <form action="script.php" method="$_POST"> на <form action="script.php" method="post">

Ответ отправил: Селев Валерий (статус: 5-ый класс) Ответ отправлен: 09.08.2007, 13:09

Отвечает: Nexus Здравствуйте, Вацура Кирилл Андреевич! Исправленный код в приложении. Во-первых, метод не $_POST, а просто post, $_POST это суперглобальный массив в пхп, к html никакого отношения не имеет. Во-вторых, isset() всегда будет возвращать true, т.к. у тебя есть это поле в форме, т.е. переменная у тебя будет объявлена, но содержаться в ней может либо пустая строка, либо нет. И еще, лучше отключи register_globals опцию в php.ini и обращайся к переменным из поста, гета, сессий и пр. через соответствующие суперглобальные массивы - $_POST, $_GET, $_SESSION соответственно Приложение: <form action="" method="POST"> Имя<input type=text name="name"> Фамилия<input type=text name="surname"> <input type=submit value="Отправить"> </form> <?php if(!$_POST["name"]){ echo "Не заполнено поле `Имя`"; } else{ //// здесь далее обрабатывааем данные с формы print $_POST['name']; } if(!$_POST["surname"]){ echo "Не заполнено поле `Фамилия`"; } else{ //// здесь далее обрабатывааем данные с формы print $_POST['surname']; } ?>

Ответ отправил: Nexus (статус: 9-ый класс) Ответ отправлен: 09.08.2007, 13:09

Отвечает: Flynn Здравствуйте, Вацура Кирилл Андреевич! Методы для формы - POST и GET. $_POST - это же глобальный массив в РНР и в обычном HTML он соответсвенно использоваться не будет. Так же для проверки сразу нескольких их можно задавать сразу в один isset. If iseet($_POST['name'],$_POST['sname'],...) {...}

Ответ отправил: Flynn (статус: 3-ий класс) Ответ отправлен: 09.08.2007, 13:11

Отвечает: Бобровский Сергей Сергеевич Здравствуйте, Вацура Кирилл Андреевич! Замените $_POST на POST. Ну это Вам уже сказали. Также у вас проверяется наличие переменной isset($_POST['surname']) вернет TRUE при наличии этой переменной, следовательно если она есть, то скрипт выведет "Не заполнено поле", а если нет, то попытается вывести имя. Вам же нужно наоборот. Для этого нужно добавить восклицательный знак. Исправленный и компактный код в приложении. Приложение: if(isset($_POST['name'], $_POST['surname']) && $_POST['name'] != "" && $_POST['surname'] != "") print "Имя: $_POST[name], фамилия: $_POST[surname]"; else print "Некорректно заполнены поля"; --------- Никогда не бойтесь делать то, что не умеете! Помните: Ноев Ковчег построил любитель, а Титаник - профессионал.

Ответ отправил: Бобровский Сергей Сергеевич (статус: 2-ой класс) Ответ отправлен: 09.08.2007, 13:32

Отвечает: M@gi$†e® Здравствуйте, Вацура Кирилл Андреевич! Ошибка: пропущен оператор ! (not) должно быть if(!isset($_POST["имя_переменной"])) --------- Aquila non Capta Muscas

Ответ отправил: M@gi$†e® (статус: 3-ий класс) Ответ отправлен: 09.08.2007, 19:41

Отвечает: Powoditch konstantin Здравствуйте, Вацура Кирилл Андреевич! Гммм. Такая простая ситуация=) Функция isset(value) - ВСЕГДА будет возвращать тру если Вы сабмитите форму, тоисть переменные name и surname будут всегда присутствувать! Вам просто напросто надо заменить isset на empty, тогда всё будет нормально.

Ответ отправил: Powoditch konstantin (статус: 4-ый класс) Ответ отправлен: 09.08.2007, 22:16

Вопрос № 97.832

Здраствуйте, уважаемые эксперты! Это программа "покер в кости", правда, изображения костей нельзя поместить здесь В этой программе (как и в некоторых других, которые я писал) есть проблема: данные после совершения круга действий теряются, т.е. например выполнен первый бросок, оставлены метки, выполнен 2-ой и показано, сколько выигрышь составил и сколько теперь наличных, а при нажатии "еще раз бросить" все начинается с начала, т.е. сумма наличных не сохраняется. Подскажите, как это исправить можно? Приложение: <html> <head> <title>Покер в кости</title> <style type="text/CSS"> body { background: green; color: tan; } </style> </head> <body> <center> <h1>Покер в кости</h1> <form> <?php if (empty($cash)){ $cash=100; } //завершение if rolldice(); if ($secondroll== TRUE){ print "<h3>Второй бросок</h3> "; $secondroll = FALSE; evaluate(); } else { print "<h3>Первый бросок</h3> "; $secondroll = TRUE; }//завершение if printstuff(); function rolldice(){ global $die, $secondroll, $keepit; print "<table border = 1><td><tr>"; for ($i = 0; $i < 5; $i++){ if ($keepit[$i]==""){ $die[$i]=rand(1,6); }else{ $die[$i]=$keepit[$i]; } //завершение if $thefile= "php" . $die[$i] . ".jpg"; //вывести изображения костей print <<<HERE <td> <img src= $thefile width = 100 heigh = 100 border = 0 ><br> HERE; //вывести флажки только при первом броске if ($secondroll==FALSE){ print <<<HERE <input type = "checkbox" name = "keepit[$i]" value = $die[$i]> </td> HERE; }//завершение if }//завершение цикла for //вывести кнопку отправить и окончание таблицы print <<<HERE </tr></td> <tr> <td colspan = "5"> <center> <input type ="submit" value ="еще раз бросить"> </center> </td> </tr> </table> HERE; }//завершение функции rolldice function evaluate(){ global $die, $cash; //обнуляем сумму выигрыша $payoff=0; $cash-=2; //считаем кости $numvals = array(6); for ($theval = 1; $theval <= 6; $theval++){ for ($dienum = 0; $dienum < 5; $dienum++){ if ($die[$dienum] == $theval){ $numvals[$theval]++; }//завершение иф }//завершение фор для диенум }//завершение фор для зеволс $numpairs = 0; $numthrees = 0; $numfours = 0; $numfives = 0; for ($i = 1; $i <= 6; $i++){ switch ($numvals[$i]){ case 2: $numpairs++; break; case 3: $numthrees++; break; case 4: $numfours++; break; case 5: $numfives++; break; } } if ($numpairs==2){ print "У вас две пары!<br> "; $payoff=1; } if ($numthrees==1){ if ($numpairs==1){ print "У вас фулхаус!<br> "; $payoff = 5; } else { print "У вас тройка! "; $payoff = 3; } } if ($numfours==1){ print "У вас четверка!<br> "; $payoff=5; } if ($numfives==1){ print "У вас пятерка! "; $payoff=10; } if (($numvals[1] == 1) && ($numvals[2] == 1) && ($numvals[3] == 1) && ($numvals[4] == 1) && ($numvals[5] == 1)){ print "У вас стрит! "; $payoff=10; } if (($numvals[2] == 1) && ($numvals[3] == 1) && ($numvals[4] == 1) && ($numvals[5] == 1) && ($numvals[6] == 1)){ print "У вас стрит! "; $payoff=10; } print "Вы поставили 2<br> "; print "Выигрыш составил $payoff <br> "; $cash += $payoff; } function printstuff(){ global $cash, $secondroll; print "Наличные: $cash "; //сохраняем переменные в скрытых полях print <<<HERE <input type = "hidden" name = "secondroll" value = "$secondroll" <input type = "hidden" name = "cash" value = "$cash"> HERE; } ?> </form> </center> </body> </html>

Отправлен: 09.08.2007, 13:00 Вопрос задал: Павел Владимирович (статус: 4-ый класс) Всего ответов: 1 Мини-форум вопроса >>> (сообщений: 0)

Отвечает: Flynn Здравствуйте, Павел Владимирович! После value = "$secondroll" у вас не закрыт тег - поэтому он не видит следующий cash. Так же я бы посоветовал выключить register_globals в php.ini и использовать массив $_POST дабы не засорять строку запроса всяким хламом, который пользователю видеть совсем не обязательно. Еще данные типа количетсва бросков и суммы лучше хранить в этом случае в сессиях (session_start();session_register('cash');$_SESSION['cash'] = $cash;).

Ответ отправил: Flynn (статус: 3-ий класс) Ответ отправлен: 09.08.2007, 13:26 Оценка за ответ: 5 Комментарий оценки: Спасибо, очень выручили.

© 2001-2007, Портал RusFAQ.ru, Россия, Москва. Авторское право: ООО "Мастер-Эксперт Про" Email: support@rusfaq.ru, тел.: +7 (926) 535-23-31 Хостинг: "Московский хостер" Поддержка: "Московский дизайнер" Авторские права | Реклама на портале Версия системы: 4.54 beta от 01.08.2007
RusFAQ.ru | MosHoster.ru | MosDesigner.ru | RusIRC.ru Kalashnikoff.ru | RadioLeader.ru | RusFUCK.ru