вопрос по php от начинающего

[sleeper_Izh]

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

Захотел сделать это всё в одном скрипте.
определяю какая кнопка нажата( редактирование, удаление записи и т.д.) по свойству "hidden" в форме.

И вот получается такой код(точнее отрывок):

if (count($_REQUEST)==1 or isset($_REQUEST['choose_page']))
{
	print_table($_REQUEST['page']);
	main_page();
}
else
{
	if (isset($_REQUEST["add_student"]))
	{
		if ($_REQUEST["add_student"]=="step1")
		{
			add_student();
		}
		else
		{
			add_student2();
		}
	}
	else
	{
		if (isset($_REQUEST["delete_id"]))
		{
			if (substr($_REQUEST["delete_id"],0,2)=="id")
			{
				delete_id();
			}
			else
			{
				delete_id2();
			}
		}
		else
		{
			if (isset($_REQUEST["edit_id"]))
			{
				if (strlen($_REQUEST["edit_id"])==1)
				{
					edit_id2();
				}
				else
				{
					edit_id();
				}
			}
			else
			{
				if (isset($_REQUEST["search_student"]))
				{
					if ($_REQUEST["search_student"]=="step1")
					{
						search_student();
					}
				}
			}
		}
	}
}

Видно много вложенных условий, определяю нажата ли какая нить кнопка, и выполняется соответствующая функция.
Это то мне и не нравится. Слишком много вложений, а если ещё добовлять функции, то будет ещё больше. С точки зрения "производительности" тоже не очень правильно, т.к. если имеется, например, $_REQUEST["search_student"], то выполняется много проверок.

Как можно сделать по проще и чтобы всё было в одном скрипте?

ЗЫ: Надеюсь всё понятно
ЗЫЫ: Простите за ошибки.
заранее спасибо.

[Johnatan]

Страшно как.. Сразу видно, дельфятник.

Покажи HTML твоей формы с кнопками.

[sleeper_Izh]

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

дельфятник

это что значит?



а код вот:

<center> 
<table border="0" align="center" cellspacing="0" cellpadding="0" >
<tr> 
<td> 
<table border="1" align="center" cellspacing="0" cellpadding="0" bordercolor="bisque" >
  <tr> 
    <th>ID_Student</th><th>Family</th><th>Name</th><th>FatherName</th><th>Sex</th><th>Birthday</th><th></th><th></th> 
  </tr> 
  <tr> 
    <td align="center"  bgcolor="thistle">&nbsp;1&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Крылов&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Денис&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Олегович&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Мужской&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;03-07-1989&nbsp;</td><td align="center" valign="center"> <form action="student.php" method="post"><input name="delete_id" type="HIDDEN" value="id1"><input name="image" type="image" src="b_drop.png"></form></td><td align="center" valign="center"> <form action="student.php" method="post"><input name="edit_id" type="HIDDEN" value="id1"><input name="image" type="image" src="b_edit.png"></form></td>

  </tr> 
  <tr> 
    <td align="center"  bgcolor="thistle">&nbsp;2&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Жильцов&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Олег&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Чеевич&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Мужской&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;01-01-1989&nbsp;</td><td align="center" valign="center"> <form action="student.php" method="post"><input name="delete_id" type="HIDDEN" value="id2"><input name="image" type="image" src="b_drop.png"></form></td><td align="center" valign="center"> <form action="student.php" method="post"><input name="edit_id" type="HIDDEN" value="id2"><input name="image" type="image" src="b_edit.png"></form></td>
  </tr> 
  <tr> 
    <td align="center"  bgcolor="thistle">&nbsp;4&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Михеевна&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Юлия&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Чеевна&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;Женский&nbsp;</td><td align="center"  bgcolor="thistle">&nbsp;01-02-1989&nbsp;</td><td align="center" valign="center"> <form action="student.php" method="post"><input name="delete_id" type="HIDDEN" value="id4"><input name="image" type="image" src="b_drop.png"></form></td><td align="center" valign="center"> <form action="student.php" method="post"><input name="edit_id" type="HIDDEN" value="id4"><input name="image" type="image" src="b_edit.png"></form></td>

  </tr> 
  <tr> 
    <td title="Add new student" align="center" valign="middle"> 
    <form action="student.php" method="post"><input name="image" type="image" src="add.gif"><input type="hidden" name="add_student" value="step1"> </form> 
    </td> 
    <td colspan="7" title="search of student" align="center" valign="middle"> 
    <form method="post"> <input type="text" title="Enter family " name="f" value=""><input name="image" type="image" src="search.gif"><input type="hidden" name="search_student" value="step1"></form> 
    </td> 
    </td> 
  </tr> 
</table> 
</td> 
<td> 
<FORM method="post"> 

<SELECT NAME="page"> 
  <OPTION  selected  value="0">1</OPTION><BR> 
  <OPTION  value="1">2</OPTION><BR> 
</SELECT><BR> 
<INPUT TYPE="submit" NAME="go" VALUE="go" > 
<input name="choose_page" type="HIDDEN" value="step1"> 
</form> 
</td> 
</tr> 
</table> 

[sleeper_Izh]

там как бы кнопки в виде рисунков - они пока не аккуратные, ещё не занимался ими

[Johnatan]

http://www.php.net/manual/ru/control...res.switch.php
Должно тебе помочь.
Но прежде чем начинать программировать, я бы тебе посоветовал почитать книжку какую-нибудь по теме.. А то ты так никуда не уйдёшь. А рассказывать азы тебе никто не будет.

Я не понимаю зачем ты вообще формы делаешь и кнопки в них?
Чем тебе не подходит конструкция:

<a href="student.php?delete_id=id3"><img src="b_edit.png" alt="" border="0" /></a>

???

[sleeper_Izh]

дык что бы всё передовалось методом post.

А книжки я читал: просто синтоксис предпочитаю использовать привычный, а суть всё равно не изменится.

[Johnatan]

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

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

[sleeper_Izh]

Да я вообще то не про синтаксис изначально спрашивал.

ну будет у меня написано так:

switch $_REQUEST['parametr'] {
case "edit_id":
    //...
    break;

.....


case "delete_id":
     //...
     break;

}

всё равно пока проверка дойдёт до последнего пунка произойдёт много "лишних" проверок, вот я и спросил можно ли как нить избавиться от нихю?
как понял, нельзя.
Спасибо.

[Johnatan]

Можно, но ты не сделаешь так, потому что прийдётся вообще всю архитектуру твоего проекта переделывать.

[SkyM@n]

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

можно ли как нить избавиться от них?

Можно, вынеся все названия страниц в БД и сравнивая одной строкой и читая из той же БД.