Script übersichtlicher machen.

[Partyboy]

Ich brauche mal wieder hilfe und zwar möchte ich diesen code bisschen übeersichtlicher machen und jetzt geht der code garnicht.Wie der da so jetzt steht geht der code garnicht,ich wolte denn auf wktool hochladen und logg mich im chat ein keine bilder hinter dem nick was ist denn an den code falsch.

var IconAdressen new Array();
var IconAdressen = new Array
("http://server1.webkicks.de/knuddel/replacer/chef1.gif", "http://freenet-homepage.de/chat-welt24/a2.gif", "http://freenet-homepage.de/chat-welt24/a3.gif");

IconUser[0]=new Array("badboy144","security");
IconUser[2]=new Array("nick4", "Nick5");
IconUser[2]=new Array("nick6", "Nick7");

for(i=0;i<IconUser.length;i++){for(j=0;j<IconUser.length;j++){ if(parent.rightFrame.document.getElementById(IconUser[j]) && parent.rightFrame.document.getElementById(IconUser[j]).innerHTML.search(IconAdressen) == -1){ parent.rightFrame.document.getElementById(IconUser[j]).innerHTML += "<img src='"+IconAdressen+"'>"; }}}} window.setInterval("OlSetIcons()", 2000)

Wer kann mir dabei helfen.Das der code wieder geht.

[BumBumBass]

wie wäre es denn mal mit 012 statt 022 bei den icon user. . schau dir mal die zahlen in den klammern an

[Partyboy]

Also bei dem ersten steht eine 0 also muss ich da eine 2 hinmachen stimms

[bine_1]

conUser[0]=new Array("badboy144","security");
IconUser[1]=new Array("nick4", "Nick5");
IconUser[2]=new Array("nick6", "Nick7");

[Partyboy]

OK danke ich werde das jetzt mal versuchen ob das jetzt klappt

Danke noch mal für die schnelle hilfe

[Partyboy]

So habe das jetzt so geändert und habe denn auf wktool hochgeladen und es kommen keine bilder nix tut sich.


var IconAdressen new Array();
var IconAdressen = new Array
("http://server1.webkicks.de/knuddel/replacer/chef1.gif", "http://freenet-homepage.de/chat-welt24/a2.gif", "http://freenet-homepage.de/chat-welt24/a3.gif");

IconUser[0]=new Array("badboy144","security");
IconUser[1]=new Array("nick4", "Nick5");
IconUser[2]=new Array("nick6", "Nick7");

for(i=0;i<IconUser.length;i++){for(j=0;j<IconUser.length;j++){ if(parent.rightFrame.document.getElementById(IconUser[j]) && parent.rightFrame.document.getElementById(IconUser[j]).innerHTML.search(IconAdressen) == -1){ parent.rightFrame.document.getElementById(IconUser[j]).innerHTML += "<img src='"+IconAdressen+"'>"; }}}} window.setInterval("OlSetIcons()", 2000)

[bine_1]

Also:

Code: Alles auswählen

 http://freenet-homepage.de/chat-welt24/a2.gif und http://freenet-homepage.de/chat-welt24/a3.gif

führt zu keinem Bild.
Ist das so richtig?

[Partyboy]

Das ist Richtig habe ja nur 2 nicks da und das ist das bild das solte kommen

http://server1.webkicks.de/knuddel/replacer/chef1.gif

Bei nick eins und nick 2

[Partyboy]

Ich habe das so gemacht wie es da steht


In diesem Beispiel würde hinter nick1 und nick2 das erste Bild angezeigt werden, bei nick3 das zweite Bild und bei nick4, nick5 und nick6 das 3. Bild.
Wie immer Nicks und URLs anpassen und in die Ankündigung kopieren.

Bei nick 1 und nick 2 das erste bild so wolte ich das haben und die anderen erst mal so lassen weil ich da noch keine nicks habe.

[Maxs]

Das liegt an ein paar Fehlern. Warum löscht du denn wichtige Scriptabschnitte oder änderst sie beim "übersichtlicher machen"? Außerdem solltest du Zeilenumbrüche beachten.

So wird es übersichtlich:

Script:Zeigen

var IconUser = new Array();
var IconAdressen = new Array();

IconUser[0] = new Array("badboy144","security");
IconAdressen[0] = "http://server1.webkicks.de/knuddel/replacer/chef1.gif";

IconUser[1] = new Array("nick4", "Nick5");
IconAdressen[1] = "http://freenet-homepage.de/chat-welt24/a2.gif";

IconUser[2] = new Array("nick6", "Nick7");
IconAdressen[2] = "http://freenet-homepage.de/chat-welt24/a3.gif";

function OlSetIcons() {
for (i=0;i<IconUser.length;i++) {
for (j=0;j<IconUser.length;j++) {
if (parent.rightFrame.document.getElementById(IconUser[j]) && parent.rightFrame.document.getElementById(IconUser[j]).innerHTML.search(IconAdressen) == -1) {
parent.rightFrame.document.getElementById(IconUser[j]).innerHTML += "<img src='"+IconAdressen+"'>";
}
}
}
}
window.setInterval("OlSetIcons()", 2000);

Die roten Stellen haben bei deinem Script einfach gefehlt. Desweiteren habe ich das Adressen-Array so angepasst, dass jede Adresse gleich unter den richtigen Usern steht. Man muss nur aufpassen, dass die Zahlen in den eckigen Klammern [] zu dem User-Array passen. Außerdem muss man bei den zahlen aufpassen, dass sie sich nicht wiederholen und richtig hochgezählt werden. Also: 0123456789... und nicht 01223847568... !

Hier der Code mit schöneren Einrückungen:

Script:Zeigen

Code: Alles auswählen

var IconUser = new Array();
var IconAdressen = new Array();
 
IconUser[0] = new Array("badboy144","security"); 
IconAdressen[0] = "http://server1.webkicks.de/knuddel/replacer/chef1.gif";

IconUser[1] = new Array("nick4", "Nick5");
IconAdressen[1] = "http://freenet-homepage.de/chat-welt24/a2.gif";

IconUser[2] = new Array("nick6", "Nick7");
IconAdressen[2] = "http://freenet-homepage.de/chat-welt24/a3.gif";

function OlSetIcons() {
  for (i=0;i<IconUser.length;i++) { 
    for (j=0;j<IconUser[i].length;j++) {
      if (parent.rightFrame.document.getElementById(IconUser[i][j]) && parent.rightFrame.document.getElementById(IconUser[i][j]).innerHTML.search(IconAdressen[i]) == -1) {
        parent.rightFrame.document.getElementById(IconUser[i][j]).innerHTML += "<img src='"+IconAdressen[i]+"'>"; 
      }
    }
  }
} 
window.setInterval("OlSetIcons()", 2000);

[Partyboy]

Achso das wusste ich ja nicht das ich drauf achten soll auf die Zeilenumbrüche gut dan weiß ich wieder was neues

Danke schön Maxs das du mir den code übersichtlicher gemacht hast so habe ich mir es vorgestellt.

Einfach nur klasse.