Аукцион чаров!

[ExDragon]

ExAuction - Это сайт реализован для MaNGOS (планируется и Trinity Core), в него входят возможности:
- Выставлять своих персонажей на аукцион для обмена.
- Смена ника, пола, (расы, фракции - требуется патч в ядро).
- Смена пароля учётной записи.
- Смена e-mail учётной записи.
- Регистрация новой учётной записи.

Последняя стабильная версия 0.29
Git

git://github.com/ExDragon/ExAuction.git

[ExDragon]

обновил до 0.2.5
Добавил безопасности
Модуль "Новости"
"404"
"getNow"

[tempura]

Может еще пару скринов для шапки? Красимшее будет, развесистее.

[ExDragon]

0.3.1 Сделал новый стиль! вот теперь он даже мне нравится
Какие ещё будут предложения может что то добавить?

[alien]

	function Plus($x)
	{
		$x++;
		return $x;
	}
	function Minus($x)
	{
		$x--;
		return $x;
	}

Лучше уж так.

	function Plus($x)
	{
		return ++$x;
	}
	function Minus($x)
	{
		return --$x;
	}

Да и зачем вобще эти функции ))

[ExDragon]

0.3.2
Расширил аукцион.. Убрал "Лоты"
Добавил "Сервер" - Статистика, кто онлайн

[Кот ДаWINчи]

Сообщение от ExDragon

0.3.2
Расширил аукцион.. Убрал "Лоты"
Добавил "Сервер" - Статистика, кто онлайн

Похоже "проект" развивается по принципу: заложили подводную лодку, а со стапелей выйдет дирижабль...

Лучше бы не распылялся по-мелочам, а сделал действительно юзабельный скрипт аукциона. С удобным, полнофункциональным, дружественным интерфейсом, а статы, онлайн и другой "стандартный" контент игрового сервера ВОВ добавил бы потом.

Пока что дырявость скрипта остается... автор же не хочет доводить своё детище до кондиции. Он просто, как майкрософт, начинает обвешивать скрипт ненужными фенечками.

и это огорчает.

[ExDragon]

Кот ДаWINчи
и что же не нравится в моём "аукционе"?
И где же "дырки" в коде? я уже все запросы проверил..
а вот что я разошелся вы правы!
Да, пожалуй сделаю удобный аукцион бес дополнительных модулей О_0 я всё изменю, положу в основу 0.4.0

Напишу всё с нуля!
А теперь мне бы очень было интересно посмотреть на "дырки"

[Sid]

Ну я думаю перед подачей в любой запрос, переменную надо прогнать хотя бы через mysql_real_escape_string() если она вводилась пользователем.

[srv38]

По мне так лучше использовать готовую библиотеку для работы с базой, коих полным полно, я допустим в своих проектах пользую "DbSimple"

		$race_1 = mysql_query('SELECT * FROM `characters` WHERE `race` = "1"');
		$row_race_1 = mysql_num_rows($race_1);//Alliance
		$race_2 = mysql_query('SELECT * FROM `characters` WHERE `race` = "2"');
		$row_race_2 = mysql_num_rows($race_2);//Horde
		$race_3 = mysql_query('SELECT * FROM `characters` WHERE `race` = "3"');
		$row_race_3 = mysql_num_rows($race_3);//Alliance
		$race_4 = mysql_query('SELECT * FROM `characters` WHERE `race` = "4"');
		$row_race_4 = mysql_num_rows($race_4);//Alliance
		$race_5 = mysql_query('SELECT * FROM `characters` WHERE `race` = "5"');
		$row_race_5 = mysql_num_rows($race_5);//Horde
		$race_6 = mysql_query('SELECT * FROM `characters` WHERE `race` = "6"');
		$row_race_6 = mysql_num_rows($race_6);//Horde
		$race_7 = mysql_query('SELECT * FROM `characters` WHERE `race` = "7"');
		$row_race_7 = mysql_num_rows($race_7);//Alliance
		$race_8 = mysql_query('SELECT * FROM `characters` WHERE `race` = "8"');
		$row_race_8 = mysql_num_rows($race_8);//Horde
		$race_10 = mysql_query('SELECT * FROM `characters` WHERE `race` = "10"');
		$row_race_10 = mysql_num_rows($race_10);//Horde
		$race_11 = mysql_query('SELECT * FROM `characters` WHERE `race` = "11"');
		$row_race_11 = mysql_num_rows($race_11);//Alliance
		$char = mysql_query('SELECT * FROM `characters`');
		$char_all = mysql_num_rows($char);
		//All Guide
		$guild = mysql_query('SELECT * FROM `guild`');
		$guild_all = mysql_num_rows($guild);
		//All Char in guide
		$guild_member = mysql_query('SELECT * FROM `guild_member`');
		$guild_member_all = mysql_num_rows($guild_member);
		//All NPC
		mysql_select_db($m_db, $cont);
		$npc = mysql_query('SELECT * FROM `npc_vendor`');
		$npc_all = mysql_num_rows($npc);
		//All Item
		$item = mysql_query('SELECT * FROM `item_template`');
		$item_all = mysql_num_rows($item);
		//All Gameobject
		$gameobject = mysql_query('SELECT * FROM `gameobject_template`');
		$gameobject_all = mysql_num_rows($gameobject);
		//All Quest
		$quest = mysql_query('SELECT * FROM `quest_template`');
		$quest_all = mysql_num_rows($quest);

