function connect($host,$user,$password,$table){
$mysql_C = mysql_connect($host,$user,$password);
mysql_select_db($table,$mysql_C);
}


function sql($sql){
$mysql_Q = mysql_query($sql);
global $mysql;
$mysql = mysql_fetch_array($mysql_Q);
}

Без этих функций,используя дедовский метод mysql_connect Тупо и тд все работает!

mid
5 месяцев, уже пора: http://irbis-team.com/15/1/8

Переделывай. Уважай себя и читателей. Ты же не 1 строчку даёшь.

inpost ,конечно,сейчас все сделаю как нужно!Просто нг помогаю на стол накрывать и тружусь:)
Сейчас исправлюсь!

<?php
function connect($host,$user,$password,$table){
$mysql_C = mysql_connect($host,$user,$password);
mysql_select_db($table,$mysql_C);
}

$mysql = mysql_fetch_array($mysql_Q);

function sql($sql){
$mysql_Q = mysql_query($sql);
global $mysql;
}

вот так нада

а чему равна $sql?
а это че такое?

sql("SELECT `title`,`content` FROM `index` WHERE `page`='$page'");

johniek_comp,пользовательские функции:)
Нее,то что предложил:

function connect($host,$user,$password,$table){
$mysql_C = mysql_connect($host,$user,$password);
mysql_select_db($table,$mysql_C);
}

$mysql = mysql_fetch_array($mysql_Q);

Совершенно не то)))
И если даже делать так,то нужно на переменную $mysql_Q в функции global прописать или return.

Беда..Надо выпить,тогда и дело само собой решиться :D :D :D

:rolleyes:

надо начинать пить

johniek_comp

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

Вот и написал

inpost
Я раз почитал попробовал что это за глобал, и применил сразу вот так

$login = "admin";                         
$password = "admin";                      
$dbname = "news";                      
$sqlhost = "localhost";    
function mysqlconnect ()
 {
 global $dbname;
 global $sqlhost;
 global $sqlusername;
 global $sqlpassword;
 global $connection;
 $connection = mysql_connect($sqlhost,$sqlusername,$sqlpassword) or die (mysql_error());
 $db = mysql_select_db($dbname) or die (mysql_error());
 }  

у меня работает, наверно запутался где-то если говоришь что не прав.

А $sql это что то типо условия..ну если

function mid($name){
echo $name;
}

то если использовать функцию так mid("ВАсек")
То в итоге выведеться: Васек

Можно без функции..Но..Мне хочется научится работать с функциями..Тем более когда много sql Запросов функция sql которая у меня там есть,очень упрощает жизнь:)

Ещё к ней надо дописать проверку,типо если есть уже такие переменные,которые создаются в той функции,удалять их,а остальное все оставить..

Все понял.Вот тут нужно было не так:

function sql($sql){
$mysql_Q = mysql_query($sql);
global $mysql;
$mysql = mysql_fetch_array($mysql_Q);
}

А так

function sql($sql){
$mysql_Q = mysql_query($sql);
$mysql = mysql_fetch_array($mysql_Q);
return $mysql;
}

Вот итог

<?php
function connect($host,$user,$password,$table){
$mysql_C = mysql_connect($host,$user,$password);
mysql_select_db($table,$mysql_C);
}


function sql($sql){
$mysql_Q = mysql_query($sql);
$mysql = mysql_fetch_array($mysql_Q);
return $mysql;
}


function shablonezator()
{
$page = $_GET['page'];
if($page == '')
{
$url = "index.php?page=index";
header("Location:$url");
}
connect("localhost","cms","cms","cms");
$mysql_array = sql("SELECT `title`,`content` FROM `index` WHERE `page`='$page'");
$title = $mysql_array['title'];
$content = $mysql_array['content'];
$time = date("H");
if($time == '7' or $time == '8' or $time == '9' or $time == '10' or $time == '11')
{
$tpltitle = file_get_contents("templates/u/title.tpl");
$tplcontent = file_get_contents("templates/u/content.tpl");
$index = str_replace('{TITLE}',$title,$tpltitle); 
$index .= str_replace('{CONTENT}',$content,$tplcontent);
echo $index;
exit();
}
if($time == '12' or $time == '13' or $time == '14' or $time == '15' or $time == '16' or $time == '17' or $time == '18' )
{
$tpltitle = file_get_contents("templates/d/title.tpl");
$tplcontent = file_get_contents("templates/d/content.tpl");
$index = str_replace('{TITLE}',$title,$tpltitle); 
$index .= str_replace('{CONTENT}',$content,$tplcontent);
echo $index;
exit();
}
if($time == '19' or $time == '20')
{
$tpltitle = file_get_contents("templates/v/title.tpl");
$tplcontent = file_get_contents("templates/v/content.tpl");
$index = str_replace('{TITLE}',$title,$tpltitle); 
$index .= str_replace('{CONTENT}',$content,$tplcontent);
echo $index;
exit();
}
if($time == '21' or $time == '22' or $time == '23' or $time == '24' or $time == '1' or $time == '2' or $time == '3' or $time == '4' or $time == '5' or $time == '6')
{
$tpltitle = file_get_contents("templates/n/title.tpl");
$tplcontent = file_get_contents("templates/n/content.tpl");
$index = str_replace('{TITLE}',$title,$tpltitle); 
$index .= str_replace('{CONTENT}',$content,$tplcontent);
echo $index;
exit();
}
}
?>

Winston
фига се...уже выпил за столом?

johniek_comp
Представим на секундочку, что каждый твой шаг должен быть логичный абсолютно!
У нас есть переменная (значение, которое меняется) и Константа (которая не меняется, одна навсегда).
Теперь у нас есть параметры для соединения с БД ОДНИ навсегда, значит это у что? Правильно, константа(!), прямая логика. А константа видна везде, она суперглобальная, и постоянно global писать не надо. И твои 10 строк можно заменить на 5, + выиграть в обычной логике приложений.

mid
То я не тебе писал. Тебе - изучи стиль, исправь и выстави обновленный код, если хочешь, чтобы я помог тебе

mid, зачем ты в каждой странице подключаешь базу? не легче создать файл, записать туда данные подключения, и просто инклюдировать?