Клубове

Бисери ....

7 отговора 523 прегледа
Bълk #1

Бисери ....

еххх, по едно време публикувах нещата от едни индийци ... снощи ми се е паднал един сайт, не знам чие творение е. Просто ще публикувам част от този опус: 


if(isset("picid")):

	$id=$_SESSION["picid"];

else:

	if(session_is_registered("gid")): 

		$url=$HTTP_SESSION_VARS["gid"];

		header("Location: gallery.php?gid=$url");

	else:

		header("Location:index.php");

	endif;

endif;


забележка: всичко това е в началото на файл, picid си $_SESSION['picid'] не са дефинирани at all

започвам да си мисля, че думички "гений" и "гениталии" имат много общо помежду си.

-----------------------
гърбавия - ковчег ще го изправи

Редактирано от Bълk на

Отговор
voyager в отговор на Bълk #2

Re: Бисери ....



priatelite.net - Твоите приятели!
Отговор
Дeшeв в отговор на Bълk #3

Re: Бисери ....

Логин логиката на phpBB. По-гнусно нещо не съм виждал. Такова нещо променяне няма .

================================ 
]define("IN_LOGIN", true);



define('IN_PHPBB', true);

$phpbb_root_path = './';

include($phpbb_root_path . 'extension.inc');

include($phpbb_root_path . 'common.'.$phpEx);



//

// Set page ID for session management

//

$userdata = session_pagestart($user_ip, PAGE_LOGIN);

init_userprefs($userdata);

//

// End session management

//



// session id check

if (!empty($HTTP_POST_VARS['sid']) || !empty($HTTP_GET_VARS['sid']))

{

	$sid = (!empty($HTTP_POST_VARS['sid'])) ? $HTTP_POST_VARS['sid'] : $HTTP_GET_VARS['sid'];

}

else

{

	$sid = '';

}



if( isset($HTTP_POST_VARS['login']) || isset($HTTP_GET_VARS['login']) || isset($HTTP_POST_VARS['logout']) || isset($HTTP_GET_VARS['logout']) )

{

	if( ( isset($HTTP_POST_VARS['login']) || isset($HTTP_GET_VARS['login']) ) && !$userdata['session_logged_in'] )

	{

		$username = isset($HTTP_POST_VARS['username']) ? trim(htmlspecialchars($HTTP_POST_VARS['username'])) : '';

		$username = substr(str_replace("\\'", "'", $username), 0, 25);

		$username = str_replace("'", "\\'", $username);

		$password = isset($HTTP_POST_VARS['password']) ? $HTTP_POST_VARS['password'] : '';



		$sql = "SELECT user_id, username, user_password, user_active, user_level

			FROM " . USERS_TABLE . "

			WHERE username = '" . str_replace("\\'", "''", $username) . "'";

		if ( !($result = $db->sql_query($sql)) )

		{

			message_die(GENERAL_ERROR, 'Error in obtaining userdata', '', __LINE__, __FILE__, $sql);

		}



		if( $row = $db->sql_fetchrow($result) )

		{

			if( $row['user_level'] != ADMIN && $board_config['board_disable'] )

			{

				redirect(append_sid("index.$phpEx", true));

			}

			else

			{

				if( md5($password) == $row['user_password'] && $row['user_active'] )

				{

					$autologin = ( isset($HTTP_POST_VARS['autologin']) ) ? TRUE : 0;



					$session_id = session_begin($row['user_id'], $user_ip, PAGE_INDEX, FALSE, $autologin);



					if( $session_id )

					{

						$url = ( !empty($HTTP_POST_VARS['redirect']) ) ? str_replace('&', '&', htmlspecialchars($HTTP_POST_VARS['redirect'])) : "index.$phpEx";

						redirect(append_sid($url, true));

					}

					else

					{

						message_die(CRITICAL_ERROR, "Couldn't start session : login", "", __LINE__, __FILE__);

					}

				}

				else

				{

					$redirect = ( !empty($HTTP_POST_VARS['redirect']) ) ? str_replace('&', '&', htmlspecialchars($HTTP_POST_VARS['redirect'])) : '';

					$redirect = str_replace('?', '&', $redirect);



					$template->assign_vars(array(

						'META' => "<meta http-equiv=\"refresh\" content=\"3;url=login.$phpEx?redirect=$redirect\">")

					);



					$message = $lang['Error_login'] . '<br /><br />' . sprintf($lang['Click_return_login'], "<a href=\"login.$phpEx?redirect=$redirect\">", '</a>') . '<br /><br />' .  sprintf($lang['Click_return_index'], '<a href="' . append_sid("index.$phpEx") . '">', '</a>');



					message_die(GENERAL_MESSAGE, $message);

				}

			}

		}

		else

		{

			$redirect = ( !empty($HTTP_POST_VARS['redirect']) ) ? str_replace('&', '&', htmlspecialchars($HTTP_POST_VARS['redirect'])) : "";

			$redirect = str_replace("?", "&", $redirect);



			$template->assign_vars(array(

				'META' => "<meta http-equiv=\"refresh\" content=\"3;url=login.$phpEx?redirect=$redirect\">")

			);



			$message = $lang['Error_login'] . '<br /><br />' . sprintf($lang['Click_return_login'], "<a href=\"login.$phpEx?redirect=$redirect\">", '</a>') . '<br /><br />' .  sprintf($lang['Click_return_index'], '<a href="' . append_sid("index.$phpEx") . '">', '</a>');



			message_die(GENERAL_MESSAGE, $message);

		}

	}

	else if( ( isset($HTTP_GET_VARS['logout']) || isset($HTTP_POST_VARS['logout']) ) && $userdata['session_logged_in'] )

	{

		if( $userdata['session_logged_in'] )

		{

			session_end($userdata['session_id'], $userdata['user_id']);

		}



		if (!empty($HTTP_POST_VARS['redirect']) || !empty($HTTP_GET_VARS['redirect']))

		{

			$url = (!empty($HTTP_POST_VARS['redirect'])) ? htmlspecialchars($HTTP_POST_VARS['redirect']) : htmlspecialchars($HTTP_GET_VARS['redirect']);

			$url = str_replace('&', '&', $url);

			redirect(append_sid($url, true));

		}

		else

		{

			redirect(append_sid("index.$phpEx", true));

		}

	}

	else

	{

		$url = ( !empty($HTTP_POST_VARS['redirect']) ) ? str_replace('&', '&', htmlspecialchars($HTTP_POST_VARS['redirect'])) : "index.$phpEx";

		redirect(append_sid($url, true));

	}

}

