[JS] onClick i nic się nie zmienia :( Opcje

[Flapper]

Kod

var pkt=25
var str=8
var zr=8
var bd=8
var int=8
var wie=8
var cha=8

//Dodaj siłę
function addstr() {
pkt=pkt-1
str=str+1
}
//Odejmij siłę
function destr() {
pkt=pkt+1
str=str-1
}

document.write("Pozostałe punkty: "+pkt+"<br>")
document.write("Siła: <input type=\"button\" value=\"-\" onClick='destr()'> "+str+" <input type=\"button\" value=\"+\" onClick=\"addstr()\"><br>")
document.write("Zręczność: "+zr+"<br>")
document.write("Budowa: "+bd+"<br>")
document.write("Inteligencja: "+int+"<br>")
document.write("Wiedza: "+wie+"<br>")
document.write("Charyzma: "+cha+"")

PHP:

[PHP] pobierz, plaintext 
<?php
(...)
<div class="forms2">
	<script language="JavaScript" src="atrybuty.js">
		</script>
 
</div>
(..)
?>
[PHP] pobierz, plaintext 

Napisałem sobie taki kodzik JS i zdawać by się mogło że działa bo niby jest Siła: 8 i przyciski + i -, jednak po kliknięciu na jakiś nie ma żadnej reakcji, nic się nie dzieje. (IMG:http://forum.php.pl/style_emoticons/default/sad.gif) Cały czas mam 8 Siły i 25 punktów. Zacząłem się uczyć JS 30 min. temu, więc jeżeli popełniłem jakiś banalny błąd to proszę nie krzyczcie na mnie. (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

[Flapper]

Działa! (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Dzięki! Fajnie że tyle tu ludzi skorych do pomocy. (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Ale jest jeszcze jedna sprawa z którą mam problem. Zrobiłem ograniczenie żeby nie można było zjechać z punktami poniżej 0. Ale chciałbym jeszcze zrobić tak żeby nie można było zmniejszyć atrybutu bardziej niż do 3 ani zwiększyć powyżej 18.
Ograniczenie punktów okazało się być bardzo proste ale z tym ograniczeniem wartości atrybutów kombinuję kombinuję i nie mogę wykombinować. (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
Mój aktualny kod:

Kod

var pkt=25;
var str=8;
var zr=8;
var bd=8;
var inte=8;
var wie=8;
var cha=8;

function dodaj(atr) {
if (pkt > 0) {
pkt -= 1;
switch (atr) {
case 'str' :
    str += 1;
    document.getElementById(atr).innerHTML = str;
    break;
case 'zr' :
    zr += 1;
    document.getElementById(atr).innerHTML = zr;
    break;    
}
document.getElementById('pkt').innerHTML = pkt;
}
else
{
    alert('Nie masz dosc punktow')
}
}

function odejmij(atr) {
pkt += 1;
switch (atr) {
case 'str' :
    str -= 1;
    document.getElementById(atr).innerHTML = str;
    break;
case 'zr' :
    zr -= 1;
    document.getElementById(atr).innerHTML = zr;
    break;    
}
document.getElementById('pkt').innerHTML = pkt;
}