Говнокод #3791 — JavaScript

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

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

JavaScript / Говнокод #3791

+167

function createIMG(src,id,width,height,border)
{
	var s='<IMG SRC=';
	s+=addquo(src);
	if(id!='')
		s+=' ID='+id;
	if(width&&height)
	{
		if(width>0)
			s+=' WIDTH='+width;
		if(height>0)
			s+=' HEIGHT='+height;
	}
	s+=' BORDER='+border+'>';
	return s;
}

Оно делает картинку

Запостил:

ursus, 27 Июля 2010

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

ursus 27.07.2010 16:59 # +2

http://www.vash-otel.ru/
адрес говна. это не тот чьё имя нельзя называть но у меня есть идея их скрестить
Ответить
- 3.14159265 27.07.2010 17:07 # +3
  
  мда
  
  nimg1 = new Image()
  nimg1.src = "left1.jpg"
  nimg2 = new Image()
  nimg2.src = "left2.jpg"
  
  
  nimg3 = new Image()
  nimg3.src = "left4.jpg"
  nimg4 = new Image()
  nimg4.src = "left5.jpg"
  
  
  nimg5 = new Image()
  nimg5.src = "left7.jpg"
  nimg6 = new Image()
  nimg6.src = "left6.jpg"
  итд...
  это уже о многом говорит
  Ответить
  - ursus 27.07.2010 17:08 # 0
    
    дооо! я их думаю скрестить. это чувак отослал мне резюме на вакансию программист C# с БОЛЬШИМ опытом работы. вот что ему написать. правду?
    Ответить
    - telnet 27.07.2010 19:27 # 0
      
      Что на заборе написано, то и ему напишите.
      Ответить
3.14159265 27.07.2010 17:01 # +4

нет слов...
по-моему, усрус таки запостил реально годный говнокод
Ответить
Мистер Хэнки 27.07.2010 19:14 # −2

я в дизайне не особо волоку... тут что, через CSS надо?
Ответить
- Анонимус 27.07.2010 20:04 # +1
  
  1) причем тут дизайн? это верстка
  2) собирать надо как-бы через dom, например
  Ответить
  - mrbig66 27.07.2010 21:56 # +1
    Или так:
    
    return sprintf('<img id="%s" src="%s" class="%s">',$id,$src,$class);
    Ответить
    - Анонимус 27.07.2010 21:59 # +4
      
      Да, в джаваскрипте особенно:)
      Ответить
      - mrbig66 28.07.2010 07:50 # 0
        
        Слона-то я и не заметил. :-)
        Ответить
    - gegMOPO4 03.12.2010 15:49 # 0
      
      id=''
      Ответить
ZX_Spectrum 27.07.2010 22:02 # +2

а из-за чего весь сыр бор, собственно?
конечно, можно сделать через тру DOM, но производительность браузеров при добавлении через старый добрый innerHTML быстрее, это факт.
Ответить
- da_ff 27.07.2010 22:38 # 0
  
  нет, это не факт. корректнее использовать documentfragment.
  Ответить
- Анонимус 28.07.2010 01:59 # +2
  
  1) нет, это не факт.
  2) код может вызвать ошибку, если HTML не будет нормальным.
  3) упускается экранирование
  4) вообще собирать картину вручную , при наличии объекта Image -- глупо
  Ответить
  - ZX_Spectrum 28.07.2010 04:13 # +1
    
    1) по-моему, на квирксмод.орг было опубликованы данные замеров скорости создания большой таблицы через ДОМ и через иннерХТМЛ - разница весьма впечатляющая в пользу второго варианта. Про documentfragment там вроде не было, но если у вас есть данные такого сравнения - будет интересно узнать.
    2) ну и javascript может вызвать ошибки, если он не будет нормальным. речь же не об ошибках.
    3) экранирование чего именно - урла, больше вроде экранировать нечего? так может для этого там и стоит в 4 строке вызов какой-то функции "addquo"?
    4) возможно.
    Ответить
    - ZX_Spectrum 28.07.2010 04:26 # 0
      
      вот, нашел ссылку
      http://www.quirksmode.org/dom/innerhtml.html
      Ответить
      - bugmenot 29.07.2010 23:53 # 0
        
        в опере 100500 картинок 85х85 через new Image()/appendChild() добавляются за ~19 сек, innerHTML += '<IMG SRC="итд" много медленнее (фактически мне надоело ждать)
        Ответить
        
        ZX_Spectrum 30.07.2010 03:13 # +1
        
        ну все, придется проводить свое независимое тестирование.
        кстати, а Вы вызывали innerHTML += '<IMG SRC="итд" для каждой картинки? тогда ой! если я угадал - попробуйте сначала сформировать строку со всеми 100500 картинками, а потом уже innerHTML += all_imgs;
        Ответить
        
        bugmenot 30.07.2010 20:36 # 0
        
        читерство, не френдли - нагнет браузер, а всякие интернет эксплореры наверняка будут плеваться диалогами с предложением абортировать скрипт
        
        14671 msec DOM 0
        6571 msec innerHTML
        10572 msec DOM 0 с элементами off-screen (не умею в childNodes массив целиком записать, так что - грубо)
        
        мне уже лень, но чистый эксперимент, дающий равные условия обоим подходам ставить уже лень
        
        Ответить
        
        bugmenot 30.07.2010 20:40 # 0
        
        лень лень лень
        Ответить
Lure Of Chaos 28.07.2010 21:08 # +1

if(width&&height) - хаха
Ответить
Wyrap_nJll-oM_qpaupu 25.08.2021 21:16 # 0

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

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

JavaScript / Говнокод #3791

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

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