Skip to content

Commit 7cb52f1

Browse files
committed
Remove for ... in loops
1 parent 63fbf8e commit 7cb52f1

File tree

6 files changed

+30
-26
lines changed

6 files changed

+30
-26
lines changed
Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
"use strict";
2-
var e;
3-
for (e in document.getElementsByTagName("*")) {
4-
if (e.onCopy) {
5-
e.onCopy = null;
2+
var e = document.getElementsByTagName("*");
3+
var i;
4+
for (i = 0; i < e.length; i += 1) {
5+
if (e[i].onCopy) {
6+
e[i].onCopy = null;
67
}
7-
if (e.onPaste) {
8-
e.onPaste = null;
8+
if (e[i].onPaste) {
9+
e[i].onPaste = null;
910
}
1011
}
Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
"use strict";
2-
var e;
3-
for (e in document.getElementsByTagName("*")) {
4-
if (e.autocomplete) {
5-
e.autocomplete = 'on';
2+
var e = document.getElementsByTagName("*");
3+
var i;
4+
for (i = 0; i < e.length; i += 1) {
5+
if (e[i].autocomplete) {
6+
e[i].autocomplete = 'on';
67
}
78
}

javascripts/DOM - Enable all.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
"use strict";
2-
var e;
3-
for (e in document.getElementsByTagName("*")) {
4-
if (e.disabled) {
5-
e.disabled = 'false';
2+
var e = document.getElementsByTagName("*");
3+
var i;
4+
for (i = 0; i < e.length; i += 1) {
5+
if (e[i].disabled) {
6+
e[i].disabled = 'false';
67
}
78
}

javascripts/DOM - Show anchors.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"use strict";
2-
var anchor, id;
2+
var i, id;
33
var anchors = document.getElementsByTagName("a");
4-
for (anchor in anchors) {
5-
id = anchor.id;
4+
for (i = 0; i < anchors.length; i += 1) {
5+
id = anchors[i].id;
66
if (id !== undefined && id.length > 0) {
77
anchor.appendChild(document.createTextNode(" #" + id));
88
}

javascripts/DOM - Show feeds.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,13 @@
11
"use strict";
22
var msg = "";
33
var links = document.getElementsByTagName("link");
4-
var link;
5-
for (link in links) {
6-
if (link.rel === "alternate" && (link.type.indexOf("rss") !== -1 || link.type.indexOf("atom") !== -1 || link.type.indexOf("feed") !== -1)) {
7-
msg += link.href + "\n";
8-
}
4+
var i, link;
5+
for (i = 0; i < links.length; i += 1) {
6+
link = links[i];
7+
if (link.rel === "alternate" && (link.type.indexOf("rss") !== -1 || link.type.indexOf("atom") !== -1 || link.type.indexOf("feed") !== -1)) {
8+
msg += link.href + "\n";
9+
}
910
}
1011
if (msg.length > 0) {
11-
window.alert(msg);
12+
window.alert(msg);
1213
}

javascripts/Google Maps to SunCalc.org.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"use strict";
22
var pieces = location.href.split('/');
3-
var piece;
4-
var coordinates;
5-
for (piece in pieces) {
3+
var i, piece, coordinates;
4+
for (i = 0; i < pieces.length; i += 1) {
5+
piece = pieces[i];
66
if (piece.startsWith('@')) {
77
piece = piece.substring(1).split(',');
88
coordinates = piece[0] + ',' + piece[1] + ',19';

0 commit comments

Comments
 (0)