我正试图为Palm创建一个应用程序。这是一个祈祷应用程序,更具体地说是一个念珠应用程序。如果你不是天主教徒,念珠就像一条项链,你可以在项链上数珠子。
例行公事如下:
所有的想法都很感激。
<script type="text/javascript">
var chaplet=new Array("The Joyful Mysteries","The Luminous Mysteries","The
Sorrowful Mysteries","The Glorious Mysteries");
var mystery=new Array();
mystery[0]=new Array("Annunc","Visit","The Nativ","Present","Temple");
mystery[1]=new Array("Baptism","Wedding", "Kingdom","Transfig","Eucharist");
mystery[2]=new Array("Agony","Scourging","Crowning","Carrying", "Crucif");
mystery[3]=new Array("Resurr","Ascension","Spirit","Assumption","Coronation");
var creed="I believe..."
var ourFather="Hallowed be"
var hailMary="Blessed are thou"
var gloryBe="As it was"
var fatima="Oh my Jesus"
var hailHolyQueen="To thee do we cry"
var dec=0;
var beadCount=0;
var mys=0;
var bead=0;
function nextBead()
{
eventSrcID=(event.srcElement)?event.srcElement.id:'undefined';
if (eventSrcID=='joy') mys=0;
if (eventSrcID=='lum') mys=1;
if (eventSrcID=='sor') mys=2;
if (eventSrcID=='glo') mys=3;
if (beadCount==0)
{
div1.innerHTML="Apostles Creed";
div2.innerHTML="";
div3.innerHTML="<p>"+creed+"</p>";
div4.innerHTML="Cross";
}
else if (beadCount==1)
{
div1.innerHTML="Our Father";
div2.innerHTML="";
div3.innerHTML="<p>"+ourFather+"</p>";
div4.innerHTML="Decade 0 Bead "+beadCount;
};
else if (beadCount<=4)
{
div1.innerHTML="For Faith, Hope and Charity";
div2.innerHTML="";
div3.innerHTML="<p>"+hailMary+"</p>";
div4.innerHTML="Decade 0 Bead "+beadCount;
};
else if (beadCount==5)
{
div1.innerHTML="Glory Be";
div2.innerHTML="";
div3.innerHTML="<p>"+gloryBe+"</p>";
div4.innerHTML="Decade 0 Bead "+beadCount;
};
else if (beadCount==6)
{
div1.innerHTML="Fatima Prayer";
div3.innerHTML="<p>"+fatima+"</p>";
div4.innerHTML="Decade 0 Bead "+(beadCount-1);
};
else if (beadCount==72)
{
div1.innerHTML="Hail Holy Queen";
div2.innerHTML="";
div3.innerHTML="<p>"+hailHolyQueen+"</p>";
div4.innerHTML="Medal";
};
else if (beadCount>6 && beadCount<72)
{
div1.innerHTML="<p>"+chaplet[mys]+"</p>";
div2.innerHTML="<p>"+mystery[mys][dec]+"</p>";
if (bead==0)
{
div3.innerHTML="<p>"+ourFather+"</p>";
div4.innerHTML="<p>Decade "+(dec+1)+" Bead "+bead+"</p>";
};
else if (bead<=10)
{
div3.innerHTML="<p>"+hailMary+"</p>";
div4.innerHTML="<p>Decade "+(dec+1)+" Bead "+bead+"</p>";
};
else if (bead==11)
{
div3.innerHTML="<p>"+gloryBe+"</p>";
div4.innerHTML="<p>Decade "+(dec+1)+" Bead "+bead+"</p>";
};
else if (bead==12)
{
div3.innerHTML="<p>"+fatima+"</p>";
div4.innerHTML="<p>Decade "+(dec+1)+" Bead "+(bead-1)+"</p>";
};
bead++;
if (bead==13) bead=0;
if (bead==0) dec=dec+1;
};
beadCount++;
div5.innerHTML="<input type='button' value='Next Bead' onclick='nextBead()' />";
};
</script>
</head>
<body>
<div id="div1">
<input type="button" id="joy" value="The Joyful Mysteries" onclick="nextBead
()" /><br />
</div>
<div id="div2">
<input type="button" id="lum" value="The Luminous Mysteries" onclick="nextBead
()" /><br />
</div>
<div id="div3">
<input type="button" id="sor" value="The Sorrowful Mysteries" onclick="nextBead
()" /><br />
</div>
<div id="div4">
<input type="button" id="glo" value="The Glorious Mysteries" onclick="nextBead
()" /><br />
</div>
<div id="div5">
</div>
</body> 发布于 2011-06-10 09:35:09
IMHO代码确实需要大量的工作。
div1、div2等对于ids来说是非常糟糕的选择。用有意义的名字。<p>元素。把它们排除在外,或者用它们代替divs直接在你的HTML中使用。如果使用它们来更改元素之间的距离,则使用样式表代替。有更多的建议,但你应该先研究这些问题,然后发布一个新的问题,当你这样做了。
顺便说一句,完全是另外一回事:你真的运行了你的代码吗?我刚刚尝试了第一次,它有几个语法错误(额外的;之前的else)。
此外,您似乎使用了事件模型。我对Palm并不熟悉,但我怀疑它使用IE模型,我相信使用标准DOM模型会更好。不要使用IE进行测试,否则你会发现它是不规范的行为。
发布于 2011-06-09 21:32:29
您可能需要考虑将一些变量索引到数组中,数组的内容是具有字符串的附加数组,然后您将设置这些字符串。
https://codereview.stackexchange.com/questions/2885
复制相似问题