So I'm scripting a blackjack game but loooking for some ways to make it look better. Ideas? Im using java scrpit <head> <!-- THREE STEPS TO INSTALL BLACKJACK: 1. Copy the coding into the HEAD of your HTML document 2. Add the onLoad event handler into the BODY tag 3. Put the last coding into the BODY of your HTML document --> <!-- STEP ONE: Paste this code into the HEAD of your HTML document --> <SCRIPT LANGUAGE="JavaScript"> --> <!-- Begin var gameOver; var cardCount; function Shuffle(max){ var num=Math.random()*max; return Math.round(num)+1; } function getSuit(){ suit = Shuffle(4); if(suit == 1) return "Spades"; if(suit == 2) return "Clubs"; if(suit == 3) return "Diamonds"; else return "Hearts"; } function cardName(card){ if(card == 1) return "Ace"; if(card == 11) return "Jack"; if(card == 12) return "Queen"; if(card == 13) return "King"; return "" + card; } function cardValue(card,strWho){ if(card == 1) { if(strWho =="You" && document.display.you.value >10){ document.display.say2.value=document.display.say2.value+" Low"; return 1;} else return 11; } if(card > 10) return 10; return card; } function PickACard(strWho){ card = Shuffle(12); suit = getSuit(); if(strWho =="You") document.display.say2.value=(cardName(card) + " of " + suit); else document.display.say1.value=(cardName(card) + " of " + suit); return cardValue(card,strWho); } function NewHand(form){ if(gameOver !=0) {form.say1.value=("Hand in Play!"); form.say2.value=(""); return;} else {form.dealer.value = 0; form.you.value = 0; cardCount=0; form.dealer.value = eval(form.dealer.value) + PickACard("Dealer"); form.you.value = eval(form.you.value) + PickACard("You"); gameOver= -1; cardCount+=1;} } function Dealer(form){ if (gameOver ==0) {form.say1.value=("Deal the Cards!"); form.say2.value=(""); return;} else if(form.you.value<10) {form.say1.value=("Not Below Ten!"); form.say2.value=("Take a Hit!"); return;} else if (cardCount <2) {form.say1.value=("Minimum 2 Cards!"); form.say2.value=("Hit Again!"); return;} else while(form.dealer.value < 17) {form.dealer.value = eval(form.dealer.value) + PickACard("Dealer");} } function User(form){ if (gameOver ==0) {form.say1.value=("Deal the Cards!"); form.say2.value=(""); return;} else {cardCount+=1; form.say1.value=("You Get..."); form.you.value = eval(form.you.value) + PickACard("You");} if(form.you.value > 21) {form.say1.value=("You Busted!"); gameOver=0; form.numgames.value=eval(form.numgames.value)-1;} } function LookAtHands(form){ if (gameOver ==0 || form.you.value<10 || cardCount <2){return;} else if(form.dealer.value > 21) {form.say1.value=("House Busts!"); form.say2.value=("You Win! $$$$$$"); gameOver=0; form.numgames.value=eval(form.numgames.value)+1;} else if(form.you.value > form.dealer.value) {form.say1.value=("You Win!"); form.say2.value=("$$$$$$$$$$$$$$$"); gameOver=0; form.numgames.value=eval(form.numgames.value)+1;} else if(form.dealer.value == form.you.value) {form.say1.value=("Game Tied!"); form.say2.value=("Try Again!"); gameOver=0; form.numgames.value=eval(form.numgames.value)-1;} else {form.say1.value=("House Wins!"); form.say2.value=("Tough Luck!"); gameOver=0; form.numgames.value=eval(form.numgames.value)-1;} } function setBj(){ gameOver=0; cardCount=0; document.display.dealer.value=""; document.display.you.value=""; document.display.numgames.value="0"; document.display.say1.value=" Hit 'Deal'"; document.display.say2.value=" To Start!"; } // End --> </script>
I suggest you would make more enters. It's kinda hard to read. Make the code easy to read as the game easy to pick up & play.
