Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 29.07.2019, 15:07
Новичок на форуме
Отправить личное сообщение для crabkilla Посмотреть профиль Найти все сообщения от crabkilla
 
Регистрация: 19.07.2019
Сообщений: 5

Изменение html элемента с помощью JS
Всё тот же код из задания про удаление элемента, но теперь требуется заменить элемент с плохой картинкой спаном, в котором будет текст Now here is a ‘span’ element. внутри этого самого спана.
<!DOCTYPE html>
  <head>
    
  </head>
  <body>
    <p style="color: navy"> There's only picture here...
     <br>
     <img src="http://ipic.su/img/img7/fs/burger.1537205489.jpg" width="100px" height="100px" />
    </p>
    <br>
    <p> This one cannot be displayed. We have to replace it. 
     <img alt="bad picture">
    </p>
    
    <script type="text/javascript">
    var pElement=document.getElementsByTagName('p')[1];
    var imgElement=document.getElementsByTagName('img')[1];
    var sElement = document.createElement('span'); 
    var txt = "Now here is a ‘span’ element.";
    sElement.innerHTML = txt;
    pElement.replaceChild(sElement,imgElement);
    </script>
  </body>
</html>

Вот что получилось, в HTML текст отображается, но значение спана все ещё не соответствует тексту, даже не знаю как это поправить
Ответить с цитированием
  #2 (permalink)  
Старый 29.07.2019, 15:19
Аватар для ksa
ksa ksa вне форума
CacheVar
Отправить личное сообщение для ksa Посмотреть профиль Найти все сообщения от ksa
 
Регистрация: 19.08.2010
Сообщений: 14,228

Сообщение от crabkilla
даже не знаю как это поправить
Его нужно добавить на страницу в нужное место...

<!DOCTYPE html>
<html>
<head>
<meta http-equiv='Content-Type' content='text/html; charset=windows-1251' />
<!--
<script src='https://code.jquery.com/jquery-latest.js'></script>
<script src="https://code.angularjs.org/1.3.9/angular.min.js"></script>
<script src="https://code.angularjs.org/1.3.9/angular-route.js"></script>
-->
<style type='text/css'>
</style>
<script type='text/javascript'>
</script>
</head>
<body>
<p style="color: navy"> There's only picture here...
	<br>
	<img src="http://ipic.su/img/img7/fs/burger.1537205489.jpg" width="100px" height="100px" />
</p>
<br>
<p> This one cannot be displayed. We have to replace it. 
	<img alt="bad picture">
</p>
<script type="text/javascript">
var imgElement=document.getElementsByTagName('img')[1];
var pElement=imgElement.parentNode;
var sElement = document.createElement('span'); 
sElement.innerHTML = "Now here is a ‘span’ element.";
pElement.removeChild(sElement,imgElement);
pElement.appendChild(sElement);
</script>
</body>
</html>
Ответить с цитированием
  #3 (permalink)  
Старый 29.07.2019, 15:25
Аватар для ksa
ksa ksa вне форума
CacheVar
Отправить личное сообщение для ksa Посмотреть профиль Найти все сообщения от ksa
 
Регистрация: 19.08.2010
Сообщений: 14,228

Сообщение от crabkilla
даже не знаю как это поправить
Или так...
pElement.appendChild(sElement);
pElement.replaceChild(sElement,imgElement);

Т.е. на момент выполнения replaceChild() оба дитя должны существовать в родителе...
Ответить с цитированием
  #4 (permalink)  
Старый 29.07.2019, 15:56
Аватар для Malleys
Профессор
Отправить личное сообщение для Malleys Посмотреть профиль Найти все сообщения от Malleys
 
Регистрация: 20.12.2009
Сообщений: 1,714

ksa, 🌚

crabkilla, вот правильное решение...
Вариант №1 Одноразовое замещение ломанных картинок после загрузки...
<!DOCTYPE html>

<head></head>

<body>
	<p style="color: navy"> There's only picture here...
		<br>
		<img src="http://ipic.su/img/img7/fs/burger.1537205489.jpg" width="100px" height="100px" />
	</p>
	<br>
	<p id="pic"> This one cannot be displayed. We have to delete it.
		<img alt="bad picture">
	</p>

	<script>
		addEventListener("load", function() {
			for(const image of document.images) {
				if(image.naturalWidth === 0) {
					const node = document.createElement("span");
					node.textContent = "Now here is a ❄️.";
					image.replaceWith(node);
				}
			}
		});
	</script>
