1. Python / Говнокод #12191

    −100

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    size = 3
lines = ["___", "bcd", "___"]
board = [[None]*size]*size
for x in range(0, size):
  for y in range(0, size):
    if lines[x][y] != '_':
      board[x][y] = lines[x][y]
print(board)

    я бы, очевидно, хотел видеть результат:
    [[None, None, None], ['b', 'c', 'd'], [None, None, None]]
    но питоша выводит:
    [['b', 'c', 'd'], ['b', 'c', 'd'], ['b', 'c', 'd']]

    Запостил: serpinski, 24 Ноября 2012

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

    • ava Fai 24.11.2012 21:08 # 0

      http://ideone.com/Ugospe

      Все ок.
      Ответить

    • ava bormand 24.11.2012 21:28 # +1

      http://ideone.com/AIAjsf

      Питонское * использует shallow-copy, а не deep-copy. Отсюда и баг.
      Ответить

      • ava inkanus-gray 24.11.2012 22:34 # −1

        Какое же утомительное занятие программировать на Питоне: копипастить и копипастить...

        http://ideone.com/ttSf2c
        Ответить

        • ava Fai 24.11.2012 22:50 # +1

          > утомительное занятие программировать на Питоне

          Вы просто не умеете функционально питонировать: http://ideone.com/kDHksR
          Ответить

          • ava guest 24.11.2012 22:53 # 0

            выглядит как отвратительный лисп
            Ответить

            • ava Fai 24.11.2012 22:55 # 0

              Каким боком это на лисп похоже?
              Ответить

              • ava guest 24.11.2012 22:56 # 0

                левым
                Ответить

              • ava wvxvw 25.11.2012 02:47 # +2 

                (make-array 
          '(3 3)
          :initial-contents
          (mapcar (let ((x 1))
                    (alexandria:compose
                     #'(lambda (y)
                         (incf x)
                         (if (oddp x) (replace y '(b c d)) y))
                     (alexandria:curry #'make-list 3 :initial-element)))
                  (make-list 3 :initial-element nil)))
#2A((NIL NIL NIL) (B C D) (NIL NIL NIL))

                ЗЫ. Долго искал повод, где бы можно было использовать эти функции, вот, нашлось.
                Ответить

                • ava Fai 25.11.2012 07:16 # +1

                  Лисп многословнее питона?
                  Ответить

                  • ava wvxvw 25.11.2012 14:48 # +3

                    Нет, это извращенный способ написать то, что автор задумал. Но, с другой стороны, есть традиция называть функции и аргументы без сокращений. Т.е. нету всяких len, str, dic. Но если кому-то захочется, - всегда можно добавить макрос, который сократит названия (мне не хочется).
                    Ну и в конце концов, Питон можно просто реализовать на Лиспе (кстати, есть даже такой проект), со всем его синтаксисом, так что нет однозначного ответа...
                    Ответить

                    • ava LispGovno 25.11.2012 15:09 # 0

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

                      • ava Fai 25.11.2012 18:19 # +1

                        > ненавижу сокращения в стандартной библиотеке

                        Трупстрауса и Александреску с тобой не согласятся. Да и Томпсон с Ритчи тоже.

                        std, cout, cin, rbegin, rend, int, char, bool, rand, endl.
                        Ответить

                        • ava LispGovno 25.11.2012 19:06 # 0

                          а александреску нигде сокращений не делал. откуда инфа? вот труп в своё время всплыл, ага
                          Ответить

                    • ava bormand 25.11.2012 15:33 # +2

                      > Питон можно просто реализовать на Лиспе
                      А можно и лисп на питоне. Тьюринг полнота такая тьюринг полнота.
                      Ответить

                      • ava wvxvw 25.11.2012 16:10 # +2

                        Нет, я имел в виду реализовать грамматические правила языка, а не механизм. Грамматика - это к Хомскому, а не к Тюрингу.
                        Ответить

                  • ava wvxvw 25.11.2012 15:14 # +1 

                    (loop with bcd = (make-array '(3 3))
   with data = #(b c d)
   for i from 0 to 8
   do (multiple-value-bind (j k)
          (floor i 3)
        (setf (aref bcd j k)
              (when (oddp j) (aref data k))))
   finally (return bcd))

                    Ну, вот так нормально будет, пожалуй, если не задумываться особенно.
                    Ответить

            • ava wvxvw 25.11.2012 02:27 # +1

              В Лиспе по крайней мере есть макросы, чтобы такую ерунду не писать самому.
              Ответить

    • ava anonimb84a2f6fd141 25.11.2012 00:10 # −2

      не говно
      Ответить

    • ava myaut 26.11.2012 00:47 # 0 

      lines = ["___", "bcd", "___"]
procBoard = lambda c: None if c == '_' else c
board = [map(procBoard, list(l)) for l in lines]
print board


      http://ideone.com/fB8sGY
      Ответить

      • ava LispGovno 26.11.2012 00:53 # 0

        Мяут, неужели это самый короткий вариант на питоне?
        Где твоя Команда R?
        Ответить

        • ava Fai 26.11.2012 08:34 # 0

          > неужели это самый короткий вариант на питоне?
          > Вы просто не умеете функционально питонировать: http://ideone.com/kDHksR

          Закоментированным записан вариант @myaut в одну строку.

          До этого у меня еще и масштабируемо было, но пришлось кастрировать, чтобы короче вышло.
          Ответить

          • ava Fai 26.11.2012 08:42 # 0

            А нет. Если подстановкой переписать, то его вариант короче (правда пришлось переписать на python3):
            board = [list(map(lambda c: None if c == '_' else c, l)) for l in lines]
print (board)
            Ответить

            • ava Fai 26.11.2012 08:58 # +2

              А вот и самый простой вариант на python:

              board = [[None if c == '_' else c for c in l] for l in lines]
print (board)
              Ответить

    • ava wvxvw 26.11.2012 11:40 # 0

      А такой вариант уже был?
      map(lambda x: map((lambda y: None) if x[0] == '_' else (lambda y: y), x), ['___', 'bcd', '___'])

      Кстати, интересный момент, "лишние" скобки убрать-то нельзя :)
      Ответить

    • ava anonimb84a2f6fd141 01.04.2013 19:43 # 0

      for x in range(0, size):
      for y in range(0, size):
      if lines[x][y]

      Взять и уебать. На сишку, быстро, решительно!
      Ответить

    • ava anonimb84a2f6fd141 01.04.2013 19:45 # 0

      []*n - в говнопитоне так делать нельзя, ололо :)

      Надо так: [list(x) for _ in range(n)]
      Ответить

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