руки бы тебе оторвать за оптимизированные запросы...
http://www.mysql.ru/docs/man/Counting_rows.html читать до понимания

[Sid]

Сообщение от srv38

По мне так лучше использовать готовую библиотеку для работы с базой, коих полным полно, я допустим в своих проектах пользую "DbSimple"

на сколько мне известно под пхп 5.3+ не идет.

Сообщение от Sid

на сколько мне известно под пхп 5.3+ не идет.

что значит не идет? Все идет, едет и работает.По крайней мере её mysql часть

[srv38]

Есть форк https://github.com/ivan1986/DbSimple можно его посмотреть, самому пока некогда.

[tempura]

Сообщение от Sid

на сколько мне известно под пхп 5.3+ не идет.

Просто так "не ходить" не может - должна быть причина. А если причина есть, то и исправить ее можно.
Скорее всего на первых версиях не было фигни какой-нибудь, но сейчас-то точно должны были починить.

[KiriX]

Сообщение от Sid

на сколько мне известно под пхп 5.3+ не идет.

Простенький и базовый функционал работает прекрасно.

[Sid]

Ну что вы набросились я ж в пхп не спец) просто помню ошибки получил когда на 5.3 запустил

[tempura]

Сообщение от Sid

Ну что вы набросились я ж в пхп не спец) просто помню ошибки получил когда на 5.3 запустил

Это не ошибки были. Ошибки - это когда ты в логе видишь записи и их анализируешь. Может быть даже исправляешь. А когда просто "чота написали, ну я и забил" - это НЕ ошибки.

Я помню у у меня на первых версиях РНР 5.3.х ошибки были: модуль интербазы не собирался под нее, потому что поддержки его не было. И через pdo с файрбёрдом не работало - опять же потому что не было поддержки. Вот это была ошибка - всем ошибкам ошибка.

Помнится тогда же мне на скаку пришлось исправлять много ereg на preg_match (кажется этот оно тогда было? а то не вспомню точно) - вот это уже были не ошибки, а вполне закономерные последствия того, что я не прочел чейнджлог ПЕРЕД установкой. Может и у тебя такие же "ошибки"?

[Sid]

Сообщение от tempura

Помнится тогда же мне на скаку пришлось исправлять много ereg на preg_match (кажется этот оно тогда было? а то не вспомню точно) - вот это уже были не ошибки, а вполне закономерные последствия того, что я не прочел чейнджлог ПЕРЕД установкой. Может и у тебя такие же "ошибки"?

оно самое

[Праведник]

Сообщение от Sid

Ну что вы набросились я ж в пхп не спец) просто помню ошибки получил когда на 5.3 запустил

там не ошибка) там нотайс)

достаточно заменить в файлах "=?" на "=" и всё будет в шоколаде.

[ExDragon]

Я попробовал использовать "DbSimple" но получилось..

