random color prob.

autopost · Jan 10, 2008

heyy guys.. im new to this place..

but i was hopin u could give me some answers on this(btw i dont know a whole lot about html.. just what ive picked up- im pretty much self-taught) ok heres my code: 
<html> 
<head> 
<script> 
var all=new Array(6) 
all[0]="A" 
all[1]="B" 
all[2]="C" 
all[3]="D" 
all[4]="E" 
all[5]="F" 
var all2=new Array(9) 
all2[0]="1" 
all2[1]="2" 
all2[2]="3" 
all2[3]="4" 
all2[4]="5" 
all2[5]="6" 
all2[6]="7" 
all2[7]="8" 
all2[8]="9" 
var randalllet=Math.round(Math.random()*6) 
var randallnum=Math.round(Math.random()*9) 
var randcol2=new Array(6) 
randcol2[0]=all[randalllet] 
randcol2[1]=all[randallnum] 
randcol2[2]=all[randallnum] 
randcol2[3]=all2[randalllet] 
randcol2[4]=all2[randalllet] 
randcol2[5]=all2[randalllet] 
var randcol3=Math.round(Math.random()*6) 
var randcol=randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3] 
document.write("hello") 
</script> 
</head> 
<body> 
</body> 
</html> 
 
 
the prob is that the colors seem to be in greyscale.. im not sure why.. prolly its an easy job to figure out but i sure cant.. i wrote the entire code myself based on somethin i saw somewhere else.. so im not suprised theres an error in it

also if there is a way to shorten it that would be much appreciated 
 
thxjust being picky first: 
 
1) <script> should be <script type='text/javascript'> 
 
2) all lines should be delimited by ; for clarity. 
 
3) - 
 
 
randcol2[0]=all[randalllet] 
randcol2[1]=all[randallnum] // could be out of range of array 
randcol2[2]=all[randallnum] // could be out of range of array 
randcol2[3]=all2[randalllet] 
randcol2[4]=all2[randalllet] 
randcol2[5]=all2[randalllet] 
 
 
 
so you make a set of random hex digits in randcol2, then randomly selected one of those digits and repeat it 6 times: 
 
var randcol=randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3] 
 
this will be grey because #xxxxxx where x is all equal is only an intensity. 
 
write this instead: 
 
var randcol=randcol2[0]+randcol2[1]+randcol2[2]+randcol2[3]+randcol2[4]+randcol2[5] 
 
 
this should help your code be error free, and create a pseudo-random colour.thx.. but now it isnt totally random because its always going to be a letter, a #, a #, a letter, a letter, and a letter making up the hex value right? 
 
 
 
oooohhh i see i made an error before- randcol2 should be: 
 
var randcol2=new Array(6) 
randcol2[0]=all[randalllet] 
randcol2[1]=all[randalllet] 
randcol2[2]=all[randalllet] 
randcol2[3]=all2[randallnum] 
randcol2[4]=all2[randallnum] 
randcol2[5]=all2[randallnum] 
 
but- that still doesnt solve the prob. and ur way it can't be ANY color- it has to have a certain order of letters and numbers.. like i said abovevar randcol3=Math.round(Math.random()*6) 
var randcol=randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3] 
 
 
I think the prob is that it doesnt recognize 'randcol3' as a number.. could that be it? and if so how could i fix that? but on the other hand: 
 
var all=new Array(6) 
all[0]="A" 
all[1]="B" 
all[2]="C" 
all[3]="D" 
all[4]="E" 
all[5]="F" 
var all2=new Array(9) 
all2[0]="1" 
all2[1]="2" 
all2[2]="3" 
all2[3]="4" 
all2[4]="5" 
all2[5]="6" 
all2[6]="7" 
all2[7]="8" 
all2[8]="9" 
var randalllet=Math.round(Math.random()*6) 
var randallnum=Math.round(Math.random()*9) 
var randcol2=new Array(6) 
randcol2[0]=all[randalllet] 
randcol2[1]=all[randalllet] 
randcol2[2]=all[randalllet] 
randcol2[3]=all2[randallnum] 
randcol2[4]=all2[randallnum] 
randcol2[5]=all2[randallnum] 
 
 
 
 
in that case, it recognizes 'randalllet' and 'randallnum' as numbers, right? so why doesn't it in the other case?var randcol=randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol3]+randcol2[randcol 
3]+randcol2[randcol3] 
[/quote] 
my way was more random than what you have above. You're only going to pick out a single item in the randcol2 array, and repeat it 6 times. This is why you're getting grey. 
 
okay, to make it completely random:

Code:

<br />
var all = "0123456789ABCDEF";<br />
<br />
var color = new array[6];<br />
<br />
for (var i = 0; i < 6; i++)<br />
{<br />
  var index = Math.round(Math.random()*15);<br />
<br />
  color[i] = all[index];<br />
}<br />
<br />
document.write("<font size=7 color=\"#"+color+"\">hello</font>")<!--content-->Okay, this is as simple as it can get:<br />
<br />
<script language="JavaScript" type="text/javascript"><!--<br />
var randcol='#';<br />
var opts='ABCDEF1234567890';<br />
while(randcol.length<7){<br />
    randcol+=opts.charAt(Math.floor(Math.random()*opts.length));<br />
}<br />
//--></script><!--content-->oooohhhh i get it now it only picks 1 random thing + repeats it.. tht makes sense im such a moron lol :) told ya i was a newb.. :D thx alot!!<!--content-->

random color prob.

autopost

Guest