На хостинге не работает самописная PHP функция?

Question

[Artqookie]

Есть функция, которая переводить некоторые блоки текста:

<?php
function translate($rus,$eng)
{
	$lang = $_GET['lang'];
	$cookie_fresh = mktime(23,59,59,12,31,date('Y')+1);
	
	if ($lang == 'eng')
	{
		setcookie('lang', 'eng', $cookie_fresh);
		header("Location: ./");
	}
	if (($lang == 'rus') or (!isset($_COOKIE['lang'])))
	{
		setcookie('lang', 'rus', $cookie_fresh);
		header("Location: ./");
	}
	
	if ($_COOKIE['lang'] == 'eng')
	{
		echo $eng;
	}
	if ($_COOKIE['lang'] == 'rus')
	{
		echo $rus;
	}
	return true;	
}
?>

Преобразование происходит так:

<?php translate('Русские буквы', 'English words'); ?>

На LAMP'e, на локалке, всё тип топ. Но, когда заливаю на виртуальный хостинг, никакой текст не отображается. В чем может быть проблема?

Comments

[Artqookie]

И вообще — ни одна функция не работает. Но, думаю, если разобраться с одной, то исправятся и другие.

Answer 1

[Алексей]

error_reporting(E_ALL);
ini_set('display_errors', true);

Comments

[Artqookie]

Ошибок нет, но куча Notice и Warnings.
Warning'сы все — «Cannot modify header information — headers already sent by (output started at» на разный строчках. Я так понимаю, не прокатывает потому что «echo» есть перед заголовками? Хотя все функции у меня перед страницей.

[Алексей]

Для начала, делают так:

header("HTTP/1.0 302 Moved");
header("Location: ./");
exit;

Если хотите гонять кукисы в любом месте, то надо использовать output_buffering = On в php.ini или использовать ob_start() + ob_end_flush()

[Artqookie]

Круто, спасибо!

Answer 2

[Shvonder]

Мой вариантик для разминки ума:

function translate($rus, $eng)
{
	static $lang;
	if (isset($_GET['lang']))
	{
		$lang = $_GET['lang'] == 'eng' ? 'eng' : 'rus';
		setcookie('lang', $lang, time() + 3600 * 24 * 365, '/');
		unset($_GET['lang']);
	}
	elseif (empty($lang))
		$lang = $_COOKIE['lang'] == 'eng' ? 'eng' : 'rus';
	echo $$lang;	
}

Comments

[Artqookie]

Эффективно :-) Мне до такого еще далеко… Не подскажете, что значит static переменная и почему тут она применена? И зачем нужен вывод именно переменной от переменной?

[Shvonder]

www.php.su/learnphp/vars/?statvars
www.php.su/learnphp/vars/?varsvars

[Shvonder]

Статическая переменная в данном случае нужна, что бы не проверять $_COOKIE и $_GET при следующих вызовах translate. И, как следствие, не использовать редирект и повторную отправку заголовков.

[Artqookie]

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

[Shvonder]

Тут вы не правы. Адресная строка для того и нужна, что бы передавать через нее параметры. И это вполне нормально, что в ней при смене языка один раз появится ?lang=eng. Вы бы лучше думали о том, что ваш сайт в английской версии индексироваться не будет. А это — проблема серьезнее.

Answer 3

[shushu]

Функции setcookie и header должны выполняться ДО любого вывода (echo, print, var_dump, print_r… etc)
А также советую обзавестись флагом, что бы куки ставились 1 раз.
Хотя лучше после header(«Location: ./»); выполнять exit();

как то так:

<?php
function translate($rus,$eng)
{
	$lang = $_GET['lang'];
	$cookie_fresh = mktime(23,59,59,12,31,date('Y')+1);
	
	$c_lang = isset( $_COOKIE['lang'] ) ? $_COOKIE['lang'] : 'ru';
	
	if ( $lang == 'eng' && $c_lang != $lang )
	{
		setcookie('lang', 'eng', $cookie_fresh);
		header("Location: ./");
		exit();
	}
	if ($c_lang != $lang && ($lang == 'rus' or !isset($_COOKIE['lang'])))
	{
		setcookie('lang', 'rus', $cookie_fresh);
		header("Location: ./");
		exit();
	}
	
	switch( $_COOKIE['lang'] ){
		case 'eng':
			echo $eng;
		break;
		case 'ru':
			echo $rus;
		break;
		
	}
	return true;	
}
?>

