Это дело на любителя конечно, н я вот не понимаю, зачем лишние движения...
Совсем не обязательно тело запроса сначала записывать в переменную. Проще, быстрее и нагляднее имхо так:
| PHP |
<?php
$result = mysql_query("INSERT into $table_name values ('','".$Array[log]."', '".$Array[pswrd]."')");
if ($result > 0) {
print "ok";
} else {
print "bad";
} |
Кроме того, константы при определении элемента массива нужно заключать в апострофы:
| PHP |
<?php
$result = mysql_query("INSERT into $table_name values ('','".$Array['log']."', '".$Array['pswrd']."')");
if ($result > 0) {
print "ok";
} else {
print "bad";
} |
Ну и совсем красиво и в полтора раза быстрее, когда используется конкатенация:
| PHP |
<?php
$result = mysql_query("INSERT into `".$table_name."` values ('','".$Array['log']."', '".$Array['pswrd']."')");
if ($result > 0) {
print "ok";
} else {
print "bad";
} |
а что бы уж совсем хорошо, операторы SQL лучше писать в одном (верхнем) регистре
| PHP |
<?php
$result = mysql_query("INSERT INTO `".$table_name."` VALUES ('','".$Array['log']."', '".$Array['pswrd']."')");
if ($result > 0) {
print "ok";
} else {
print "bad";
} |
ну и так совсем читабельно:
| PHP |
<?php
$result = mysql_query("
INSERT INTO `".$table_name."`
VALUES ('','".$Array['log']."', '".$Array['pswrd']."')"
);
if ($result > 0) {
print "ok";
} else {
print "bad";
} |
Это сути дела не меняет, но разобраться в таком коде бывает на много проще.
А по сути, функция
mysql_query() при инсерте не возвращает никаких чисел, только true или false. По этому проверть надо так:
| PHP |
<?php
$result = mysql_query("
INSERT INTO `".$table_name."`
VALUES ('','".$Array['log']."', '".$Array['pswrd']."')"
);
if ($result) {
print "ok";
} else {
print "bad";
} |
А фигурные скобки лучше писать на одном уровне, тогда будет понятно, что открыто и где закрыто. Вот так:
| PHP |
<?php
if ($result)
{
print "ok";
}
else
{
print "bad";
} |
А в данном случае вообще без них:
| PHP |
<?php
if ($result)
print "ok";
else
print "bad"; |