1. Nie ma sensu marnować Twojego cennego czasu na deklaracje tablicy w ten sposób:
var k = new Array (10); //operator new jest tu zbędny, a dla przejrzystości [] w tablicach a nie ()
k[0] = "Polska";
k[1] = "Rosja";
k[2] = "Ukraina";
k[3] = "Niemcy";
k[4] = "Wielka Brytania";
k[5] = "Macedonia";
k[6] = "Białoruś";
k[7] = "Słowacja";
k[8] = "Czechy";
k[9] = "Węgry";
Skoro można to zrobić o tak:
var k = ["a","b","c","..."]
2. Nazwy zmiennych
Coś co wykładowca na studiach tłukł mi do głowy, a ja nie wiedziałem czemu... Jak zaczniesz pracować jako web developer i dostaniesz kod z nazwami obiektów typu "k"..., albo"l"..., albo napisane w jakimś dziwnym języku to idzie się zdenerwować, uwierz mi ;) Podstawy są bardzo ważne do nie popełniania błędów w przyszłości - nazwy jak i cały kod po angielsku, a nazwy zmiennych coś mają Ci mówić jak pierwszy raz na kod popatrzysz ;)
3. Mniej istotne
var haslo = "";
var dlugosc ="";
// "var password;" bez " = '' " robi to samo, a zaoszczędzasz czas, ale to mikro optymalizacja
wskazówka: albo wstawiaj białe znaki przed i po znakach matematycznych typu " = " albo nie, bo to też niszczy przejrzystość kodu
Generalnie, to pokazany wycinek kodu prezentuje się ładnie :)