else

{

	//

	// Do a full login page dohickey if

	// user not already logged in

	//

	if( !$userdata['session_logged_in'] )

	{

		$page_title = $lang['Login'];

		include($phpbb_root_path . 'includes/page_header.'.$phpEx);



		$template->set_filenames(array(

			'body' => 'login_body.tpl')

		);



		if( isset($HTTP_POST_VARS['redirect']) || isset($HTTP_GET_VARS['redirect']) )

		{

			$forward_to = $HTTP_SERVER_VARS['QUERY_STRING'];



			if( preg_match("/^redirect=([a-z0-9\.#\/\?&=\+\-_]+)/si", $forward_to, $forward_matches) )

			{

				$forward_to = ( !empty($forward_matches[3]) ) ? $forward_matches[3] : $forward_matches[1];

				$forward_match = explode('&', $forward_to);



				if(count($forward_match) > 1)

				{

					$forward_page = '';



					for($i = 1; $i < count($forward_match); $i++)

					{

						if( !ereg("sid=", $forward_match[$i]) )

						{

							if( $forward_page != '' )

							{

								$forward_page .= '&';

							}

							$forward_page .= $forward_match[$i];

						}

					}

					$forward_page = $forward_match[0] . '?' . $forward_page;

				}

				else

				{

					$forward_page = $forward_match[0];

				}

			}

		}

		else

		{

			$forward_page = '';

		}



		$username = ( $userdata['user_id'] != ANONYMOUS ) ? $userdata['username'] : '';



		$s_hidden_fields = '<input type="hidden" name="redirect" value="' . $forward_page . '" />';



		make_jumpbox('viewforum.'.$phpEx, $forum_id);

		$template->assign_vars(array(

			'USERNAME' => $username,



			'L_ENTER_PASSWORD' => $lang['Enter_password'],

			'L_SEND_PASSWORD' => $lang['Forgotten_password'],



			'U_SEND_PASSWORD' => append_sid("profile.$phpEx?mode=sendpassword"),



			'S_HIDDEN_FIELDS' => $s_hidden_fields)

		);



		$template->pparse('body');



		include($phpbb_root_path . 'includes/page_tail.'.$phpEx);

	}

	else

	{

		redirect(append_sid("index.$phpEx", true));

	}



}


--
Екстремно програмиране на български.
[линк]
Отговор
Bълk в отговор на Дeшeв #4

Re: Бисери ....

баси ужаса .....

-----------------------
гърбавия - ковчег ще го изправи
Отговор
voyager в отговор на Дeшeв #5

Re: Бисери ....

Мда
Аз изобщо по-гнусно нещо то пхпбб май не съм виждал

priatelite.net - Твоите приятели!
Отговор
Дeшeв в отговор на voyager #6

Re: Бисери ....

Интересното на софтуер като пхпбб (а и други такива съм виждал) е, че сигурно автора се кефи на това, което е създал. Ако се заядеш можеш да кажеш: "глей ся, батка, имам си абстракция на достъпа до данните, отделил съм логиката от презентацията с темплейти. 3-слойна архитектура -- бе изобщо шедьовър."

Кой ти гледа, че кода е нечитаем и непроменяем .

--
Екстремно програмиране на български.
[линк]
Отговор
voyager в отговор на Дeшeв #7

Re: Бисери ....

Ами той нали знаеш, че авторът не е един. Цяла армия от опънатосорсаджии се гордеят каква невероятна абстракция са създали . Толкова абстрактно, че и те не знаят кое как работи.
Все се каня да извадя малко коментари от постнюке (друг подобен шедьовър) - един от любимите ми, който срещам няколко пъти там, беше нещо такова:

//Why is this here? Keep it commented for now

priatelite.net - Твоите приятели!
Отговор
Na4inae6t в отговор на Bълk #8

Обяснете на ламера

какво толкова му е ло6ото на първия скрипт, мисля 4е може да е сетната променливата в друга страница и може би трябва да се напи6е така
"
if(isset($_SESSION["picid"])):

$id=$_SESSION["picid"];

"

Отговор
Отговор в темата