java script for döngüsü hata

[Pco]

selamlar,
aşağıdaki kodu sayfamda deniyorum,döngü işler gibi görünüyor fakat sayfanın alt kısmı açılmıyor sanki sonsuza kadar dönüyormuş gibi görünüyor.

function showroom( yap ) {
var i=0;
for (i=0;i<=yap;i++)
{
document.getElementById( i ).style.visibility="visible";
document.getElementById( i ).style.display="block";
}
}

Kullanım şekli:

<select style="width:200px;" name="select" id="select">
                <option value="1" onclick="showroom('1');">1</option>
                <option value="2" onclick="showroom('2');">2</option>
                <option value="3" onclick="showroom('3');">3</option>
                <option value="4" onclick="showroom('4');">4</option>
                <option value="5" onclick="showroom('5');">5</option>
              </select>

Hata nerede?

[chesterx]

yap kelimesini tanıtmak gerekmiyor mu yeniyim ama
var yap;
şeklinde ?

PHP Kodu:

 function showroom( yap ) {
var i=0;
for (i;i<=yap;i++)
{
document.getElementById( i ).style.visibility="visible";
document.getElementById( i ).style.display="block";
}
} 


Bir de i değerine iki kere değer verilmiş sanırım umarım yardımcı olabilmişimdir.

[rset_]

döngü normal görünüyor.
hem style.visibility hem de style.display kullanmışsın. ondan da olabilir. sadece birini kullanman gerek bence.
bir de:
onclick yerine onselect kullanmayı dene. ayrıca showroom('1') yazarken 1'i tırnak içine almasan da olur. sayı çünkü.

[rset_]

chesterx, mesajından alıntı:

yap kelimesini tanıtmak gerekmiyor mu yeniyim ama
var yap;
şeklinde ?

hayır onu tanımlamak gerekmiyor. sadece alması gereken değerin onclick="showroom(x);" şeklinde verilmesi yeterli.

[Pco]

hayır dediginiz gibi olmadı.
Kodlar aşagıda. normal şekilde yapdıgımda oluyor fakat for içine atınca olmuyor.

<script>
function showroom(yap) {
var i=0;
for (i=0;i<=yap;i++)
{
document.getElementById(i).style.visibility="visible";
document.getElementById(i).style.display="block";
}}
function normal(yap) {
document.getElementById(yap).style.visibility="visible";
document.getElementById(yap).style.display="block";
}
</script>
<select style="width:200px;" name="select" id="select">
  <option value="1"  onclick ="showroom('1');">1</option>
                <option value="2" onclick ="showroom('2');">2</option>
                <option value="3" onclick ="showroom('3');">3</option>
                <option value="4" onclick ="showroom('4');">4</option>
                <option value="5" onclick ="showroom('5');">5</option>
</select>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="1">
  <tr>
    <td bgcolor="#993333">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="2">
  <tr>
    <td bgcolor="#999966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="3">
  <tr>
    <td bgcolor="#009966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="4">
  <tr>
    <td bgcolor="#FF6600">&nbsp;</td>
  </tr>
</table>

<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="5">
  <tr>
    <td bgcolor="#660000">&nbsp;</td>
  </tr>
</table>
<a href="javascript:normal(1);">1. ac
</a>

[Sniper]

hem for(i=0) hem de var i=0 demişsin. var orada fazla, onu kullanmasan da olur. yap'ı ise parseInt ile sayıya çevirerek tekrar dene:

<script>
function showroom(yap) {
for (i=0;i<=parseInt(yap);i++)
{
document.getElementById(i).style.visibility="visible";
document.getElementById(i).style.display="block";
}}
function normal(yap) {
document.getElementById(yap).style.visibility="visible";
document.getElementById(yap).style.display="block";
}
</script>
<select style="width:200px;" name="select" id="select">
  <option value="1"  onclick ="showroom('1');">1</option>
                <option value="2" onclick ="showroom('2');">2</option>
                <option value="3" onclick ="showroom('3');">3</option>
                <option value="4" onclick ="showroom('4');">4</option>
                <option value="5" onclick ="showroom('5');">5</option>
</select>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="1">
  <tr>
    <td bgcolor="#993333">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="2">
  <tr>
    <td bgcolor="#999966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="3">
  <tr>
    <td bgcolor="#009966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="4">
  <tr>
    <td bgcolor="#FF6600">&nbsp;</td>
  </tr>
</table>

<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="5">
  <tr>
    <td bgcolor="#660000">&nbsp;</td>
  </tr>
</table>
<a href="javascript:normal(1);">1. ac
</a>

[cemal]

İd'i 0 olan bir tablon yok. Bu yüzden i=0 değil , i= 1 olmalı. Ayrıca idlere değer verirken bir harfle veya _ ile başlayabiliriz, numara ile başlayamayız. Bunu sayı, harf, _ takip edebilir.
Tabloların idlerini t1, t2,.. şeklinde değiştirdim.
i daha önce tanımlanmadıysa ve ilk kez for içinde geçiyorsa var i=1 diye tanımlaman gerekir.

<script type="text/javascript">
function showroom(yap) {
for (var i=1;i<=Number(yap);i++)
{
document.getElementById("t"+i).style.visibility="visible";
document.getElementById("t"+i).style.display="block";
}}
function normal(yap) {
document.getElementById("t"+yap).style.visibility="visible";
document.getElementById("t"+yap).style.display="block";
}
</script>
<select style="width:200px;" name="select" id="select">
  <option value="1"  onclick ="showroom('1');">1</option>
                <option value="2" onclick ="showroom('2');">2</option>
                <option value="3" onclick ="showroom('3');">3</option>
                <option value="4" onclick ="showroom('4');">4</option>
                <option value="5" onclick ="showroom('5');">5</option>
</select>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="t1">
  <tr>
    <td bgcolor="#993333">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="t2">
  <tr>
    <td bgcolor="#999966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="t3">
  <tr>
    <td bgcolor="#009966">&nbsp;</td>
  </tr>
</table>
<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="t4">
  <tr>
    <td bgcolor="#FF6600">&nbsp;</td>
  </tr>
</table>

<table width="23%" height="87" border="0" cellspacing="0" style="visibility: hidden; display: none;" id="t5">
  <tr>
    <td bgcolor="#660000">&nbsp;</td>
  </tr>
</table>
<a href="javascript:normal(1);">1. ac
</a>