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

    −117

    1. 1
    2. 2
    3. 3
    4. 4
    cur.execute("UPDATE public.driver SET name=(%(name)s) WHERE id_driver=(%(id_driver)s)", {"name" : name, "id_driver" : id_driver})
    cur.execute("UPDATE public.driver SET surname=(%(surname)s) WHERE id_driver=(%(id_driver)s)", {"surname" : surname, "id_driver" : id_driver})
    cur.execute("UPDATE public.driver SET patronymic=(%(patronymic)s) WHERE id_driver=(%(id_driver)s)", {"patronymic" : patronymic, "id_driver" : id_driver})
    cur.execute("UPDATE public.driver SET phone_number=(%(phone_number)s) WHERE id_driver=(%(id_driver)s)", {"phone_number" : phone_number, "id_driver" : id_driver})

    Обновляем модельку в бд.
    По стопам http://govnokod.ru/17196

    Запостил: heyzea1, 01 Декабря 2014

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

    • cur.execute("""SELECT id_driver, name, surname, patronymic, phone_number, id_car, public.users.login, is_freelancer
                          FROM public.driver
                          INNER JOIN public.users ON public.users.id_user = public.driver.id_driver
                          WHERE id_taxi=%(id_taxi)s AND (phone_number ILIKE %(filtr)s OR name ILIKE %(filtr)s or surname ILIKE %(filtr)s)
                          LIMIT %(count)s OFFSET %(start_pos)s
                          """, {"id_taxi": id_taxi,"filtr" : '%'+filtr+'%', "start_pos" : (page - 1) * records_per_page, "count" : records_per_page})
      row = cur.fetchall()
      return [{"id_driver" : id_driver, "name" : name, "surname" : surname, "patronymic" : patronymic, "phone_number" : phone_number, "id_car" : id_car, "login" : login} for id_driver, name, surname, patronymic, phone_number, id_car, login in row]

      Допиливание этого говна вызывает во мне триждыбл**ую ненависть. Дорогой коммерческий проект в наследство на поддержку, да.
      Ответить
      • Что не так?
        Ответить
        • наверное то, что чтобы добавить пару выходных полей в метод, который по смыслу должен возвращать вообще всю инфу из бд, мне приходится править код как минимум в 3ех местах.
          Ответить
    • Расмус нашёл оригинальный способ размножения.
      Ответить

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