Path: csiph.com!2.us.feeder.erje.net!feeder.erje.net!1.eu.feeder.erje.net!newsfeed.datemas.de!weretis.net!feeder4.news.weretis.net!feeder5.news.weretis.net!news.solani.org!.POSTED!not-for-mail
From: Thomas 'PointedEars' Lahn <PointedEars@web.de>
Newsgroups: de.comp.lang.javascript
Subject: Re: Gluecksrad
Date: Wed, 16 Nov 2016 18:29:08 +0100
Organization: PointedEars Software (PES)
Lines: 142
Message-ID: <10725728.O9o76ZdvQC@PointedEars.de>
References: <o04r2k$i78$1@news.albasani.net> <o0fcmj$hji$1@news.albasani.net> <3307965.kQq0lBPeGt@PointedEars.de> <o0hf12$p81$1@news.albasani.net> <1a76ea3d-6da6-d368-78b5-3073fe844d07@arnowelzel.de>
Reply-To: Thomas 'PointedEars' Lahn <cljs@PointedEars.de>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8Bit
X-Trace: solani.org 1479317349 20554 eJwNxskBwDAIA7CVSjjMOsXE+4/Q6qX0siKisiKV8mHHzL99FDLed9bOEjC0W8zdXgrVzEN9P/MSVA== (16 Nov 2016 17:29:09 GMT)
X-Complaints-To: abuse@news.solani.org
NNTP-Posting-Date: Wed, 16 Nov 2016 17:29:09 +0000 (UTC)
User-Agent: KNode/4.14.2
X-NNTP-Posting-Host: eJwNwYEBwCAIA7CXhtIyzgGl/5+wJdg0nnCCDkFzg4xUeD7esONqiF1rZ2ddqPB7tWYqDB8eqxFK
Cancel-Lock: sha1:foKO5p4A+BKq6YJzoo2JMFzpnQQ=
X-User-ID: eJwNyskBACAIA7CVVA7LOAh2/xH0nZj49Nrq5mo0diJbGO09icDKqxKUcUxXYGSFo+LHT7jSe6FVrQ6PNPcDZsEWGA==
Xref: csiph.com de.comp.lang.javascript:4800

Arno Welzel wrote:

> <https://arnowelzel.de/wp/tools/teamspeak-serverstatus>
> 
> Im Quelltext ab Zeile 146.

| <script type="text/javascript">
| /* <![CDATA[ */

Die zweite Zeile ist für XHTML sinnvoll, und dann auch nur nötig, wenn das 
XHTML-Dokument mit einem XML-Medientyp ausgeliefert wird (was den XML-Parser 
triggert) *und* der Inhalt des Elements Markup-Begrenzer wie “<” enthält; in 
HTML5 ohne XHTML-Syntax (siehe Dein doctype und Dein “html”-Element) ist sie 
unnötiger Unfug.

| function doUpdate() {

Die benutzerdefinierte Funktion sollte in einem eigenen Namespace definiert 
werden, um Namenskollisionen zu vermeiden.  Die einfachste Möglichkeit dafür 
ist die Definition in einer unmittelbar aufgerufenen anonymen Funktion:

  (function () {
    function doUpdate ()
    {
      // …
    }

    …doUpdate…
  }());

| 	try {
| 		req=new XMLHttpRequest();
| 	}
| 	catch(ms) {
| 		try {
| 			req=new ActiveXObject("Msxml2.XMLHTTP");
| 		}
| 		catch(nonms) {
| 			try {
| 				req=new 
ActiveXObject("Microsoft.XMLHTTP");
| 			}
| 			catch (failed) {
| 				req = null;
| 			}
| 		}
| 	}
| 

Es ist nicht sinnvoll, das XHR-Objekt für jeden Request neu zu erstellen;
es wird bei open() automatisch zurückgesetzt.

Deine Benennung der Exception-Variablen ist unüblich.

Auf MSXML2 zu testen ist in dieser massgeschneiderten Funktion unnötig, da 
Du keine der zusätzlichen Features verwendest.

| 	if(req != null) {
| 		req.open("GET", "/tools/tss3/status.php?lang=de", true);
| 		
| 		req.onreadystatechange = function() {
| 			switch(req.readyState) {
| 			case 4:
| 				if(req.status==200)
| 				{
| 					
document.getElementById("stats").innerHTML = req.responseText;
| 				}
| 				setTimeout("doUpdate()", 5000);
| 				break;
| 
| 			default:
| 				return false;
| 				break;     
| 			}
| 		};
| 
| 		req.setRequestHeader("Content-Type","application/x-www-
form-urlencoded");
| 		req.send(null);
| 	}

Die switch-Anweisung ist fehlerträchtig (“switch-case fall-through”) und 
hier unnötig bis falsch.


Es sollte

  var timeout;
  // …
    timeout = window.setTimeout(doUpdate, 5000);

heissen:

<https://developer.mozilla.org/de/docs/Web/API/WindowTimers/setTimeout#Passing_string_literals>
<https://de.wikipedia.org/wiki/Content_Security_Policy#Verbotene_Konstrukte_und_Alternativen>

window.clearTimeout(timeout) sollte dann onunload aufgerufen werden.


Der Aufruf von XHR::setRequestHeader() ist *so* (ungetestet) fehlerträchtig 
und hier unnötig.

| ( function( $ ) {
| 	$(document).ready(function () {
| 		setTimeout("doUpdate()", 5000);
| 	});
| } )( jQuery );

Wenn Du jQuery verwendest (was hier unnötig erscheint), solltest Du auch 
jQuery.ajax() verwenden statt einer eigenen Lösung:

<http://api.jquery.com/jQuery.ajax/>


*Nur* jQuery(document).ready(…) in einer anonymen Funktion aufzurufen ist 
unnötig fehlerträchtig.


Empfohlen wird, statt jQuery(document).ready(function () { … }) einfach 
jQuery(function () { … }) aufzurufen:

<http://api.jquery.com/ready/>

| /* ]]> */

Unfug, siehe oben.

| </script>

Das script-Element sollte sich im head-Element befinden, nicht innerhalb 
eines div-Elements innerhalb eines article-Elements.  Optimal werden 
gemeinsam genutzte Teile per URI referenziert, wobei die referenzierte 
Ressource minimiert ausgeliefert werden kann.  Das async- und das defer-
Attribut können dann benutzt werden, um das Script erst dann zu laden,
wenn es benötigt wird.

-- 
PointedEars
FAQ: <http://PointedEars.de/faq> | SVN: <http://PointedEars.de/wsvn/>
Twitter: @PointedEars2 | ES Matrix: <http://PointedEars.de/es-matrix>
Please do not cc me. / Bitte keine Kopien per E-Mail.