</body>

</html>


Вариант №2 Живое замещение ломанных картинок ...
<!DOCTYPE html>

<head></head>

<body>
	<script>
		const observer = new MutationObserver(mutationRecords => {
			for(const mutationRecord of mutationRecords) {
				if(mutationRecord.type === "childList") {
					for(const node of mutationRecord.addedNodes) {
						if(node instanceof HTMLImageElement) {
							if(!node.hasAttribute("src"))
								replaceBrokenImage(node);
							else
								node.addEventListener("error", ()=> replaceBrokenImage(node));
						}
					}
				}
			}
		});
		observer.observe(document.body, { childList: true, subtree: true });
		function replaceBrokenImage(image) {
			const node = document.createElement("span");
			node.textContent = "Now here is a ❄️.";
			image.replaceWith(node);
		}
	</script>
	<p style="color: navy"> There's only picture here...
		<br>
		<img src="http://ipic.su/img/img7/fs/burger.1537205489.jpg" width="100px" height="100px" />
	</p>
	<br>
	<p id="pic"> This one cannot be displayed. We have to delete it.
		<img alt="bad picture">
	</p>
</body>

</html>
Ответить с цитированием
  #5 (permalink)  
Старый 29.07.2019, 16:20
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

Malleys,
создание элементов в цикле дорогая операция
для этого есть клонирование, фрагмент, insertAdjacentHTML ...
кроме того сам подход замены битых картинок средствами js напоминает (........)
вы правильно прочитали
для этого достаточно пару строк в nginx или htaсcess, обработать сервером

Последний раз редактировалось Vlasenko Fedor, 29.07.2019 в 16:23.
Ответить с цитированием
  #6 (permalink)  
Старый 29.07.2019, 16:35
Аватар для Malleys
Профессор
Отправить личное сообщение для Malleys Посмотреть профиль Найти все сообщения от Malleys
 
Регистрация: 20.12.2009
Сообщений: 1,714

Сообщение от Poznakomlus
для этого достаточно пару строк в nginx или htaсcess, обработать сервером
Для этого есть Service Worker.

Сообщение от Poznakomlus
кроме того сам подход замены битых картинок средствами js напоминает (........)
Пока вы не придумали, что же оно означает... JavaScript является языком программирования общего назначения. Сеть является не единственным источником данных! Это может быть и файловая система и локальный кеш...

Сообщение от Poznakomlus
создание элементов в цикле дорогая операция
Однако программирование работает именно так! Циклы используют и при манипуляции узлов DOM и при разработке игр... А вы предлагаете unroll делать? (надеюсь не руками!)

Сообщение от Poznakomlus
пару строк в nginx или htaсces
и в итоге логика приложения будет размазана уже по 2-ум местам

Сообщение от Poznakomlus
создание элементов в цикле дорогая операция
Кто-то где-то это пукнул (я так понимаю в контексте JS) и вы повторяете как мантру! А если вы вычисляли какие-то тесты с 10 000 000 итерациями цикла с созданием элементов, то вот когда будет в приложении именно такое кол-во итерации, тогда вы и говорите, что это дорогая операция!

Последний раз редактировалось Malleys, 29.07.2019 в 16:46.
Ответить с цитированием
  #7 (permalink)  
Старый 29.07.2019, 16:46
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

https://learn.javascript.ru/multi-insert
читаем учебник
Ответить с цитированием
  #8 (permalink)  
Старый 29.07.2019, 16:55
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

img.onload = function() {
    // the image is ready
};
img.onerror = function() {
    // the image has failed
};

Это для
Сообщение от Malleys
Для этого есть Service Worker
Ответить с цитированием
  #9 (permalink)  
Старый 29.07.2019, 17:31
Аватар для Malleys
Профессор
Отправить личное сообщение для Malleys Посмотреть профиль Найти все сообщения от Malleys
 
Регистрация: 20.12.2009
Сообщений: 1,714

Сообщение от Poznakomlus
https://learn.javascript.ru/multi-insert
читаем учебник
Читаю...

Однако, всё это так и так это знаю! И я обычно не использую innerHTML+=, insertAdjacentHTML поскольку это ломает объекты или является не ООП подходом!

Я так понимаю, вы хотели доказать какую-то, как вам кажется, мою ошибочность... однако статья, на которую вы дали ссылку, как раз таки подтверждает все мои взгляды!