Comments

[shushu]

Немного перемудрил :)
Вот, это наверное должно работать:

<?php
function translate($rus,$eng)
{
	$lang = isset( $_GET['lang'] ) ? $_GET['lang'] : '';
	$cookie_fresh = mktime(23,59,59,12,31,date('Y')+1);
	
	switch( $lang ){
		case 'ru':
			setcookie('lang', 'ru', $cookie_fresh);
			header("Location: ./");
			exit();
		break;
		case 'ru':
			setcookie('lang', 'eng', $cookie_fresh);
			header("Location: ./");
			exit();
		break;
	}
	
	$c_lang = isset( $_COOKIE['lang'] ) ? $_COOKIE['lang'] : 'ru';
	
	switch( $_COOKIE['lang'] ){
		case 'eng':
			echo $eng;
		break;
		case 'ru':
			echo $rus;
		break;
		
	}
	return true;	
}
?>

[Artqookie]

Флажок нигде не использован, не?

Answer 4

[Horzerus]

1) В самом начале скрипта напишите вызов функции:
ob_start();
чтобы весь вывод кешировался и возвращался браузеру в конце работы скрипта.
Это поможет избавиться от warning-ов «Cannot modify header information...», чтобы они пока не мешались…
Если всеравно такие warnings будут, то значит что-то все-таки выводится перед вызовом ob_start(). Например, что-то может быть перед первым "<?php" (это могут быть невидивые символы utf8)

2) Сама по себе функция «не работать» не может, если она объявлена и вызывается. Поэтому просто «дебажте код» — добавьте во все ключевые места (перед вызовом, в самой функции в начале вызова, в тело if которые должно выполниться и т.д.) echo «XX\n»;
XX — числа по порядку или слова. И проанализируйте, что будет выведено.

Если нету других способов дебага, то этот способ помогает быстро локализовать место, где код невыполняется как хотелось бы…

Также, может помочь вывод var_dump($_COOKIE); чтобы быть уверенным что кука пришла от браузера.

И еще, на всякий случай, в строке:
$lang = $_GET['lang'];
если атрибут «lang» не передан в URL, то в этом месте будет ошибчка — нет такого index в массиве.
Надо бы, по хорошему, что-то типа этого:
$lang = isset($_GET['lang'])? $_GET['lang']: '';

Comments

[Horzerus]

3) после header(«Location: ...»); всегда добавляйте die();

[Artqookie]

Так развернуто… Благодарю.

Answer 5

[Александр]

Наверное проблема в том, что не инклудится файл с фукнцией.

Comments

[Artqookie]

Функция находится на той же странице. Сайт (страничка) — одна. И прямо в ней все функции.

Answer 6

[Shvonder]

Проверьте, правильное ли время на сервере? Правильное ли на компьютере?

Comments

[Shvonder]

Функция написана некорректно. При многократном ее вызове заголовки будут отправляться после echo

[Shvonder]

укажите домен в setcookie

[Shvonder]

setcookie('lang', 'eng', $cookie_fresh, '/');

Answer 7

[Bruteo]

$lang = $_GET['lang'];
заменить на $_COOKIE['lang']

Comments

[Bruteo]

Нет, не так

Answer 8

[Анатолий]

На сколько я знаю функция setcookie не изменяет массив $_COOKIE дальше в скрипте, лишь после перезагрузки, возможно в этом дело.

Answer 9

[DileSoft]

header(«Location: ./»); — вот это меня сильно пугает. Лучше используйте абсолютный путь.

Answer 10

[MNX]

Внесу свою лепту — возможно дело в UTF-8 BOM.