[js] PRosty skrypt a jednak..., Mały problem Opcje

[miallih]

Witam, mam mały problem ze skryptem,

[JAVASCRIPT] pobierz, plaintext 
<table>
		<th colspan="5"> Rozdaj staty</th>
		<tr>
			<td>1</td><td><input type="button" id="minus_sila" value= "<"></td><td><span id="sila" >8</span></td><td><input type="button"  id="plus_sila" value= ">"></td><td>5</td></tr>
		<tr>
			<td>1</td><td><input type="button"  id="minus_zrecznosc" value= "<"></td><td><span id="zrecznosc" >8</span></td><td><input type="button" id="plus_zrecznosc" value= ">"></td><td>5</td></tr>
	 </table>
<script>
var staty = ["sila", "zrecznosc"];
for (var i = 0; i < staty.length; i++)
	{
	 document.getElementById("plus_"+staty[i]).onclick = function (evt) {plus(i); };
	 document.getElementById("minus_"+staty[i]).onclick = function (evt) {minus(i); };
	}
function plus(i)
	{
	 var e=document.getElementById(staty[i]);
	 e.innerHTML++;
	}
function minus(i)
	{
	 var z=document.getElementById(staty[i]);
	 z.innerHTML--;
	}	
</script>
[JAVASCRIPT] pobierz, plaintext 

Skrypt z założenia prosty, ma zmieniać siłę i zręczność. jak przechodzi po pętli for, to alerty działają jak mają. Jak klikam na strzałki to nic się nie dzieje.
Oco chodzi?

Dzięki Thek, niestety i tak nie pomogło :/

Powód edycji: [thek]: Pierwszy i ostatni raz zmieniam bbcode, następny raz od razu zamykam temat

[krowal]

Spróbuj tak:

[JAVASCRIPT] pobierz, plaintext 
e.innerHTML = parseInt(e.innerHTML) + 1;
[JAVASCRIPT] pobierz, plaintext 

[zegarek84]

tak jak napisał @krowal + poczytaj o zasięgu zmiennych i domknięciach - w js w zasadzie prawie wszystko jest przekazywane przez referencję - zależy od kontekstu w prawdzie ale łatwiej mi będzie w ten sposób opisać jeszcze jedną lukę jaka tutaj jest:
function (evt) {plus(i); }
każda z tych funkcji i minus też przy danym zdarzeniu wykona funkcję z wartością i = staty.length; - a dlaczego - jak już wspomniałem poczytaj o domknięciach czy closures w js się to nazywa...
do przekazania zmiennej "i" z osobna jako argumentu możesz posłużyć się np. funkcją anonimową:
document.getElementById("plus_"+staty[i]).onclick = (function(i){return function (evt) {plus(i); };})(i);

poza tym elementy drzewa DOM mógłbyś wcześniej poprzypisywać do jakichś zmiennych coby ich stale nie szukać i odwoływać się do nich przez zmienne...

[miallih]

Jako iż jestem zabiegany póki co, to nie mogę do końca przysiąść przy skrypcie, ale jest problem, bo zastosowałem się do waszych wskazówek, i talej i tak nie działa :/ Klikam na buttony, a tam nic :/

[Kemsan]

[JAVASCRIPT] pobierz, plaintext 
<table>
	<th colspan="5"> Rozdaj staty</th>
		<tr>
			<td>1</td>
			<td>
				<input type="button" id="minus_sila" name="add_button" value= "<">
			</td>
			<td>
				<span id="sila" >8</span>
			</td>
			<td>
				<input type="button" id="plus_sila" name="add_button" value= ">">
			</td>
			<td>5</td>
		</tr>
		<tr>
			<td>1</td>
			<td>
				<input type="button" id="minus_zrecznosc" name="add_button" value= "<">
			</td>
			<td>
				<span id="zrecznosc" >8</span>
			</td>
			<td>
				<input type="button" id="plus_zrecznosc" name="add_button" value= ">">
			</td>
			<td>5</td>
		</tr>
</table>
<script>
var stats = [ "sila", "zrecznosc" ], elements = [ ];
window.onload = function(){
	for( i in stats ){
		document.getElementById( 'plus_'+stats[ i ] ).onclick = function(){
			plus( this.id.split( '_' )[1] );
		}
		document.getElementById( 'minus_'+stats[ i ] ).onclick = function(){
			minus( this.id.split( '_' )[1] );
		}
 
		elements[ stats[ i ] ] = document.getElementById( stats[ i ] );
	}
}
 
function plus( i )
{
	elements[ i ].innerHTML = parseInt( elements[ i ].innerHTML ) + 1;
};
 
function minus( i )
{
	elements[ i ].innerHTML = parseInt( elements[ i ].innerHTML ) - 1;
};
</script>
[JAVASCRIPT] pobierz, plaintext 

Działa, testowałem.

Ten post edytował Kemsan 7.05.2011, 22:11:19

[miallih]

Jezu dzięki, teraz zostaje tylko zrozumienie skryptu, bo już jak rowerowałem do domu to myślałęm nawet żeby przez php wywoływać onclicka ze zmienną w buttonie, ale wiem że mając tego typu skrypt jeszce go zużyje gdzie indziej. I teraz nasuwa się problem laika. Jako, że generalnie to ma być zwracane z ajaxowego requesta jako document.innerHTML. W momencie kiedy zostawimiy window.onload to nam sie wszystko posypie, bo nie będzie pierwszy onload działał. Chyba będę musiał przysiąć do HF javaScript :] tam powinno się znaleźć rozwiązanie

[Kemsan]

Działa także bez window.onload (){ }, dodałem tylko dla pewności