За последние 24 часа нас посетили 18184 программиста и 878 роботов. Сейчас ищут 473 программиста ...

Мордой в код

Тема в разделе "Беседы", создана пользователем SeAnSolovev, 5 фев 2024.

  1. SeAnSolovev

    SeAnSolovev Новичок

    С нами с:
    5 дек 2023
    Сообщения:
    2
    Симпатии:
    0
    Приветствую! В далёком 2018 взял под крыло заброшенный проект. Долгое время практически ничего не менялось, после началось обучение php на основе данного кода и вот подступился процесс миграции на новые версии. Сейчас перескакиваем с версии 5.5 на 7.4
    Проблем особо нет, но хотелось бы услышать ваше мнение об качестве кода и что стоит поменять. По причине единственного рабочего полигона в виде данного проекта, сам оценить не могу. Несколько людей уже высказались, что «Писал либо школьник, либо шизофреник» без явного объяснения почему. Буду признателен, если ткнёте носом. Спасибо!

    GitHub: https://github.com/EngineGPDev/EngineGP/tree/EGPv3-main
     
  2. Survivor

    Survivor Новичок

    С нами с:
    8 фев 2023
    Сообщения:
    54
    Симпатии:
    6
    Нет предела совершенству.
    На мой взгляд, главное чтобы работало без ошибок и не грузило систему.
    Второе - оформление кода, его читаемость.

    Судить не готов, зеленый ещё, куда мне!)
     
    SeAnSolovev нравится это.
  3. Дюран

    Дюран Активный пользователь

    С нами с:
    9 мар 2018
    Сообщения:
    244
    Симпатии:
    17
    Вы vendor забыли из гита убрать :)
    Процедурное нечто, на файлах. Без структуры, типа mvc. Код - лапша, все намешано, и БД и html.
    Смысл, о чем тот проект, ускользает, сайт какой то?
    Имхо, так делали лет 10 назад, если не больше
     
  4. SeAnSolovev

    SeAnSolovev Новичок

    С нами с:
    5 дек 2023
    Сообщения:
    2
    Симпатии:
    0
    Какого-то определённого сайта нет, есть своё небольшое коммьюнити. Думаю мигрировать на symfony или lavraver

    Сейчас исправление, дополнение, удаление функционала, занимает критический объём времени. Примерно за полгода - год смогу что-то похожее с упрощённым функционалом релизнуть
     
  5. don.bidon

    don.bidon Активный пользователь

    С нами с:
    28 мар 2021
    Сообщения:
    829
    Симпатии:
    129
    Больше-больше, тот же MVC в PHP начали внедрять гораздо раньше )
     
    SeAnSolovev нравится это.