Что не так с кодировкой?

Question

[hrvasiliy]

В БД установлена кодировка utf-8, на сайте такая же. Записываю в БД из текстового поля данные на русском языке. В БД вижу нечитаемые символы. Попробовал добавить перед выполнением запроса:

mysqli_query($mysqli, "SET NAMES 'utf8' COLLATE 'utf8_general_ci'");

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

Есть ли способ решить эту проблему другим путем? Раньше у меня не возникало таких проблем, может быть потому что стоял apache?

Comments

[nirvimel]

В БД вижу нечитаемые символы.

А чем смотрите то? У просмотрщика ведь тоже могут быть проблемы с кодировкой.

[hrvasiliy]

nirvimel: Смотрю в PhpMyAdmin. Если через него прописываю русские символы - все ОК

Answer 1

[neol]

По умолчанию сервер mysql использует кодировку latin1.
Если я ничего не путаю, то пришедший от вас запрос в utf8 конвертируется в latin1, а затем получившаяся абракадабра (нормально utf8 в latin1 не конвертируется) конвертируется опять в utf8. На выходе получается фигня.

Добавление в my.cnf

collation-server = utf8_general_ci
character-set-server = utf8

с последующим перезапуском mysql по идее должно решить проблему.

Ну или можно добавить в тот же my.cnf

init-connect = "SET NAMES 'utf8' COLLATE 'utf8_general_ci'"

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

Comments

[hrvasiliy]

А почему могло раньше работать? На данный момент я оставил вот так:

public function query($query)
	{
		$mysqli = mysqli_connect(HOST, USER, PASSWORD, NAME);
		mysqli_query($mysqli, "SET NAMES 'utf8' COLLATE 'utf8_general_ci'");
		$res = mysqli_query($mysqli, $query);
		return $res;
	}

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

[neol]

hrvasiliy: Не знаю где, как и почему это работало.

А вот ваш код ужасен. Создавать новое подключение к mysql на каждый запрос - это жестко. Вы его хоть в статическую переменную сохраняйте.

Что-то типа:

public function query($query)
  {
    static $mysqli = mysqli_connect(HOST, USER, PASSWORD, NAME);
    static $charset_initialized = mysqli_query($mysqli, "SET NAMES 'utf8' COLLATE 'utf8_general_ci'");
    $res = mysqli_query($mysqli, $query);
    return $res;
  }

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

Не говоря уже про 2015 год и процедурный стиль программирования (=

PS Код не проверял.

[hrvasiliy]

neol: использую mvc, это кусочек из базового класса модели. Обращаюсь к этому методу, когда надо выполнить какой-то запрос. А зачем я буду записывать результат выполнения функции mysqli_query в статическую переменную? Я же не буду ее использовать...

[neol]

hrvasiliy: Чтобы не выполнять этот запрос ещё раз. Это конечно костыль, но вполне рабочий.

[hrvasiliy]

neol: Пожалуй, так и сделаю. Скажите, а в чем преимущество в данном случае использовать ООП? Я перешел на MVC, потому что это реально удобно, что все аккуратно и легко расширяемо. А в данном случае использование ООП как-то быстрее? Или в чем смысл?

[hrvasiliy]

neol: И не могли бы пояснить, как сделать без костыля? :)

[hrvasiliy]

neol: И по какой-то причине ошибка на этой строке, где использую static - убираю и снова все работает.

Parse error: syntax error, unexpected '(', expecting ',' or ';'  in

[neol]

hrvasiliy: Ну по хорошему ваш класс для работы с БД должен инициализировать подключение к БД в конструкторе и там же выполнять все нужные запросы с кодировками. А в остальные места он должен передаваться уже готовеньким (dependency inversion, ага). Соответственно ваш метод query сведётся к одной строчке:

return $this->mysqli->query($query);

Врядли это будет работать быстрее (хотя может и будет), но точно будет проще и правильнее.

Чуть менее правильный вариант: Singleton. В принципе все тоже самое, но без инверсии зависимостей. Тестировать сложнее.

В вашем же куске кода весь ООП сводится к куче знаков вокруг обычной функции, которая точно также будет работать вне класса (я его и принял за процедурный код вначале, не заметив public). Этакий ООП Шрёдингера (=

Пока сойдёт с костылём, а вообще погуглите про паттерны, SOLID, DI и прочую абракадабру. Я сильно сомневаюсь в своих способностях объяснить это в двух словах.

По поводу ошибки, рабочий вариант:

public function query($query)
  {
    static $mysqli;
    if(!isset($mysqli)) {
           $mysqli = mysqli_connect(HOST, USER, PASSWORD, NAME);
           $mysqli->query("SET NAMES 'utf8' COLLATE 'utf8_general_ci'");
    }
    $res = $mysqli->query($query);
    return $res;
  }
}

[hrvasiliy]

neol: Если вы обратились к методу query, не правильнее ли уже будет сделать так? :)

public function query($query)
		{
			static $mysqli;
			if(!isset($mysqli)) {
		       $mysqli = new mysqli(HOST, USER, PASSWORD, NAME);
		       $mysqli->query("SET NAMES 'utf8' COLLATE 'utf8_general_ci'");
			}
			$res = $mysqli->query($query);
			return $res;
		}

PS: У вас лишняя фигурная скобка :)

[neol]

hrvasiliy: Правильнее. Ну да, скобочек отвесил многовато.

Кстати этот код для продакшена не готов, т. к. нет никакой обработки ошибок. Если вы её убрали для упрощения - ок. Иначе стоит почитать документацию PHP, там есть примеры.

[hrvasiliy]

neol: Понял вас, спасибо!

Answer 2

[Макс]

есть кодировка сайта, кодировка БД и кодировка клиента. Что у вас в my.ini написано?
сорри, в my.cnf, в [client]

Comments

[hrvasiliy]

port и socket

[Макс]

hrvasiliy: почитайте gahcep.github.io/blog/2013/01/05/mysql-utf8

[hrvasiliy]

Макс: Прочитал, обновил конфиг, перезагрузил служба, почему-то никаких изменений

Answer 3

[Руслан Федосеев]

кодировка обычно выставляется сразу после коннекта к базе...

Comments

[hrvasiliy]

я же написал, что раньше как-то работало и без лишнего обращения к БД