Warning: mysql_errno(): supplied argument is not a valid MySQL-Link resource in /home/dklab/domains/www/lib/DbSimple/demo/lib/DbSimple/Mysql.php on line 184
Warning: mysql_error(): supplied argument is not a valid MySQL-Link resource in /home/dklab/domains/www/lib/DbSimple/demo/lib/DbSimple/Mysql.php on line 184
SQL Error: at /home/dklab/domains/www/lib/DbSimple/demo/test/DbSimple/connect.php line 6
Array
(
    |code] => 
    [message] => 
    [query] => mysql_connect()
    [context] => /home/dklab/domains/www/lib/DbSimple/demo/test/DbSimple/connect.php line 6
)

[tempura]

Прикольно. Хост-порт-логин-пароль забыл написать?

$lDB = DbSimple_Generic::connect("mysql://".$this->_config['db_user'].":".$this->_config['db_pw']."@".$this->_config['db_ip']."/".$this->_config['lk_db']);

[tempura]

Судя по тому, что тут из конфига запрашиваются данные - это шестая строка из connect.php?

Сообщение от tempura

Судя по тому, что тут из конфига запрашиваются данные - это шестая строка из connect.php?

неее, это я эксперементирую с кодом афтора...
точнее переношу всё в ООП: smarty, dbsimple, mvc и т.п.
чесно говоря, проще переписать заного, используя фреймверк; но раз уж человек отказывается от "велосипеда".....

[ExDragon]

Minimajackчто я не отказываюсь, я не знаю как этим пользоваться!
и так ООП хм нужно написать главный класс.. $this->_config['db_ip']
это что? помогите написать.. а лучше киньте ссылку на статью о ООП (кроме php.su) на русском языке

модуль_лист

$modul = array (
//	Modul				  Sub module			  Path sub module				V*	V**		Name
	'news' 		=> 	array('main'		=>	array("modules/news/main.php",		1,	-1,		4),
						  'edit'		=>	array("modules/news/edit.php",		1,	-1,		15)	),
	'login'		=> 	array('main'		=>	array("modules/login.php",			1,	-1,		19)	),
	'reg' 		=> 	array('main'		=>	array("modules/reg.php",			1,	-1,		20)	),
	'message' 	=> 	array('main'		=>	array("modules/message.php",		1,	-1,		21)	),
//	Block			Path
	'menu' 		=> 	"modules/block/menu.php",
	'yeslogin' 	=> 	"modules/block/yeslogin.php",
	'nologin' 	=> 	"modules/block/nologin.php",
	'404'		=> 	"modules/404.php"
);
//V* - viwe sub module: 1 - true, 0 - false
//V** - Viwe for GMLevel: -1 - Dont't reg user, 0 - Login user, 1 - Moderator, 2 - GM, 3 - Admin

.htaccess

DirectoryIndex index.php
RewriteEngine On
Options +FollowSymlinks
RewriteBase /
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule (.+) /index.php?m=$1 [L]

класс

	function Viwe($viwe, $dont)
	{
		if (isset($viwe)) return $viwe;
		else return $dont;
	}
	function getModule()
	{
		$do = explode('/', $_GET['m']); 
		$m = preg_replace('/[^a-zA-Z0-9\s]/', '', $do[0]);
		$s = preg_replace('/[^a-zA-Z0-9\s]/', '', $do[1]);
		
		$mod = $this->Viwe($do[0], $this->default_module);
		$sub = $this->Viwe($do[1], 'main');
		
		if ($this->modul[$mod][$sub][1] == 1)
		{
			if ($this->Viwe($_SESSION['gm'], -1) >= $this->modul[$mod][$sub][2]) return $this->modul[$mod][$sub][0];
			else return $this->modul[$this->default_module]['main'][0]; 
				
		}
		else return $this->modul['404']; 
	}

ой какой кошмар )

я переделал модульность.. лучше или хуже?

[Lordronn]

http://www.tormix.com/index.php?do=a...=6&s=oop_v_php
http://i-novice.net/osnovy-oop-prodolzhenie/
http://forum.0day.kiev.ua/index.php?showtopic=146080

[tempura]

ExDragon
Все просто ужасно. Начиная с англ. языка. Что означает слово "viwe", так часто использующееся?

Сообщение от ExDragon

я переделал модульность.. лучше или хуже?

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

[ExDragon]

хм а можно пример..

я тебе уже 2 раза писал...
вот тут пример
только понятие модуля надо или удалить или расширить( как набор: контроллер, модели и вид)
ниже идут классы, которые удобно использовать для написания своих...
читать про шаблоны программирования.

