Newsbox mit automatischer änderung!!

[Patrick Andrieu]

Hallo Rabiator!

danke für deine Antwort.

Den Dank nehme ich an, aber wenn Du willst, dass man Dir weiterhilft, solltest Du die Tipps und Korrekturen beherzigen, die man Dir nahelegt bzw. aufzeigt. Ich füge hier ein den Code, den Du in Deinem zweiten (jetzt gesperrten) Thread gepostet hast:

<script language="JavaScript">
<!--
var MessageArray = new Array();
var timer = 120000; // 1000 millisek. = 1 Sek

MessageArray[0]="Das ist Text 1.";
MessageArray[1]="Das ist Text 2.";

function showNews(offset)
 {
         if
         (offset < MessageArray.length)

{

document.getElementById("NewsField").innerHTML = MessageArray[offset]
         offset++
         setTimeout("showNews("+offset+")", timer)
         }
         else
         {

offset = 0;
         setTimeout("showNews("+offset+")", 0)
         }
         }

-->
         </script>

Da steht immer noch »<script language="JavaScript">« und der Code ist immer noch unübersichtlich. So könnte es aussehen, um sicher auch für Dich besser lesbar zu sein:

  
<script type="text/javascript">  

  
var MessageArray = new Array();  
var timer = 120000; // 1000 millisek. = 1 Sek  
var offset;  
  
MessageArray[0]="Das ist Text 1.";  
MessageArray[1]="Das ist Text 2.";  
  
function showNews(offset) {  
         if (offset < MessageArray.length) {  
           document.getElementById("NewsField").innerHTML = MessageArray[offset]  
           offset++  
           setTimeout("showNews("+offset+")", timer)  
         }  
         else {  
           offset = 0;  
           setTimeout("showNews(offset)", 0)  
         }  
}  

  
</script>  

Sollte Dir die -eigentlich bevorzugte- Klammeranordnung nicht gefallen, dann wenigstens so:

  
function showNews(offset)  
{  
         if (offset < MessageArray.length)  
         {  
           document.getElementById("NewsField").innerHTML = MessageArray[offset]  
           offset++  
           setTimeout("showNews("+offset+")", timer)  
         }  
         else  
         {  
           offset = 0;  
           setTimeout("showNews(offset)", 0)  
         }  
}  

denn sonst, sollte Dein oder ein anderes Stück Code wachsen, verlierst Du schnell den Überblick zu welchem Block welche Klammer gehört, und machst es für andere unmöglich, in Deinem Code »einzusteigen«. Also bitte im eigenen Interesse etwas Disziplin in dieser Hinsicht.

habe die Klammer auch gesehen und das form rausgeschmissen.

Zumindest etwas ;)

Da du dich anscheinend mit der materie auskennst eine frage kann man hier

Viele andere hier sind viiiiel besser ;)

das Array mit text aus einer txt datei oder js datei füllen??
damit wäre dei News neu zu generieren viel einfacher.

JavaScript kann nicht aus Dateien lesen, reines JavaScript läuft clientseitig, das heißt, nur auf dem Browser. Und außer Cookies kann JavaScript weder auf die Festplatte schreiben noch lesen.

Natürlich kannst Du den gesamten JavaScript-Code inklusive der News-Texte auslagern und, falls Du die Anzeige dieser News in mehreren Dateien brauchst, in jeder davon z.B. mit:

  
<script type="text/javascript" src="news.js"></script>  

einbinden. Die News-Texte kannst Du dabei in der selben Form belassen, wie oben. Oder ist das zu unübersichtlich?

Viele Grüße aus Frankfurt/Main,
Patrick