VPF::Правильная разработка - Форум программистов

MrDmitry

Дата 19.12.2013, 02:08 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 556
Регистрация: 10.11.2006

Репутация: нет
Всего: нет

Привет всем. Стоит довольно простая задача, есть база данных, нужно добавлять, редактировать, удалять из нее записи при помощи попапов. В качестве верстки и всех прочих объяснений получил вот такой вот файлик http://rghost.ru/51060171 smile

(В общем верстка не важна главное функционал)

В общем начал работу над проектом и понял что я совершенно не квалифицирован в вопросе программирования ((

Код получился не читабельным (В нем путаюсь даже я) и не функциональным. Но в принципе все что я сделал на этот момент работает. Собственно вот то что я сделал http://rghost.ru/51060141(БД + Сами скрипты)

Так вот я прошу помощи, настав те на путь истинный(кому не лень читать все что я там написал) как все это дело сделать более функциональным, сделать более понятным и читабельным.

Заранее спасибо! )

gta4kv

Дата 19.12.2013, 05:14 (ссылка)

(нет голосов)

Загрузка ...

Walking around mad.

Профиль
Группа: Завсегдатай
Сообщений: 1868
Регистрация: 25.7.2006

Репутация: 21
Всего: 91

А что вам не нравится? Работает и бог с ним..

Это сообщение отредактировал(а) gta4kv - 19.12.2013, 05:15

_zorn_

Дата 19.12.2013, 07:24 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1076
Регистрация: 21.8.2007

Репутация: 6
Всего: 12

О мой бог

Код


if ($_POST['action']=='1')
 {
if ($_POST['v1']=='')
 {
  $result =$_POST['v6'].';'.$_POST['v7'].';'.$_POST['v8'].';'.$_POST['v9']; 
 }
else 
 {
  $result = $_POST['v1'].';'.$_POST['v2'].';'.$_POST['v3'].';'.$_POST['v4'].';'.$_POST['v5'];
 }
 $result1 = $_POST['v10'].';'.$_POST['v11'].';'.$_POST['v12'];
.....
}elseif ($_POST['action']=='2'){

}if ($_POST['action']=='3')
 {

 if ($_POST['nmt']=='client_info')
  {
  if ($_POST['vals1']=='')
   {
    $result =$_POST['vals6'].';'.$_POST['vals7'].';'.$_POST['vals8'].';'.$_POST['vals9']; 
   }
  else 
  {
   $result = $_POST['vals1'].';'.$_POST['vals2'].';'.$_POST['vals3'].';'.$_POST['vals4'].';'.$_POST['vals5'];
  }
}
....

1. Форматирование. Отступы гуляют как хотят.
2. ИМЕНА ПЕРЕМЕННЫХ. Особенно доставляют цифровые action. Символьные названия действий нынче не в моде ?
3. Про switch/case ты не слышал ?

Вобщем такое впечатление, что код через обфускатор пропустили smile

Это сообщение отредактировал(а) _zorn_ - 19.12.2013, 07:27

MrDmitry

Дата 19.12.2013, 10:54 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 556
Регистрация: 10.11.2006

Репутация: нет
Всего: нет

Ну собственно о чем я и говорил. Код получился совершенно не читабельным и не удобным (
ок.
1. Форматирование
2. Информативное название переменных
3. Можно пример в каком месте вы имете ввиду использовать switch/case? за место моих action? и чем так плоха переменная action? На мой взгляд это самая информативная из переменных?

А что насчет функциональности?

PS код я буду переписывать в соответствии с вашими поправками )

Это сообщение отредактировал(а) MrDmitry - 19.12.2013, 10:58

Fortop

Дата 19.12.2013, 15:39 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2200
Регистрация: 13.11.2007
Где: Донецк

Репутация: 20
Всего: 42

Код не осилил...

Делаем декомпозицию задачи.
Что нужно?
Авторизация пользователя?
Добавление новых записей?
Просмотр?
Редактирование существующих?
Удаление?
Выясняем что нам нужно из входных данных для каждого из подпункта.
Что мы возвращаем в качестве результатов.
В соответствии с п.1 и п.2 реализуем соответствующие функции.

На первое время достаточно будет.

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

--------------------

Мир это Я.
Живее всех живых.

_zorn_

Дата 27.12.2013, 10:37 (ссылка)

(голосов:1)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1076
Регистрация: 21.8.2007

Репутация: 6
Всего: 12

Цитата(MrDmitry @ 19.12.2013, 18:54

)

Можно пример в каком месте вы имете ввиду использовать switch/case?

Например

Код

if ($_POST['action']=='1')
....
elseif ($_POST['action']=='2')
....
if ($_POST['action']=='3')

так думаю симпатишней будет

Код


switch($_POST['action'])
{
  case '1':
  ...
  break;
  case '2':
  ...
  break;
  case '3':
  ...
  break;
}

Это сообщение отредактировал(а) _zorn_ - 27.12.2013, 10:38

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »