Timer Problems in Javascript

Stevesparty

08-19-2007, 07:00 PM

I have a series of timers I am using for an online game and when I try to add another timer I cannot get it to work. All the other timers seem to work fine but not the new one.

<head>
</head>
<body bgcolor="black">

<a class="one" href="javascript:maReset(2)">90</a> - 
<a class="one" href="javascript:maReset(12)">60</a> - 
<a class="one" href="javascript:maReset(13)">45</a>
<a class="one" href="javascript:setFlight()">Travel Time!</a></td>

<a class="one" href="javascript:maReset(4)">Whack</a>
<a class="one" href="javascript:setWhack()">Whack Time!</a>

</td>

<a class="one" href="javascript:maReset(5)">Aggravated</a> - 

<a class="one" href="javascript:maReset(6)"> Study / Train</a> - 
<a class="one" href="javascript:maReset(10)">Work DH</a>
<a class="one" href="javascript:maReset(10)"></a>
<a class="one" href="javascript:setAgg()">Aggravated Time!</a> </td></tr><tr>

<a class="one" href="javascript:maReset(1)">Earn</a>
<a class="one" href="javascript:setEarn()">Earn Time!</a>

<a class="one" href="javascript:maReset(9)">Case</a>
<a class="one" href="javascript:setCase()">Case Time!</a>

<a class="one" href="javascript:maReset(7)">Launder</a>
<a class="one" href="javascript:setLaunder()">Launder Time!</a>

</td>

<a class="one" href="javascript:maReset(8)">Push Drugs</a> - 
<a class="one" href="javascript:maReset(11)">BM
<a class="one" href="javascript:setDrugs()">Drugs/BM Time!</a>
 

<a class="one" href="javascript:maReset(14)">LogOut
<a class="one" href="javascript:setlogout()">Time to Logout</a>

<a class="one" href="javascript:maReset(15)">Train
<a class="one" href="javascript:settrain()">Time to Train</a>

</td>

</tr>

<script language="JavaScript">


</script>
</body>

The timer which isn't working is the "settrain" timer. I must have made a mistake somewhere in that mess but I honestly can't find where I messed up. Any help would be appreciated, Thanks!

rwedge

08-19-2007, 08:36 PM

'mBm' and 'mBs' are not defined
document.getElementById("traintim").innerHTML = ""+mBm+"min,"+mBs+"sec";

Stevesparty

08-19-2007, 08:49 PM

I think I changed it so that it is defined:
var mBm=parseInt(matrain/60);
var mBs=parseInt(matrain-(parseInt(matrain/60)*60))

if(matrain <=0){
document.getElementById("traintim").innerHTML="Train Time!";}else{
document.getElementById("traintim").innerHTML = ""+mBm+"min,"+mBs+"sec";
}
if(matrain ==0){alert("Train Time!");}
It, however, still doesn't work. Is there anything else I have to do to define them?

rwedge

08-19-2007, 09:12 PM

matrain counts for me

Stevesparty

08-19-2007, 11:46 PM

Thats weird because it definitely doesn't count for me. Can you post a copy of the code that works for you?

rwedge

08-20-2007, 01:56 AM

<head>
</head>
<body bgcolor="black">

<table>
<tr>
<td>
<a class="one" href="javascript:maReset(2)">90</a> - 
<a class="one" href="javascript:maReset(12)">60</a> - 
<a class="one" href="javascript:maReset(13)">45</a>
<a class="one" href="javascript:setFlight()">Travel Time!</a>
<a class="one" href="javascript:maReset(4)">Whack</a>
<a class="one" href="javascript:setWhack()">Whack Time!</a>
<a class="one" href="javascript:maReset(5)">Aggravated</a> - 
<a class="one" href="javascript:maReset(6)"> Study / Train</a> - 
<a class="one" href="javascript:maReset(10)">Work DH</a>
<a class="one" href="javascript:maReset(10)"></a>
<a class="one" href="javascript:setAgg()">Aggravated Time!</a>
<a class="one" href="javascript:maReset(1)">Earn</a>
<a class="one" href="javascript:setEarn()">Earn Time!</a>
</td>
</tr>
<tr>
<td>
<a class="one" href="javascript:maReset(9)">Case</a>
<a class="one" href="javascript:setCase()">Case Time!</a>
<a class="one" href="javascript:maReset(7)">Launder</a>
<a class="one" href="javascript:setLaunder()">Launder Time!</a>
<a class="one" href="javascript:maReset(8)">Push Drugs</a> - 
<a class="one" href="javascript:maReset(11)">BM
<a class="one" href="javascript:setDrugs()">Drugs/BM Time!</a>
<a class="one" href="javascript:maReset(14)">LogOut
<a class="one" href="javascript:setlogout()">Time to Logout</a>
<a class="one" href="javascript:maReset(15)">Train
<a class="one" href="javascript:settrain()">Time to Train</a>
</td>
</tr>
<table>