пример, синглетон( php > 5.3 )

<?php
class Registry extends ArrayObject{
	private static $_instance;
	private static $_registry = null;
	private static $_registryClassName = 'Registry';
	
	public function __construct( )
	{
	}
    
    public static function getInstance()
    {
        if (self::$_registry === null) {
            self::setInstance(new self::$_registryClassName());
        }

        return self::$_registry;
    }
    
    public static function setInstance(Registry $registry)
    {
        self::$_registry = $registry;
    }
    
	public static function get($index)
	{
		$instance = self::getInstance();
		return $instance->offsetGet($index);
	}
	
	public static function set($index, $value)
	{
		$instance = self::getInstance();
		$instance->offsetSet($index, $value);
	}
}

только кашу кода придется чистить...там черт ногу сломит
зы если нужен пример правильной модульности прочитай( посмотри код ) про любой ООП фреймверк с mvc

[ExDragon]

Отона где собака зарыта http://ru.wikipedia.org/wiki/MVC

помогите сделать шаблонизатор в моём случае http://sourceforge.net/projects/exbl...w.rar/download

[KiriX]

Сообщение от ExDragon

помогите сделать шаблонизатор в моём случае http://sourceforge.net/projects/exbl...w.rar/download

Что значит "помогите"??? Ты попытайся для начала сам сделать. Если возникнут трудности, уверен, тебе помогут.
А то, что говоришь ты можно трактовать как "сделайте за меня". Этим точно никто "за тебя" заниматься не станет...

Сообщение от ExDragon

помогите сделать шаблонизатор в моём случае http://sourceforge.net/projects/exbl...w.rar/download

Шаблонизаторы

[ExDragon]

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

[tempura]

Так а в чем именно тупик-то?
Проблема открыть файл?


Кстати, а почему на сорсфорже нет (я не вижу) собственно файлов-то? Мощный рабочий инструмент используется исключительно как архив - это дерьмо (простите) собачье.
Ой, сорри... Нашел...

Сообщение от ExDragon

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

пример

<?php
require_once('includes/Smarty/Smarty.class.php');

class View extends Smarty{
	function View()
	{
		parent::__construct();
		$config = Registry::get('config');
		$this->left_delimiter = '{';
		$this->right_delimiter =  '}';
		$this->debugging = true;
		$this->compile_check = true;
		$this->allow_php_tag = true;
		$this->template_dir = 'skin/'.$config['skin'];
		$this->compile_dir = 'templates_c/'.$config['skin'].'/';
	}
}

устроит?

[tempura]

вот еще момент... Меня как-то совершенно убивают куски:

header("Location: http://".$_SERVER['HTTP_HOST']."

и лучше бы без них обойтись. Мне кажется, что вместо (например)

header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&news=1");

достаточно будет писать

header("Location: /".$lk_dir."/?m=message&news=1");

и это будет гораздо предпочтительнее по многим причинам.



Еще вот такие циклы:

102 //---------
103 if (isset($_POST['lotsDEL'])){
104         if ($auc->ControlGuid_User($_POST['lotsDEL'])){
105                 if ($auc->DEL_Auction($_POST['lotsDEL'])){
106                         header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=1");
107                 } else {
108                         header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
109                 }
110         } else {
111                 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
112         }
113 }
114 //---------
115 if (isset($_POST['lotsSHOP_DEL'])){
116         if (!$auc->ControlGuid_User($_POST['lotsSHOP_DEL'])){
117                 if ($auc->DEL_Shop($_POST['lotsSHOP_DEL'])){
118                         header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=2");
119                 } else {
120                         header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=3");
121                 }
122         } else {
123                 header("Location: http://".$_SERVER['HTTP_HOST']."/".$lk_dir."/?m=message&d=0");
124         }
125 }

Вызывают стойкое ощущение дежавю. Может быть их как-то можно сократить в одну функцию с подстановкой переменных? (сорри, я в терминах не менделеев, может это и не "функции" с "переменными" называется)

[ExDragon]

насчёт SVN как можно быстро работать с этим? а то черепашка ну очень долго грузится...
я ещё не залил на свн...

Смарт.. говорят весьма проблемная по...

[Sid]

лучше гит или меркуриал, они хотя бы историю локально сохраняют.

[virusav]

Говорят, что репозитарий на гитхабе платный.