1. JavaScript / Говнокод #14409

    +161

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    var summa = 0;
for (i = 0; i < data.length; i++) {
    var j = 0;
    for (var key in data[i]) {
        j++;
        if (j == 3) summa += parseFloat(data[i][key]);
    }
}

    вытаскиваем сумму из массива вида: [{MODEL:"ID-2112",COUNT:"1",PRICE:"14070"}, {...}]
    не! ну трети же, чо..

    Запостил: belukov, 24 Января 2014

    Комментарии (11) RSS

    • ava wvxvw 24.01.2014 14:28 # +2

      Стоит отдельно отметить замечательный формат данных - в котором все числа передаются как строки. Кстати типичная беда ПХП - в нем почему-то нужно явно приводить числа к числам, чтобы сериализация правильно отработала.
      А еще есть 100500 туториалов по использованию Гуглмапс, в которых ж.скриптерам рекомендуется передавать числа как строки, скорее всего благодаря опечатке авторов, но тем не менее растиражированой миллионами.
      Ответить

      • ava anonimb84a2f6fd141 24.01.2014 16:14 # +1

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

      • ava WGH 25.01.2014 16:57 # +3

        Я думаю, это не опечатка. В JavaScript единственный числовый тип - double. Так что при передаче больших чисел могут начаться проблемы.
        > JSON.parse('{"a": 44444444444444444}')
        { a: 44444444444444450 }
        Ответить

        • ava 1024-- 25.01.2014 17:10 # 0

          Гуглмапс знает Землю до 15 знака?
          По-моему, для 99.9% сайтов такая точность не требуется. А если и требуется, то это чаще всего уже не число по смыслу (например, ID видео на YouTube)
          Ответить

        • ava wvxvw 25.01.2014 17:14 # −1

          Ну, хорошо, вот мы передали "точное" большое число (которое на самом деле строка), и что с ним дальше делать? Тогда уже нужен свой аналог Bignum писать и все такое. Но это явно не тот случай. Да и числа в примере как-то не особенно большие.
          Ответить

          • ava WGH 25.01.2014 17:18 # 0

            А обычно это не "число", а какой-то суррогатный ID. С ним не надо ничего делать, только передать дальше (например, сформировать ссылку на какой-нибудь твит, или вызвать стороннний API).
            Ответить

            • ava wvxvw 25.01.2014 17:22 # 0

              > price
              > count
              > суррогатный айди
              I don't think so!
              Ответить

              • ava WGH 25.01.2014 18:07 # +1

                Ну а что? :) Суррогатная цена в у.е.
                Ответить

    • ava torbasow 25.01.2014 15:50 # 0

      О. А почему не просто data.reduce(function(previousValue, currentValue) previousValue+parseFloat(currentValue["price"]), 0)?
      Ответить

      • ava wvxvw 25.01.2014 17:17 # −1 

        _.sum(_.pluck(data, 'price'));

        Если уж гнаться за погнными метрами кода.
        Ответить

    • ava guest 25.01.2014 16:56 # 0

      как вариант: Есть хранимая процедура, которая возвращает результсет с 3 колонками. Всё это помещается в массив линкед мапов и передается в браузер. Имена колонок вполне могут поменяться, но колонка с прайсом будет всегда третья
      Ответить

    Добавить комментарий