Какая ошибка в коде?

Question

[Данил Ситдиков]

Class Carriage

    passengers = []

    constructor: (passengersData) -> 
        for passenger in passengersData 
            passengers.push 
                name: passenger.name 
                age: passenger.age

    getPassengers: () -> passengers

    getAverageAge: () -> 
        ageSum = 0 
        ageSum += passenger.age for passenger in passengers 
        ageSum / passengers.length

писал "одну приложуху" , и в слаке скинул "одному фронтендеру" который сказал что код написанный данным способом мягко говоря с ошибочно. Этот класс используется только для создания сущностей вагонов некоего поезда и подсчёта среднего возраста пассажиров в вагоне. Что в этом коде такого? (синтактических ошибок нет)

Comments

[Илья Герасимов]

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

[numfin]

этот код больше на питон похож, чем на js

[Даша Циклаури]

1) почему нельзя было сразу passengers = passengersData?
2) почему нельзя в конструкторе сразу считать средний возраст? ведь нигде массив пассажиров не изменяет

[Ivanq]

numfin, CoffeeScript.

[Kovalsky]

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

[Vitaliy]

ошибка в том, что это CoffeeScript)))

[Данил Ситдиков]

В слаке ментор, злой ментор который каждый раз пинает, когда я не так пишу код, вот сижу думаю что не так. До функций типа таких докопался. Пример абстрактный так что не спрашиваете почему такая простая. Сказал что при минификации вебпаком будут проблемы.

something = function (inputA, inputB, inputC) { 
    this.paramA = inputA + inputB; 
    this.paramB = inputB + inputC; 
    this.paramC = inputC - inputA;

    this.getSomething = function () { 
        return this.paramA + this.paramB * this.paramC; 
    } 
}

Answer 1

[Павел]

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

constructor : function(passengersData) {
    for (passenger in passengersData) {
        passengers.push({
            name: passenger.name,
            age: passenger.age
        });
    }
}

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

constructor: (passengersData) -> 
    for passenger in passengersData 
        passengers.push 
            name: passenger.name 
            age: passenger.age

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

Answer 2

[Bobsans]

Видимо все дело в "Кофейном скрипте")

Answer 3

[bioroot]

Обычно говорят что всё плохо и не говорят где именно редкие... "редиски". Я в таких случаях крайне рекомендую взять этого товарища за грудки, потрясти и спросить что конкретно не так. Таких "менторов", к сожалению, кругом пруд пруди. Пока вы не получите конкретный ответ, ваше общение не принесёт никакой пользы. А конкретные указания бывают очень разные в том числе и настолько нелепые, что потом будете удивляться. Но это будет хотя бы предметом обсуждения и вы можете либо сказать "Ой, ступил", либо объяснить почему вы выбрали именно такую реализацию.

А иначе получается как в анекдоте. Вы все здесь ..., а я Д'Артаньян.