Вот...
  • https://learn.javascript.ru/multi-in...rt-bench-tbody Бенчмарк
    Потыкал 8 раз на каждый тест, вот циферки, которые выскакивали в alert...
    • TBODY сразу DOM: 84 109 99 79 90 94 81 95
    • Отложенная вставка TBODY в DOM: 90 87 96 94 110 90 76 75
    Я считаю, что оба способа хороши!
  • https://learn.javascript.ru/multi-in...cumentfragment DocumentFragment

    Цитата:
    Важно для старых браузеров
    Оптимизация, о которой здесь идёт речь, важна в первую очередь для старых браузеров, включая IE9-. В современных браузерах эффект от нее, как правило, небольшой, а иногда может быть и отрицательным.
    Как видите, то что вы предлагаете, является сильно не популярным и крайним взглядам, характерным для микроскопически малого кол-ва пользователей IE9- 📉

    Цитата:
    Понять текущее положение вещей вы можете, запустив следующий небольшой бенчмарк.
    с фрагментом работает примерно в 0,9...1,5 раз медленней... В общем думаю оба варианта хороши! (Без фрагмента было 10 15 13, с фрагментом 14 22 12)

    И там делается вывод в конце
    Цитата:
    DocumentFragment позволяет минимизировать количество вставок в большой живой DOM. Эта оптимизация особо эффективна в старых браузерах, в новых эффект от неё меньше или наоборот отрицательный.
    Да, оно так и есть! 😜

Я прочитал статью и запустил тесты, не знаю, что именно вы хотели мне сказать...

Посмотрел с телефона (результаты такие же, т. е. вставка через фрагмент незначительно медленней)

Т. е. разумная точка зрения такая, что можно использовать оба варианта! 🎉

На компьютере был Chrome 75 и телефоне Chrome 74!

Сообщение от Poznakomlus
https://learn.javascript.ru/multi-insert
читаем учебник
А вы прочитали?

Сообщение от Poznakomlus Посмотреть сообщение
img.onload = function() {
    // the image is ready
};
img.onerror = function() {
    // the image has failed
};

Это для
Сообщение от Malleys
Для этого есть Service Worker
Вы воспринимаете всё как-то фанатично... Типо есть единственно истинный способ вставки, единственный истинный способ обработки ошибки загрузки картинки...

Это способ подходит только если вы работаете с конкретным изображением... А если вы вставляете HTML-код в DOM и в коде есть картинки, то правильней использовать именно Service Worker.

И Service Worker предоставляет централизованный способ обработки запрашиваемых ресурсов, будь то шрифты, картинки интерфейса, данные приложения, шаблоны, всё, что нужно для работы приложения. 😎

Последний раз редактировалось Malleys, 29.07.2019 в 17:55.
Ответить с цитированием
  #10 (permalink)  
Старый 29.07.2019, 18:12
Аватар для Vlasenko Fedor
Профессор
Отправить личное сообщение для Vlasenko Fedor Посмотреть профиль Найти все сообщения от Vlasenko Fedor
 
Регистрация: 13.03.2013
Сообщений: 1,572

1. не стоит перекручивать задачи
2. в автономных приложениях битую картинку должен видеть разработчик и править
3. единственно где проверяют картинки это контент редактируемый пользователями на сервере. сервера настраивают всегда. Не стоит считать конфигурацию сервера вредом. Это наоборот полезно и нужно.
4. когда вы утверждаете в правильности вашего решения, вы думаете что можете ошибаться? Зачем эта замена (надо заменять сервером). И если уж так горит js то по событию image onerror заменять src на base64 одно пиксельной картинкой
onload и onerror для image здесь отлично работают
5. исходя из ваших убеждений разработчики языка написали много ненужных вещей clone, onerror ...
ох уж эти разработчики
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Изменение стиля через JS. Blizzard Общие вопросы Javascript 4 18.04.2017 13:53
как из js передать параметр в html wins Общие вопросы Javascript 6 22.12.2013 15:56
Изменение цвета ссылки с помощью js Alex Pacifik Элементы интерфейса 3 31.01.2012 15:08
Частичное изменение содержимого элемента kuhok Events/DOM/Window 28 07.09.2009 22:44
Как с помощью JS "на-лету" менять часть HTML кода greendoc Общие вопросы Javascript 2 18.03.2008 20:43