Насколько плох такой проект?

Question

[di]

Хотел написать простенький чат.
Что впринципе сумел реализовать -
клиент HTML+CSS+JS(чистый javascript, очень не хотелось тянуть грузную и тяжелую Jquery),
websocket сервер написал на PHP (не сторонник серверного js, другие серверные языки знаю плохо),
микросервис для регистрации и авторизации по JWT(дописывал второпях без фреймворков,ибо не нашел ни одной простенькой либы, так что подозреваю что там дыры в безопасности).
Сообщения хранятся в БД(Mysql), зарегистрированные пользователи там же.
Ставил целью написать именно чат с наиболее прозрачной реализацией.
Получилось впринципе вот так - https://github.com/Delgus/chat

Хотел использовать этот проект чтобы показывать в будущем работодателям, но с такой реализацией клиента даже стыдно демку разворачивать. Думал улучшить клиентскую часть,найти что-нибудь готовое(на Vue например),но поиски по ключевым словам "frontend chat vue"выводят уже готовые приложения где и backend на js. Придется залезать в код и выкидывать половину(
Стоит ли улучшать проект или же реализовать еще что-нибудь(работодатели же не будут ничего смотреть скорее всего, посмотрят на количество проектов и мои ноль звезд)?

P. S. Я понимаю что это больше похоже на саморекламу, но если взгляните хоть одним глазком на проект и посоветуете куда расти буду очень благодарен

Comments

[Радость моя]

на тебе звезду, и мечтай о чем нибудь великом :)

[Алексей Ярков]

function post(url, method, data) {
    var token = '';
    var xhr = createRequest();
    xhr.open(method, url, false);
    xhr.onreadystatechange = function () {
        if (xhr.readyState === 4) {
            token = xhr.responseText;
        }
    };
    xhr.send(data);
    return token;
}

var sign_up_form = document.getElementById("signup-form");
var login_form = document.getElementById("login-form");
sign_up_form.onsubmit = function (e) {
    e.preventDefault();
    var data = new FormData(sign_up_form);
    var answer = post('/auth-server/signup.php', "POST", data); // Работает? О_о
    if (answer === 'true') {
        alert('Вы успешно зарегистрировались!!!');
    }
}
login_form.onsubmit = function (e) {
    e.preventDefault();
    var data = new FormData(login_form);
    var token = post('/auth-server/login.php', "POST", data); // Работает? О_о
    if (token !== 'false') {
        document.cookie = 'jwt=' + token + '; path=/';
        window.location = "/";
    } else {
        alert("Неверные логин или пароль!!!");
    }
}

[hOtRush]

Бывает и намного хуже, для junior'а норм

[Олег]

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

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

[di]

Алексей Ярков, Да. Необходимо веб-сервер настроить чтобы скрипты php из папки auth-server работали ,в Apache2 по умолчанию вроде работает, nginx надо настраивать. что совершено не объяснено нигде. ну пожалуй регистрация и авторизация - самые слабые места в проекте) Спасибо за комментарий

[di]

Олег Муравейко, верно заметили. Спасибо)

[Олег]

В общем. Был такой не большой тест. Я замордованной сейчас отладкой . Отвлекся на 3 минуты, чтобы глянуть код.
Мне пришлось открыть 5-7 файлов , чтобы понять чего Вы там пытались сделать.
В результате я не увидел работы с потоками.

Все ! Вы не заинтересовали.
вебсокеты применяют, когда нужно организовать моментальный обмен сообщениями.

Ваш проект даже не 10й. Есть как минимум 4 на гите со звездами 400 - 2000 .
Если учебный проект. В реадми стоит это указать и дать, что ли обзор существующих решений.
Коротенько описав, что же Вы сделали упрощенно из них в учебных целях.

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

[Алексей Ярков]

Delgus,

nginx надо настраивать. что совершено не объяснено нигде

Так уж и нигде? )))

[di]

SmInc,

А у вас не было мыслей, что на работу надо не только устроиться, но и работать там?
А если вы не можете ни с нуля написать, ни имеющееся доделать\переделать, то зачем вы на работе?

На работе платят.

Ну к сведению я работаю чуть менее года в очень маленькой команде (backend на Yii2), но хочется расти далее (например сложно освоить Docker если на работе его не используют, сложно изучить JS и современные фреймворки, пока на работе JQuery и bootstrap и нет хороших фронтендеров), поэтому потихоньку собираю портфолио с мечтой поработать когда-нибудь в большой команде. И вот возник вопрос - потратить свободное время доводя этот тренировочный проект с чатом до ума, или попробовать еще что-то запилить.

Спасибо за комментарий)

[di]

Алексей Ярков, я имел ввиду что в ридми не написал я об этом,что является моим косяком)) так то nginx + php-fpm я локально я использую))

Answer 1

[Max Kostinevich]

Не поленитесь, и разверните онлайн-демо и добавьте ссылку в описание проекта, и добавьте скриншоты/gif в readme.md - все это вам значительно поможет, если попросят показать рабочий проект.

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

С технической точки зрения - есть куда расти, install.php - это полный трэш, но это все придет с опытом. Главное - у вас есть законченный и рабочий проект.

Answer 2

[Иван Шумов]

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

В любом случае - молодец, но есть над чем работать: от оформления кода до развертывания и мониторинга (сервер-то может упасть)