javascript problem z polem wyboru

Jestem początkujący, a chciałem utworzyć skrypt który oblicza wymiary ekranu, takowy udało mi się utworzyć

<input type="text" id="przekatna" placeholder="przekątna" size=16 /><br>
<input type="text" id="format1" placeholder="format x" size=5 />
<input type="text" id="format2" placeholder="format y" size=5/><br>
<input type="text" id="obramowka" placeholder="grubość obramówki" size=16 /><br>
    <button onclick="vcheck()">Sprawdź</button> <br>
    <div id="liczba">Wpisz dane</div>
<script type="text/javascript">
      function vcheck()
        {
		 var val="Wpisz dane";
         var valp=document.getElementById("przekatna").value.trim();
		 var vals1=document.getElementById("format1").value.trim();
		 var vals2=document.getElementById("format2").value.trim();
		 var valo=document.getElementById("obramowka").value.trim();
         if(val.length<=0) val="Wpisz przekątną, format i obramówkę monitora";
		 else if(valo<0||vals2<=0||vals1<=0||valp<=0) val="Błędne dane";
         else if(valo>=0&&vals2>0&&vals1>0&&valp>0) {
			var p = Math.pow((valp*2.54),2);
			var s1 = vals1;
			var s2 = vals2;
			var o = valo;
			var l = Math.pow(s1,2)+Math.pow(s2,2);
			var x = Math.sqrt(p/l);
			s1=Math.round((s1*x)+(2*o));
			s2=Math.round((s2*x)+(2*o));
			var wyswietl ="≈" + s1 +"cm"+" x "+ s2+"cm";
			val=wyswietl;
		 }
         else val="Niepoprawne dane";
         document.getElementById("liczba").innerHTML=val;
        }
    </script>

lecz chciałem go "upiększyć" używając <select> w html, niestety mam problem gdyż format ekranu nie jest przypisywany do zmiennych, nie wiem jak to zrobić

<label for="przekatna">Przekątna:</label></br>
<input type="text" id="przekatna" placeholder="przekątna" size=16 /><br>
<label for="format">Wybierz Format:</label></br>
<select id="format" name="format" onchange="window['display_'+this.value]();">
	<option data-value=0 value="dis">Format</option>
	<option data-value=1 value="dis" >1:1</option>
	<option data-value=2 value="dis" >3:2</option>
	<option data-value=3 value="dis" >4:3</option>
	<option data-value=4 value="dis" >5:4</option>
	<option data-value=5 value="dis" >16:9</option>
	<option data-value=6 value="dis" >16:10</option>
	<option data-value=7 value="dis" >17:9</option>
	<option data-value=8 value="dis" >21:9</option>
	<option data-value=9 value="dis" >32:9</option>
    <option data-value=10 value="en" >inny</option>
</select></br>
<input type="text" id="format1" placeholder="format x" size=5 disabled />
<input type="text" id="format2" placeholder="format y" size=5 disabled /><br>
<label for="obramowka">Grubość obramówki:</label></br>
<input type="text" id="obramowka" placeholder="grubość obramówki" size=16 /><br>
    <button onclick="vcheck()">Sprawdź</button> <br>
    <div id="liczba">Wpisz dane</div>
<script type="text/javascript">
      function vcheck(w)
        {
		 var val="Wpisz dane";
         var valp=document.getElementById("przekatna").value;
		 var valo=document.getElementById("obramowka").value;
         var vals1;
         var vals2;
		 var wartosc=document.getElementById("format").getAttribute('data-value');
		 switch(wartosc) {
		case 0:
    val="Wybierz format";
		break;
		case 1:
    vals1=1;
    vals2=1;
		break;
		case 2:
    vals1=3;
    vals2=2;
		break;
		case 3:
    vals1=4;
    vals2=3;
	break;
		case 4:
    vals1=5;
    vals2=4;
	break;
		case 5:
    vals1=16;
    vals2=9;
	break;
		case 6:
    vals1=16;
    vals2=10;
	break;
		case 7:
    vals1=17;
    vals2=9;
	break;
		case 8:
    vals1=21;
    vals2=9;
	break;
		case 9:
    vals1=32;
    vals2=9;
	break;
		case 10:
    vals1=document.getElementById("format1").value.trim();
	vals2=document.getElementById("format2").value.trim();
	}
         if(val.length=0) val="Wpisz przekątną, format i obramówkę monitora";
		 else if(valo<0||vals2<=0||vals1<=0||valp<=0) val="Błędne dane";
         else if(valo>=0&&vals2>0&&vals1>0&&valp>0) {
			var p = Math.pow((valp*2.54),2);
			var s1 = vals1;
			var s2 = vals2;
			var o = valo;
			var l = Math.pow(s1,2)+Math.pow(s2,2);
			var x = Math.sqrt(p/l);
			s1=Math.round((s1*x)+(2*o));
			s2=Math.round((s2*x)+(2*o));
			var wyswietl ="≈" + s1 +"cm"+" x "+ s2+"cm";
			val=wyswietl;
		 }
         else val="Niepoprawne dane";
         document.getElementById("liczba").innerHTML=val;
        }
		function display_en() {
            document.getElementById("format1").disabled = false;
			document.getElementById("format2").disabled = false;
        }
        function display_dis() {
            document.getElementById("format1").disabled = true;
			document.getElementById("format2").disabled = true;
        }
    </script>

