Můj první kód (JavaScript) - Kontrola – JavaScript, AJAX, jQuery – Fórum – Programujte.com

Čaute, včera sem poprvé otevřel nějakej kurz pro JavaScript s tím že nějako musím využít volnej čas. Dokončil jsem nějakou druhou lekci a dostal jsem úkol vytvořit hru "Kámen, nůžky, papír". Absolutně jsem netušil jak na to, tak pomocí pak funkcí z googlu jsem to dal dokupy. Zdá se že to funguje jak má, ale byl bych moc rád, kdyby mi někdo zkontroloval takové ty "nováčkovské chyby" aneb čemu se mám příště vyhnout. Moc dík komukoliv kdo na to koukne.

var kladnyVysledek = function(prvni, druhe)
{
	document.write("Vyhral jsi, vybral sis " + prvni + " a on " + druhe);	
};
var zapornyVysledek = function(prvni, druhe)
{
	document.write("Prohral jsi, vybral sis " + prvni + " a on " + druhe);	
};
KeywordArray = new Array(3);  
KeywordArray[0] = "kamen";  
KeywordArray[1] = "papir";  
KeywordArray[2] = "nuzky";   
randno = Math.floor ( Math.random() * KeywordArray.length ); 
var game = function(you, him)
{
	if (you !== "kamen" && you !== "papir" && you !== "nuzky")
	{
		document.write("Neplatny predmet, zkus to znovu.");
		
	}
	if (you == him)
	{
		document.write("Remiza, oba jste si vybrali stejny predmet");
	}
	else
	{
		if (you == "kamen")
		{
			if (him == "papir")
			{
				zapornyVysledek(you, him);
			}
			else
			{
				kladnyVysledek(you, him);
			}
		}
		else
		{
			if (you == "papir")
			{
				if (him == "kamen")
				{
					kladnyVysledek(you, him);
				}
				else
				{
					zapornyVysledek(you, him);
				}
			}
			else
			{
				if (you == "nuzky")
				{
					if (him == "papir")
					{
						kladnyVysledek(you, him);
					}
					else
					{
						zapornyVysledek(you, him);
					}
				}
			}
		}
	}
};	
game(prompt("Co si vyberes? (kamen, nuzky, papir - bez diakritiky)"), KeywordArray[randno]);

(Pro test funkčnosti jsem použil http://writecodeonline.com/javascript/ )

Dulezite je umet ve FF pouzivat chybovou konzolu, ktera ti chyby v zapise ukazuje, pripadne editor, ktery obarvuje syntaxi.

var game = function(you, him)

function game (you, him) // vetsinou se to zapisuje takto

KeywordArray = new Array(3);

KeywordArray = []; // usporne se to da zapsat takto

Kladny a zaporny vysledek se da preci napsat do jedne funkce, ne?

var vysledek = function(prvni, druhe)
a = "Vyhral jsi, vybral sis " + prvni + " a on " + druhe;
b = "Prohral jsi, vybral sis " + prvni + " a on " + druhe;
c = "Remiza, oba jste si vybrali stejny predmet";
bool = (you == "kamen" && him == "nuzky") || (you == "nuzky" && him == "papir") || (you == "papir" && him == "kamen") ? 0 : 1; // vyhra nebo prohra + remiza
bool = bool === 1 && you === him ? 2 : bool; // vyhra / prohra / remiza
document.write(bool===0 ? str1 : bool===1 ? str2 : str3);
};
vysledek(you, him);

A samozrejme to chce zkontrolovat, zda tam uzivatel nedal treba "kvetinac", ale to si tam uz prepises. Pohlidal bych si ale, aby vsechny podminky byli postavene stejne, pokud to jde. Cili, jak mas nahore prvni zaporny vysledek a u ostatnich kladny, tak pro moji velkou podminku, kladneho vysledku pro vsechny tri situace, jsem to musel prepsat.

A kdy prejdes na onclick <b onclick="funkce();">aaa</b> a getElementById() a innerHTML? Document.write je takove krkolomne z hlediska dalsiho pouziti.

Moc dík kouknu na to. Jinak když napíše květináč tak mu to hodí "Neplatný předmět, zkus to znovu".