[JS] zmiana zawartości diva (znika treść, a nie powinna)

Problem pewnie będzie banalny, ale zwyczajnie już nie wiem jak. A piszę, bo zdaję sobie sprawę, że ktoś, kto się zna, pomoże mi w 5 minut. Przynajmniej taką mam nadzieję...

http://xxx

co muszę dodać do skryptu, który mam na wyżej podanej stronie, aby przy kliknięciu w pierwszej lub drugiej kolumnie nie znikała treść w kolumnie następnej?

<script type="text/javascript"> 
function menu1(element) { 
 i = 0; 
 while (e = document.getElementById("m" + i++)) {
   e.style.display = "none";
 } 
 document.getElementById(element).style.display = "inline"; 
} 
menu1('m0');
 
function menu2(element) { 
 i = 5; 
 while (e = document.getElementById("m" + i++)) { 
   e.style.display = "none";
 } 
 document.getElementById(element).style.display = "inline"; 
} 
menu2('m5');
 
function menu3(element) { 
 i = 10; 
 while (e = document.getElementById("m" + i++)) { 
   e.style.display = "none";
 } 
 document.getElementById(element).style.display = "inline"; 
} 
menu3('m10');
</script>

Z góry dzięki za pomoc!

pozdrawiam

Obawiam się, że ten skrypt to w ogóle nadaje się na śmietnik :/.

Od razu widać, że totalnie przeczy podstawowej zasadzie rozsądnego programowania: DRY. Poza tym, może troszkę za sprytnie używa rzeczy typu użycie operatora przypisania w warunki pętli while, a do tego jeszcze inkrementacja indeksu w tym samym wyrażeniu. I na pewno nazwa argumentu element jest bardzo zła. To nie jest element (obiekt DOM), tylko ID elementu (String).

Czemu to nie działa? To oczywiste. Na początku każdej z funkcji ukrywasz wszystkie elementy w dokumencie, które mają ID z literą m oraz liczbą począwszy od pewnej wartości w górę. Czyli: od pewnej liczby, do końca. A Ty chyba chcesz tylko ukryć elementy z pewnego zakresu, prawda? Tzn. w przypadku drugiej kolumny elementy m5-m9, a nie m5-[do końca].

Skoro liczbę elementów w menu masz zakodowaną na stałe, to możesz też użyć tej liczby i iterować się przez elementy o ID w pewnym zakresie. Naturalniejsze byłoby tu użycie pętli for zamiast while.

Takie zakodowane na stałe menu byłoby jednak nieelastyczne (teraz też jest).

Rozumiem, że funkcja musi schować wszystkie elementy z danej sekcji oprócz jednego? Lepiej byłoby, żeby np. funkcja dla pierwszej kolumny pobierała wszystkie elementy p znajdujące się w #menu1, iterowała się przez nie i ukrywała je (tak naprawdę, to #menu1 powinno być tagiem

• , a nie

thx, dokładnie o coś takiego mi chodziło. żeby ktoś... naprowadził mnie na sposób myślenia js. jakkolwiek to brzmi :) ostatecznie i tak trzeba to opanować, a na teraz zastosuję jakieś inne rozwiązanie.

pozdrawiam