Wrzuć to proszę na http://jsfiddle.net i podeślij nam link do swojej wersji.

https://jsfiddle.net/fydeo3gs/
prosze

działający bez listy:
https://jsfiddle.net/agboft2r/

Trochę poprawiłem - https://jsfiddle.net/c5t2b3rv/
Do osadzania JS oraz CSS są osobne okienka, w ten sposób jest większy porządek.

A co do samego skryptu - jakbyś napisał w 2-3 zdaniach co on ma dokładnie robić, bo nieważne co wpisuję, zawsze dostaję informację "błędne dane".
Wyjaśnij proszę co chcesz osiągnąć oraz jak ten skrypt byś chciał, żeby działał.

chcę aby zamiast wpisywania za każdym razem proporcji ekranu, te najpowszechniejsze były zawarte w liście, np 16:9, po wybraniu tej opcji vols1 przyjmowałoby 16 a vols2 9, chce zachować możliwość użycia "inne" aby móc wpisywać inne formaty jak 18:9 (dla telefonu). wersja bez listy działa bez problemu

Pytanie - czy chcesz to po prostu mieć zrobione, czy chcesz wspólnie dojść do tego, dlaczego to nie działa? :P

Jeśli czekasz na gotowca to daj znać, ogarniemy. Ale jeśli chcesz (na co szczerze mówiąc liczę) sam pokombinować i się przy okazji, z naszą pomocą, czegoś nauczyć, to pierwsza podpowiedź: olewamy listę z wyborem zdefiniowanych typów/rozmiarów. Wybieramy "inny" i ręcznie wpisujemy proporcje. Teraz to zasadniczo mamy praktycznie to samo, co we wcześniejszym wariancie, czyli bez listy, wartości wprowadzone "z łapy". Zastanów się, dlaczego to nie działa oraz jak możesz sprawdzić, czy wartości są odpowiednio przesyłane.

Druga rada - jakbyś zmienił nazwy zmiennych na bardziej opisowe, to o wiele łatwiej by się przeglądało kod. Raczej mało kto (poza Tobą) patrząc na fragment

if(valo>=0&&vals2>0&&vals1>0&&valp>0)

domyśli się, co ten kod robi oraz co poszczególne zmienne oznaczają. A za miesiąc albo dwa, Ty także nie będziesz miał zielonego pojęcia, więc żeby cokolwiek zmienić, będziesz musiał (tak, jak my teraz) szukać w kodzie, w którym miejscu do tych zmiennych zostały przypisane wartości i na tej podstawie ustalać, czym dana zmienna jest.

Ja chciałbym to zrobić w ramach nauki :) . Jutro do tego siądę i się zastanowie oraz pozmieniane nazwy zmiennych na bardziej zrozumiałe.

Zamykam temat
problemem był błąd pobierania danych z data-value którego nie potrafiłem rozwiązać, więc użyłem ponownie value aby w trochę inny sposób zrobić 2 funkcję, która używała value wcześniej i teraz obie funkcje używają tej samej zmiennej dzięki mocy if-ów :D

Działa:
https://jsfiddle.net/9huqtne8/

A napisz co to był za błąd, którego nie umiałeś rozwiązać. Może ktoś coś wymyśli :)

var wartosc=document.getElementById("format").getAttribute('data-value');

Nie dawało zapisanych wartości 1,2,3...10 lecz zwracało jakiś tekst (teraz nie pamiętam jaki), niezależnie od wybranej opcji taki sam