Подробности

Как мы работали со студией Мануфактура Софт из Воронежа


Данный проект создан для того, чтобы показать всем будущим клиентам которые прямо сейчас выбирают студию которой хотят доверить разработку своего сайта, как именно велась работа по разработке сайта vrachi.ru воронежским интерактивным агенством Мануфактура Софт под руководством Дмитрия Провоторова куда мы обратились.

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

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

Всё описанное на сайте это конкретный пример работы заказчика со студией Мануфактура Софт в течении 2013-2015 годов.

Подробно эта работа описана в блоге на этом сайте.

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


Основные наши замечания


1. Мануфактура Софт не уложилась в сроки.
Заказы от августа 2013 года, срок выполнения по ним работ - два месяца, акты к эти заказам октябрь 2014, подписаны вообще в июле 2015
Ccылка на заказы и акты.

2. Сайт написан с очень большим количеством ошибок в программном коде.
Аудит баз данных Mysql
Аудит Nginx+App

3. После того как мы его полностью оплатили, нам потребовалось несколько месяцев, чтобы уговорить исполнительного директора этой замечательной компании Дмитрия Провоторова передать нам код нашего сайта для его самостоятельного размещения на своём сервере.


Все подробности и доказательства вы можете узнать в блоге и по ссылкам ниже



Акты и заказы которые подтверждают сдачу нашего сайта студией Мануфактура Софт с опозданием на год.

Переписка с этой прекрасной студией Мануфактура.

Видео замечания к сайту.

Текстовые замечания к сайту.

Мои комментарии на vc.ru к статье Дмитрия Провоторова после которых он подал ко мне иск в Арбитражный суд.

Аудит нашего сайта от компании ТРИНЕТ.

Аудит баз данных Mysql

Аудит Nginx+App

Аудио записи заседаний Арбитражного суда.

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


Вкратце о коде написанным программистами студии Мануфактура


Аудит базы данных Mysql


  • В целом структура плохо нормализована, типы данных выбраны неоптимально, пропущены ограничения по внешним ключам (целостность данных).
  • Индексы вообще нигде не используются (кроме стандартного PRIMARY, который создается вместе с таблицей).
  • Нестандартный default_collation=utf8mb4_0900_ai_ci (зачем?)
  • table added_education:
  • id_doctor - отсутствует FK связь (Constraint)
  • startyear - INT - неоптимальный тип данных (должен быть TINYINT)
  • stopyear - INT - неоптимальный тип данных (TINYINT)
  • active - INT - неоптимальный тип данных (TINYINT)
  • needaccept - неоптимальный тип данных (TINYINT)
  • table table admin:
  • id_user - INT (11) -отсутствует FK связь (Constraint)
  • table area:
  • id_region - отсутствует FK связь с таблицей regions (Constraint)
  • table articles:
  • id_doctor - отсутствует FK связь с таблицей doctors (Constraint)
  • datecreated - INT - правильно было бы DATETIME / TIMESTAMP
  • draft - INT - неоптимальный тип данных (TINYINT)
  • del - неоптимальный тип данных (TINYINT)
  • activated - неоптимальный тип данных (TINYINT)ъ
  • Продолжение следует...


А это менее детальный анализ кода от Мануфактура Софт


  • Закомменченый код
  • Нет 12 factor app
  • Документация неконсистентная
  • Слишком большие методы
  • Неконсистентный стиль кода
  • Нет интернационализации, тексты местами захардкожены в коде
  • protected $_id_account = '2dc8c497-b6fb-d064-d158-aa9572da54e1';//'b0724f00-18cd-3254-4138-bba46ab06f53'; - какие-то креды захардкожены
  • Местами хардкодится то, что нужно выносить в константы
  • Файлы по 2000 строк
  • Кое-где есть SQL-inj уязвимости
  • Явно не использовались линтеры для форматирования кода
  • ООП используется лишь формально
  • Комментарии бесполезны
  • Код часто дублируется
  • Мусора дофига — закомментированный код, комменты типа "//исключаем повторы, может затупил", пропущенные модификаторы доступа в классах.
  • Методы длинные, вложенность выше нормы.
  • HTML/CSS намешан в PHP сильно.
  • Чрезмерное увлечение наследованием.
  • Строки не вынесены в константы.
  • Запутанное построение страниц
  • Много разных блоков и функций участвовало в построении страницы и они были раскиданы по разным местам.
  • Большая связанность кода, одна функция генерала разный контент в зависимости от входных параметров.
  • Продолжение следует...

Подробно всю историю можете прочитать в блоге.

В этом споре нас поддержали многие ребята из открытого клуба разработчиков и профессионалов IT-индустрии Воронежа, несмотря на то, что адвокат выступающий в Арбитражном суде со стороны Мануфактура Софт входит в комитет этого прекрасного сообщества.