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

    0

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    function sortWithIndeces(toSort: any) {
  for (let i = 0; i < toSort.length; i++) {
    toSort[i] = [toSort[i], i];
  }
  toSort.sort(function(left: any[], right: any[]) {
    return left[0] < right[0] ? -1 : 1;
  });
  toSort.sortIndices = [];
  for (let j = 0; j < toSort.length; j++) {
    toSort.sortIndices.push(toSort[j][1]);
    toSort[j] = toSort[j][0];
  }

  return toSort;
}

sortWithIndeces(arr);

arr.sortIndices.forEach((item: any, index: number) => {
  result[index] = data[item];
});

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

    Запостил: igpo, 09 Августа 2019

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

    • ava guest8 09.08.2019 14:09 # −999

      показать все, что скрытоvanished
      Ответить

      • ava bormand 09.08.2019 14:11 # 0

        Тайпскрипт?
        Ответить

      • ava Orange_Ikarus 09.08.2019 14:43 # 0

        Можно было в «ES4». Его даже реализовали в какой-то древней версии Mozilla Suite, но потом убрали.
        Ответить

        • ava bormand 09.08.2019 14:47 # 0

          Флеш видимо оттуда и получил свою типизацию?
          Ответить

          • ava Orange_Ikarus 09.08.2019 15:18 # 0

            Синтаксис одинаковый, вот только AS2 появился в 2003 году (как выглядел AS1, я сходу не нагуглю), а черновик ES4 опубликовали в 2007.
            Ответить

    • ava igpo 09.08.2019 14:23 # 0

      И кстати, использовать any - очень плохая практика.
      Ответить

      • ava bormand 09.08.2019 14:34 # +2

        Ну здесь же общий алгоритм, который работает для любых типов с оператором "меньше"... Или в тайпскрипте для них есть какой-нибудь Comparable?
        Ответить

        • ava igpo 09.08.2019 14:40 # 0

          если просто, то ts - это типизация для js
          ts не добавляет никаких новых функций
          `left[0] < right[0]` работает потому что ts считает что там тип any, то есть эти значения могут быть числами
          сложно рефакторить код, где используются any
          Ответить

        • ava igpo 09.08.2019 14:41 # 0

          и left[0][0][0][0][0][0][0][0][0][0][0][0][0] тоже отработает, но упадёт в runtime
          Ответить

          • ava petux 10.08.2019 13:04 # 0

            [0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0][0]
            Кто проживает на дне океана?
            Ответить

      • ava guest8 09.08.2019 14:50 # −999

        показать все, что скрытоvanished
        Ответить

        • ava igpo 09.08.2019 14:59 # 0

          у меня к автору кода тоже есть такой вопрос
          Ответить

    • ava BEKTOPHblu_nETyX 09.08.2019 20:01 # +1 

      toSortWithIndices =: /:~ ; /:
   toSortWithIndices 34 2 12 54 85 41 3 54 12
┌────────────────────────┬─────────────────┐
│2 3 12 12 34 41 54 54 85│1 6 2 8 0 5 3 7 4│
└────────────────────────┴─────────────────┘
      Ответить

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