Почему не загружаются данные в таблицу?

Question

[Николай Громов]

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

<?php
include_once ('core/config_class.php');
include_once ('SxGeo.php'); 
$mysqli = new mysqli($server, $user, $pswd, $db);
if (mysqli_connect_errno()) {
    printf("Connect failed: %s\n", mysqli_connect_error());
    exit();
}
$mysqli -> select_db($db) or die ("Невозможно открыть $db");
function getRealIpAddr() {
    if (!empty($_SERVER['HTTP_CLIENT_IP']))        // Определяем IP
    { $ip=$_SERVER['HTTP_CLIENT_IP']; }
    elseif (!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
    { $ip=$_SERVER['HTTP_X_FORWARDED_FOR']; }
    else { $ip=$_SERVER['REMOTE_ADDR']; }
    return $ip;
}
if (strstr($_SERVER['HTTP_USER_AGENT'], 'YandexBot')) {$visitor='YandexBot';} //Выявляем поисковых ботов
elseif (strstr($_SERVER['HTTP_USER_AGENT'], 'Googlebot')) {$visitor='Googlebot';}
else { $visitor=$_SERVER['HTTP_USER_AGENT']; }
$ip = getRealIpAddr();
$SxGeo = new SxGeo('SxGeo.dat', SXGEO_BATCH | SXGEO_MEMORY);
$country = $SxGeo->getCountry($ip);
$gorod = $SxGeo->get($ip);
unset($SxGeo);
$date = date("H:i:s d.m.Y");        // определяем дату и время события
$host = $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];    // определяем страницу сайта
$ip   = addslashes ($ip);
$date = addslashes ($date);
$host  = addslashes ($host);
$gorod = addslashes ($gorod);
$country = addslashes ($country);
$visitor = addslashes ($visitor);
$query = ("INSERT INTO `pre_visitors` (date, visitor, ip, country, cities, host) VALUES ('"
    .$date."', '".$visitor."', '".$ip."', '".$country."', '".$gorod."', '".$host."')");
$mysqli -> query($query);
printf ($query);
?>
<footer>
    <p><strong>Все права защищены © copyright </strong></p>


	<div id="qoo-counter">
      <a href="#">
           <img src="images/010.png" alt="Укоротить ссылку"/> 

			<div id="qoo-counter-visits"></div>
            <div id="qoo-counter-views"><?php printf($mysqli->insert_id);?></div>

        </a>
    </div>
<?php $mysqli -> close();?>

Answer 1

[Дмитрий]

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

Comments

[Николай Громов]

А для чего еще проверку делать? select_db($db) и new mysqli() проверяются что еще то проверять?

[Дмитрий]

Николай Громов:
$date = addslashes ($date); - не верно
$date = date("H:i:s d.m.Y"); - зачем?
$mysqli -> query($query); - проверка
и так далее...

[Дмитрий]

Николай Громов: php.net/manual/ru/mysqli.real-escape-string.php

[Николай Громов]

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

<?php
include_once ('core/config_class.php');
include_once ('SxGeo.php'); 
$mysqli = new mysqli($server, $user, $pswd, $db);

$mysqli -> select_db($db) or die ("Невозможно открыть $db");
if (!$mysqli->set_charset("utf8")) {
    printf("Ошибка при загрузке набора символов utf8: %s\n", $mysqli->error);
    exit();
} 
function getRealIpAddr() {
    if (!empty($_SERVER['HTTP_CLIENT_IP']))        // Определяем IP
    { $ip=$_SERVER['HTTP_CLIENT_IP']; }
    elseif (!empty($_SERVER['HTTP_X_FORWARDED_FOR']))
    { $ip=$_SERVER['HTTP_X_FORWARDED_FOR']; }
    else { $ip=$_SERVER['REMOTE_ADDR']; }
    return $ip;
}

if (strstr($_SERVER['HTTP_USER_AGENT'], 'YandexBot')) {$visitor='YandexBot';} //Выявляем поисковых ботов
elseif (strstr($_SERVER['HTTP_USER_AGENT'], 'Googlebot')) {$visitor='Googlebot';}
else { $visitor=$_SERVER['HTTP_USER_AGENT']; }

$ip = getRealIpAddr();


$SxGeo = new SxGeo('SxGeo.dat', SXGEO_BATCH | SXGEO_MEMORY);
$country = $SxGeo->getCountry($ip);
$gorod = $SxGeo->get($ip);
unset($SxGeo);
$date = date("H:i:s d.m.Y");        // определяем дату и время события
$host = $_SERVER['HTTP_HOST'] . $_SERVER['REQUEST_URI'];    // определяем страницу сайта
$ip   = $mysqli->real_escape_string($ip);
$date = $mysqli->real_escape_string($date);
$host  = $mysqli->real_escape_string($host);
$gorod = $mysqli->real_escape_string($gorod);
$country = $mysqli->real_escape_string($country);
$visitor = $mysqli->real_escape_string($visitor);


$query = "INSERT INTO `pre_visitors` (date, visitor, ip, country, cities, host) VALUES ('"
    .$date."', '".$visitor."', '".$ip."', '".$country."', '".$gorod."', '".$host."')";
if ($mysqli -> query($query)){
printf("%d строк вставлено.\n", $mysqli->affected_rows);
}
?>

[Дмитрий]

Николай Громов:
[code lagn=php]
$mysqli = new mysqli("localhost", "my_user", "my_password", "world");

/* проверка соединения */
if ($mysqli->connect_errno) {
printf("Не удалось подключиться: %s\n", $mysqli->connect_error);
exit();
}

if ($mysqli -> query($query)){
printf("%d строк вставлено.\n", $mysqli->affected_rows);
}
else{
printf("Errorcode: %d\n", $mysqli->errno);
}
[/code]

Answer 2

[Николай Громов]

Всем спасибо разобрался. Я забыл что изменил в таблице имя одного столбца с cities на region, и поэтому запрос не отрабатывал!

Comments

[Дмитрий]

Надо включать вывод ошибок в окно браузера, чтобы не морочить себе голову

[Николай Громов]

Эту ошибку не вывело бы. Тут дело в памяти и внимательности.