Använda en array istället...

Hej!
Jag har skapat ett enkelt login-skript som funkar perfekt men då kan jag inte använda fler användare och lösenord. Men är det någon som kan det här med array och kunna använda if tillsammans. Koden jag använder är:

var namn="foo";
var pass="foo";

function check(){
if (document.mama.user.value == namn && document.mama.pass.value == pass) {
alert("Var god vänta...")
window.location =("index.html");
}
}

Hejsan! :)

Du är medveten om att anv.namn och lösenord är fullt synliga för besökarna?

var a = [
  // ["anv.namn","lösenord"]
  ["a","b"],
  ["c","d"],
  ["e","f"]
];

function check(){
  for (var i = 0; i < a.length; ++i){
    if (a[i][0] == document.mama.user.value && a[i][1] == document.mama.pass.value){
      alert("...");
      location.href = "index.html";
    }
  }
}

Jo, jag ska sätta något klick-skydd så att man inte kan högerklicka på sidan.

fattyman skrev:

Jo, jag ska sätta något klick-skydd så att man inte kan högerklicka på sidan.

Det kommer att hjälpa föga. ;) I flera webbläsare kan man stänga av den möjligheten för skripts.

Eller lite smartare:

var user = [];
user["a"] = "b";
user["c"] = "d";
user["e"] = "f";

function check()
{
	if (user[document.mama.user.value] != null && user[document.mama.user.value] == document.mama.pass.value)
	{
		alert("...");
		location.href = "index.html";
	}
}

Inte säkrare, men snyggare.

Erik Juhlin skrev:

Inte säkrare, men snyggare.

Potejto, potato ;)

Tomajto, potato! :p

Javascript Säkerhet FTW.

Erik Juhlin skrev:

Tomajto, potato! :p

Jag är härmed övertygad!

Jag kommer nog bara använda den privat, som en mindre sida, för det finns inget särskilt intressant på min sida, nu iallafall.
Men tack för hjälpen! Den hjälpte väldigt mycket! :)

Glöm inte att använda funktionen acceptera som slutgiltigt svar på det inlägg som löste ditt problem. :)