Seite 1 von 1

Script übersichtlicher machen.

Verfasst: 09.01.2012, 20:04
von 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.

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 15:01
von BumBumBass
wie wäre es denn mal mit 012 statt 022 bei den icon user. . schau dir mal die zahlen in den klammern an

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 15:40
von Partyboy
Also bei dem ersten steht eine 0 also muss ich da eine 2 hinmachen stimms

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 15:49
von bine_1
conUser[0]=new Array("badboy144","security");
IconUser[1]=new Array("nick4", "Nick5");
IconUser[2]=new Array("nick6", "Nick7");

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 15:55
von Partyboy
OK danke ich werde das jetzt mal versuchen ob das jetzt klappt

Danke noch mal für die schnelle hilfe

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 16:00
von 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)

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 16:03
von 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? ;)

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 16:05
von 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

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 16:07
von 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.

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 17:41
von 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);

Re: Script übersichtlicher machen.

Verfasst: 10.01.2012, 18:13
von 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.