<script type="text/javascript">
var maEarn=(0), maFly=(0), maWhack=(0), maAgg=(0), maLaunder=(0), maDrugs=(0), maCase=(0), malogout=(0), matrain=(0);
refreshID=setInterval("maRefresh()",1000);
function maInitialize() {
warning.innerHTML=maHeader;
main.style.visibility="visible";
maRefresh();
}
function maReset(mType) {
if (mType==1) { maEarn=(2.1*60); };
if (mType==2) { maFly=(90*60) };
if (mType==12) { maFly=(60*60) };
if (mType==13) { maFly=(45*60) };
if (mType==4) { maWhack=(60*60); };
if (mType==5) { maAgg=(30*60); };
if (mType==6) { maAgg=(20*60); };
if (mType==10) { maAgg=(15*60); };
if (mType==7) { maLaunder=(20*60); };
if (mType==8) { maDrugs=(60*60); };
if (mType==11) { maDrugs=(15*60); };
if (mType==9) { maCase=(3*60); };
if (mType==14) { malogout=(60*60); };
if (mType==15) { matrain=(20*60); };
}
function maRefresh() {
maEarn=maEarn-1;
maFly=maFly-1;
maWhack=maWhack-1;
maAgg=maAgg-1;
maLaunder=maLaunder-1;
maDrugs=maDrugs-1;
maCase=maCase-1;
malogout=malogout-1;
matrain=matrain-1;
var mTm=parseInt(maFly/60), mTh=parseInt(mTm/60), mTs=parseInt(maFly-(mTm*60));
mTm=parseInt(mTm-(mTh*60));
if(maFly <=0){
document.getElementById("airtim").innerHTML="Travel Time!";
} else {
document.getElementById("airtim").innerHTML = ""+mTh+"hrs"+mTm+"min,"+mTs+"sec";
}
if(maFly ==0){
alert("Travel Time!");
}
var mWm=parseInt(maWhack/60);
var mWs=parseInt(maWhack-(parseInt(maWhack/60)*60));
if(maWhack <=0){
document.getElementById("watim").innerHTML="Whack Time!";
} else {
document.getElementById("watim").innerHTML = ""+mWm+"min,"+mWs+"sec";
}
if(maWhack ==0){
alert("Whack Time!");
}
var mPm=parseInt(maEarn/60);
var mPs=parseInt(maEarn-(parseInt(maEarn/60)*60))
if(maEarn <=0){
document.getElementById("earntim").innerHTML="Earn Time!";
} else {
document.getElementById("earntim").innerHTML = ""+mPm+"min,"+mPs+"sec";
}
if(maEarn ==0){alert("Earn Time!")
}
var mFm=parseInt(maAgg/60);
var mFs=parseInt(maAgg-(parseInt(maAgg/60)*60));
if(maAgg <=0){
document.getElementById("aggtim").innerHTML="Aggravated Time!";
} else {
document.getElementById("aggtim").innerHTML = ""+mFm+"min,"+mFs+"sec";
}
if (maAgg ==0){
alert("Aggravated Time!");
}
var mLm=parseInt(maLaunder/60);
var mLs=parseInt(maLaunder-(parseInt(maLaunder/60)*60));
if(maLaunder <=0){
document.getElementById("lautim").innerHTML="Launder Time!";
} else {
document.getElementById("lautim").innerHTML = ""+mLm+"min,"+mLs+"sec";
}
if (maLaunder ==0){
alert("Launder Time!");}
var mDm=parseInt(maDrugs/60);
var mDs=parseInt(maDrugs-(parseInt(maDrugs/60)*60));
if(maDrugs <=0){
document.getElementById("drgtim").innerHTML="Drugs/BM Time!";
} else {
document.getElementById("drgtim").innerHTML = ""+mDm+"min,"+mDs+"sec";
}
if (maDrugs ==0){
alert("Time to push drugs/BM goods!");
}
var mFm=parseInt(malogout/60);
var mFs=parseInt(malogout-(parseInt(malogout/60)*60));
if(malogout <=0){
document.getElementById("logouttim").innerHTML="Time to Logout";
} else {
document.getElementById("logouttim").innerHTML=""+mFm+"min,"+mFs+"sec";
}
if (malogout ==0){alert("Time to Logout!")}
var mPm=parseInt(maCase/60);
var mPs=parseInt(maCase-(parseInt(maCase/60)*60))
if(maCase <=0){
document.getElementById("castim").innerHTML="Case Time!";
} else {
document.getElementById("castim").innerHTML = ""+mPm+"min,"+mPs+"sec";
}
if(maCase ==0){
alert("Case Time!");
}
var mLm=parseInt(matrain/60);
var mLs=parseInt(matrain-(parseInt(matrain/60)*60))
var mBm=parseInt(matrain/60);
var mBs=parseInt(matrain-(parseInt(matrain/60)*60))
if(matrain <=0){
document.getElementById("traintim").innerHTML="Train Time!";}else{
document.getElementById("traintim").innerHTML = ""+mBm+"min,"+mBs+"sec";
}
if(matrain ==0){
alert("Train Time!");}
} //close refresh
function setFlight() {
var petim=prompt("Minutes Until You Can Travel Again:",parseInt(90));
maFly=petim*60;
}
function setWhack() {
var petim=prompt("Minutes Until You Can Whack Again:",parseInt(60));
maWhack=petim*60;
}
function setEarn() {
var petim=prompt("Minutes Until You Can Earn Again:",parseInt(.20));
maEarn=petim*60;
}
function setAgg() {
var petim=prompt("Minutes Until You Can Commit An Aggravated Crime Again:",parseInt(30));
maAgg=petim*60;
}
function setLaunder() {
var petim=prompt("Minutes Until You Can Launder Again:",parseInt(20));
maLaunder=petim*60;
}
function setDrugs() {
var petim=prompt("Minutes Until You Can Push Drugs/BM Goods Again:",parseInt(60));
maDrugs=petim*60;
}
function setCase() {
var petim=prompt("Minutes Until You Can Do Another Case:",parseInt(3));
maCase=petim*60;
}
function setlogout() {
var petim=prompt("Minutes till you logout:",parseInt(60));
malogout=petim*60;
}
function settrain() {
var petim=prompt("Minutes till you can Train:",parseInt(20));
matrain=petim*60;
}
</script>
</body>

Stevesparty

08-20-2007, 02:38 AM

Still doesn't work... Could there be another problem (browser etc)?

rwedge

08-20-2007, 03:10 AM

It did run in IE until the case alert was called. It did not work for FF because the capital T in traintim.

Stevesparty

08-20-2007, 07:03 AM

It works!! Thank you very much for your help!