Говнокод #16207 — SQL — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

SQL / Говнокод #16207

−166

CREATE OR REPLACE FUNCTION "GET_SOTR_FULL_NAME" (sotr_id in number)
  RETURN  varchar2 IS
cursor surname (sotr_id number)
 is
  select fc_fam from tsotr
  where tsotr.fk_id=sotr_id;
cursor name (sotr_id number)
 is
  select fc_name from tsotr
  where tsotr.fk_id=sotr_id;
cursor patronym (sotr_id number)
 is
  select fc_otch from tsotr
  where tsotr.fk_id=sotr_id;
 fio   varchar2(90);
 fam   varchar2(30);
 im    varchar2(30);
 otch  varchar2(30);
BEGIN
  open surname (sotr_id);
   fetch surname into fam;
  close surname;
  open name (sotr_id);
   fetch name into im;
  close name;
  open patronym (sotr_id);
   fetch patronym into otch;
  close patronym;
   fio:=fam||' '||im||' '||otch;
  return fio;
END;

По ИД сотрудника получаем его ФИО

Запостил:

Dimm_J, 23 Июня 2014

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

wvxvw 23.06.2014 16:59 # 0

ФИО должно быть 92 символа... Это человек так с джоинами боролся?
Ответить
- bormand 23.06.2014 17:06 # 0
  
  Фамилия - 30, имя - 30, отчество - 30. Итого - 90. Все норм.
  Ответить
  - defecate-plusplus 23.06.2014 17:09 # 0
    
    причем, латинских, ога
    Ответить
    - bormand 23.06.2014 17:27 # +1
      
      Rhoshandiatellyneshiaunneveshenk Koyaanisquatsiuth Williams
      
      P.S. Почему латинских?
      P.P.S. 32 символа, имя не влезло.
      Ответить
      - defecate-plusplus 23.06.2014 17:40 # +1
        
        потому что varchar2(30) это в точности varchar2(30 byte)
        альтернативы:
        nvarchar2(30) - строго юникодная строка
        или varchar2(30 char) - 30 символов в установленной кодировке БД (обычно никто не играет с огнем и ставит utf)
        Ответить
        
        defecate-plusplus 23.06.2014 17:45 # 0
        
        > в точности varchar2(30 byte)
        селф-фикс: регулируется параметром NLS_LENGTH_SEMANTICS, который по умолчанию byte
        Ответить
        
        bormand 23.06.2014 18:20 # 0
        
        Бля, так это еще и байты...
        
        Кстати, а почему он varchar2? Первая версия варчара оказалась неудачной?
        Ответить
        
        defecate-plusplus 23.06.2014 18:25 # +2
        
        да
        к слову, varchar давно синоним для varchar2, но все как-то не осмеливаются перейти на человеческое название
        и сам оракл почему-то не советует
        
        The VARCHAR datatype is synonymous with the VARCHAR2 datatype. To avoid possible changes in behavior, always use the VARCHAR2 datatype to store variable-length character strings.
        Ответить
        
        bormand 23.06.2014 18:33 # 0
        
        Эх, а в слонёнке с этим всё просто: SQL defines two primary character types: character varying(n) and character(n), where n is a positive integer. Both of these types can store strings up to n characters (not bytes) in length.
        Ответить
        
        Lokich 24.06.2014 11:17 # 0
        
        мне человек, имеющий сертификаты по ораклу, на этот вопрос отвечал, что они накосячили с типом varchar, и он получился корявый. поэтому добавили varchar2, чтобы не нарушать совместимость с уже имеющимися базами при переходе на новые версии. после чего, они порезали его, и сделали вместо него синоним varchar2
        Ответить
        
        kegdan 24.06.2014 12:41 # −1
        
        Насколько я помню из моего непродолжительного анального сношения^W^W общения с Oracle BDSM^W RDBMS у них с большинством типов какие то косяки, все крашется, взрывается...
        Ответить
        
        bormand 23.06.2014 18:36 # 0
        
        > обычно никто не играет с огнем и ставит utf
        Зато в мускуле по-умолчанию только latin1_swedish_ci, только хардкор... Наверное каждый, кто юзал мускуль, на эту хрень хоть раз налетал, замечая, что "как-то оно не так сортирует русские буквы", а потом матерился и перезаливал базу...
        Ответить
        
        defecate-plusplus 23.06.2014 19:00 # 0
        
        оракл при установке сам предлагает al32utf8, надо сильно постараться поменять его на latin1
        Ответить
        
        bliznezz 23.06.2014 19:45 # 0
        
        обычно latin_iso1, и потом при проблемах оказыватся битая кодировка дампа.
        Ответить
  - bormand 23.06.2014 17:30 # +1
    
    Ой, 92, я пробелы не заметил.
    Ответить
Vince 23.06.2014 18:59 # +4

ах ты ж ебаный ты нахуй!
Ответить
eth0 23.06.2014 20:08 # +1

Лютая магия с курсорами.
Ответить
- defecate-plusplus 23.06.2014 21:07 # 0
  
  какая ж магия
  очевидно, чукча не знал, как из одного курсора сразу в три переменных зафетчить, что можно склеить строки прямо в селекте
  да и смысл в хранимой процедуре тут, она ничего особенного не делает - только лишний раз контекст переключать
  Ответить
  - bliznezz 24.06.2014 08:18 # +1
    
    это задел для будущей оптимизации.
    Ответить
    - kegdan 24.06.2014 11:14 # +3
      
      Скорее для анальной окупации
      Ответить
  - Lokich 24.06.2014 12:59 # +4
    зачем вообще этот курсор нужен? очевидно же, что функция используется в запросе, для каждой строки, и для каждой строки она вызывается. энивэй, переключение контекста pl-sql <---> sql это всегда плохо, и занимает не мало времени.
    
    select *, (select fc_fam||' '||c_name||' '||fc_otch from tsotr) as FIO where tsotr.fk_id=sotr_id from users_table;
    
    что-то типа этого решило бы все его проблемы с производительностью
    Ответить
    - bormand 24.06.2014 13:03 # +1
      
      У тебя where отклеилось и куда-то не туда прилипло.
      Ответить
      - Lokich 24.06.2014 13:56 # 0
        
        главное, все поняли, что я хотел сказать :)
        Ответить
    - eth0 24.06.2014 19:49 # 0
      
      Именно такой вопрос у меня и возник.
      Ответить

Говнокод: по колено в коде.

SQL / Говнокод #16207

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

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