Как писать более правильно?

Question

[TrustyWork]

Один из этих примеров я показывал своему ученику, преднамеренно не буду говорить какой.
В ответ услышал "а мне 4 сениора говорили что я все делаю правильно".
Прошу тостер о независимом мнении, дабы спасти неофита от пути говнокодера.

var hp = 100;
function damage( point){
     hp = hp - point;
}

damage( 100);
console.log( hp);

Или

var ship = {
    hp: { cur: 100, max: 100},
    damage: function( point){
        this.hp.cur = this.hp.cur - point;
        return this.hp;
    }
}

var hp = ship.damage( 10);
console.log( hp);

Answer 1

[bromzh]

Лучше сразу приучать к новому синтаксису es6 (классы, let/const):

export class Human {
    constructor({ hp }) {
        this.hp = {
            current: hp,
            max: hp,
        };
    }

    damage(point) {
        this.hp.current -= point;
    }
}
let human = new Human({ hp: 100 });
human.damage(10);
console.log(human.hp.current);

Comments

[TrustyWork]

Переучивать всегда тяжело, лучше делать это постепенно. Но, вы все верно написали.

[bromzh]

Александр: объект в JS на первом месте, про них надо в практически сразу рассказывать. Ну и это "ООП похожее на java" намного легче для понимания, чем возня с прототипами. Про прототипы надо знать, но пользоваться нагляднее новым синтаксисом. Чтобы попугать детей, можно посмотреть, во что превращается код c 2-мя пустыми классами, один из которых наследуется, после трансформации.

[bromzh]

TrustyWork: очень наглядно взять код с новым синтаксисом, и посмотреть на вывод babel или typescript-компилятора.

[TrustyWork]

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

[bromzh]

TrustyWork: ну новый синтаксис есть и для базовых вещей: let/const, функции с параметрами по-умолчанию, деструктуризация (при присваивании, в параметрах функции), строки-шаблоны (очень хопошо он описан тут.
Например, объяснить, что раньше было только var, а теперь есть let и const. Показать на примере, чем они отличаются и сказать, чтобы var в новом коде не использовалось никогда. Чтобы было легче, можно настроить eslint с нужными правилами. Это очень хорошо приучит писать нормальный код.

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

Ученику же можно дать задание переписывать куски старого кода в новом стиле. Так он и проект изучит, и поймёт на живом примере, что скрывается за новым синтаксисом , и избавит проект от некоторого легаси.

[bromzh]

Александр: У вас довольно устаревшие взгляды на JS индустрию. Вы ведь даже не знали про source map до недавнего времени)
Какое-то время до JS никому особо не было дела. Между 3-й и 5-й версиями прошло 10 лет, между 5-й и 6-й - 4 года. Там было много проблем. Очень показательный пример: существование сотни библиотек для упрощения ООП в JS. Проблема была и её надо было решать.
Сейчас всё резко изменилось, JS очень бурно развивается. Крупные игроки вроде гугла и майкрософта взялись за ум и всё зашевелилось. Браузеры развиваются теперь намного активнее. Появились инструменты, которые позволяют без проблем использовать новое в старых браузерах. Самый сок всего этого в том, что это стандарт. Сейчас я использую транспайлер и полифиллы. Потом мне достаточно будет их убрать. Мне не придётся переписывать код, убирать устаревшие библиотеки, которые по-своему реализовывали какие-то части стандарта и прочее. Поддержка только упрощается. При этом, можно даже не трогать те части программы, которые не изменяются. А тот код, который меняется можно медленно переводить на новый стандарт.

> Много сейчас вакансий где надо знать новый стандарт?
Полно, достаточно поискать "es6" на hh.
> Вот через год-два можно кидаться в новый ES, т.к. он будет хорошо выдержан разработчиками и временем.
А всё это время что делать? Использовать свои нестандартные костыли и веловипеды? Чем раньше начать использовать новый стандарт в проект, тем лучше.
> ваши прототипы - все сложная загогулина больше знать это не надо
Но я же не говорил такого. Нужно показать как было раньше, и как есть сейчас.
> Есть десятилетиями проверенный стандарт
Стандарт явно был фиговый, сейчас стараются это исправить.
> в реальности продукт должен работать на веб-браузерах с IE8-9 и выше
Статистика использования IE9 - около 1%. В реальности я бы просто не стал бы работать в компании, где есть требования поддерживать IE8-9.

Просто я не вижу смысла учить человека старым технологиям, вот и всё. Он просто будет неконкурентноспособен на рынке.

[TrustyWork]

bromzh: я бы не был таким критичным. JS была в застое много лет, за это время вышло много библиотек и мало специалистов. Сейчас эта ситуация меняется, но старые библиотеки не будут переписаны настолько быстро, а бизнес требует проверенных и эффективных решений в первую очередь. ES2016 эффективен с точки зрения программиста, для менеджера преимущества промайсов или генераторов мутны и не очевидны, особенно не очевидны преимущества переписывания старого кода, работающего и на старых браузерах тоже, на новый, работающий только на новых.

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

Answer 2

[Ленар Фаттахов]

В таком маленьком, изолированном от реальности примере невозможно определить. По мне так оба способа неправильные, объект ship лучше создавать через оператор new.

Comments

[TrustyWork]

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