| |
|
|
| | скажите пожалуйства на ваше усмотрение ниже написанный подход есть ли рациональным?
<?
$tabl= array(1=>'aaa',2=>'bbb',3=>'ccc');
$templ=array(1=>'main.tpl',2=>'basic.tpl',3=>'info.tpl');
$tab=$_GET['tabl'];
$tpl=$_GET['templ'];
function get_data($tab,$tpl,$smarty)
{
$query = "SELECT * FROM $tab";
$result = mysql_query($query);
.......
$smarty->display($tpl);
}
switch ($page)
{
case 'basic':
get_data($tab,$tpl,$smarty);
break;
}
?>
|
| |
| |
|
|
| |
|
|
| |
для: moonfox
(28.11.2007 в 22:36)
| | | А зачем нужны два массива, объявленные в самом начале кода, если Вы их не используете? | |
| |
|
|
| |
|
|
| |
для: Unkind
(28.11.2007 в 22:59)
| | | очепятка вышла
$query = "SELECT * FROM $tabl[$tab]";
$smarty->display($templ[$tpl]); | |
| |
|
|
| |
|
|
| |
для: moonfox
(28.11.2007 в 23:07)
| | | Тогда первой строчкой в теле функции надо поставить global $tabl, $templ;
И неплохо было бы обезопасить себя от ошибок, которые возникнут при неверном имени таблицы/шаблона.
<?php
//...
$tab = isset( $tabl[$tab] ) ? $tabl[$tab] : '';
$tpl = isset( $templ[$tpl] ) ? $templ[$tpl] : '';
if( $tab == '' || $tpl == '' ) die( 'Неверные данные' );
//...
?>
|
| |
| |
|
|
| |
|
|
| |
для: Unkind
(28.11.2007 в 23:50)
| | | Как это говориться там? У дураков мысли сходятся? Причем в одну и ту же минуту? :) | |
| |
|
|
| |
|
|
| |
для: Zirkon
(28.11.2007 в 23:52)
| | | > Как это говориться там? У дураков мысли сходятся?
Не знаю где это там и что там говорится. | |
| |
|
|
| |
|
|
| |
для: Unkind
(29.11.2007 в 00:38)
| | | Шутка :) | |
| |
|
|
| |
|
|
| |
для: moonfox
(28.11.2007 в 23:07)
| | | Как мне кажется, для начала неплохо было бы объявить эти массивы как глобальные, для примера в том же get_data. :) | |
| |
|
|
| |
|
|
| |
для: Zirkon
(28.11.2007 в 23:50)
| | | get_data($tab,$tpl,$smarty,$tabl, $templ)
или
global $tabl, $templ;
с этим ястно... переменная то в не фукции
меня просто интересовала рациональность самого подхода
Zirkon: да вы же видели что я вношу в get_data значения внешних переменных уже не важно что в уточнении опустил глобализацию то.
спасибо за помощь! | |
| |
|
|
