Форум

atanov · #1 (**permalink**) 22.10.2018, 17:08

Доброго времени суток.

Есть такой незамысловатый html:

table
{
	border-collapse: collapse;	
}
td
{
	height: 20px;
	width: 20px; 
	border: solid black 1px;
}
.td1
{
	height: 20px;
	width: 60px; 
	border: solid black 1px;
}

<link rel = "stylesheet" type = "text/css" href = "114.css">
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">1</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">2</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">3</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">4</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">5</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">6</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">7</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<button>Click</button>
<script src = "114.js"></script>

document.getElementsByTagName('button')[0].addEventListener('click', gggg);
function gggg()
{
	let a = document.getElementsByClassName('td1').length;
	for (let i = 0; i < a; i++)
	{
		document.getElementsByClassName('td1')[i].parentNode.parentNode.parentNode.remove();
	}
}

Задачка - как удалить таблицы, в которых есть td с классом td1.
Объясните пожалуйста полному

почему нечётные удаляются???

Dilettante_Pro · #2 (**permalink**) 22.10.2018, 17:38

<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">1</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">2</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">3</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">4</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">5</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">6</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">7</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<script>
let a = document.getElementsByClassName('td1');

for (var i = a.length - 1; i >= 0; i--) { 
   a[i].parentNode.parentNode.parentNode.remove();
};
</script>

рони · #3 (**permalink**) 22.10.2018, 17:41

Сообщение от atanov

почему нечётные удаляются???

потому что getElements это живая коллекция, при удалении создаётся новый список элементов.
решение удалять с конца или document.querySelectorAll вместо getElements

Dilettante_Pro · #4 (**permalink**) 22.10.2018, 17:49

Сообщение от рони

document.querySelectorAll вместо getElements

Простая замена по скрипту ТопикСтартера не поможет - он же все время коллекцию перечитывает.

j0hnik · #5 (**permalink**) 22.10.2018, 18:03

Сообщение от Dilettante_Pro

Простая замена по скрипту ТопикСтартера не поможет - он же все время коллекцию перечитывает.

поможет если ее перед циклом сохранить в переменную

document.getElementsByTagName('button')[0].addEventListener('click', gggg);
function gggg()
{
	let a = document.getElementsByClassName('td1').length;
	var x = document.querySelectorAll('.td1');
	for (let i = 0; i < a; i++){
		x[i].parentNode.parentNode.parentNode.remove();
	}
}

рони · #6 (**permalink**) 22.10.2018, 18:10

Dilettante_Pro,
да в его случае достаточно убрать i из строки удаления

Dilettante_Pro · #7 (**permalink**) 22.10.2018, 18:10

Сообщение от j0hnik

поможет если ее перед циклом сохранить в переменную

Дык об этом я и говорю. Это уже не простая замена.

рони · #8 (**permalink**) 22.10.2018, 18:11

atanov,

<!DOCTYPE html>

<html>
<head>
  <title>Untitled</title>
  <meta charset="utf-8">
  <style type="text/css">
     table
{
	border-collapse: collapse;
}
td
{
	height: 20px;
	width: 20px;
	border: solid black 1px;
}
.td1
{
	height: 20px;
	width: 60px;
	border: solid black 1px;
     background-color: #FF0000;
}

  </style>

</head>

<body>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">1</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">2</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">3</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">4</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">5</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">6</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td class = "td1">7</td>
		</tr>
	</tbody>
</table>
<table>
	<tbody>
		<tr>
			<td></td><td></td><td></td>
		</tr>
	</tbody>
</table>
<button>Click</button>
<script>
document.getElementsByTagName('button')[0].addEventListener('click', gggg);
function gggg()
{
	let a = document.getElementsByClassName('td1').length;
	for (let i = 0; i < a; i++)
	{
		document.getElementsByClassName('td1')[0].parentNode.parentNode.parentNode.remove();
	}
}

</script>
</body>
</html>

рони · #9 (**permalink**) 22.10.2018, 18:12

j0hnik,
строки 4 и 5 лучше поменять с друг другом.

Dilettante_Pro · #10 (**permalink**) 22.10.2018, 18:16

Сообщение от рони

в его случае достаточно убрать i из строки удаления

А также All

let a = document.querySelectorAll('.td1').length;

for (var i = 0; i < a; i++) { 
   document.querySelector('.td1').parentNode.parentNode.parentNode.remove();
};

Форум

Справочник

Discord чат

Курсы javascript