Nie działa else

pytanie zadane 8 lutego 2023 w JavaScript przez niezalogowany

<!DOCTYPE html>
 <html>
 <head>
 <meta charset="utf-8">

</head>

<body>
<input type="number"id = "num1"value = "1">

<input type="number"id = "num2"value = "1">


<button onclick = "my()"></button>
<p id = "result1"></p>


<p id = "result2"></p>

<script>
	function my()

	{

const num1 = document.getElementById("num2").value.toString();

const num2 = document.getElementById("num2").value.toString();

const num1_ = num1[0];

const num2_ = num2[0];
const num = [num1,num2];

numbers = {a:num1_,b:num2_};

let y = 0;
for (let x in numbers,y++)
{
	if (numbers[x] == "-")
		{
			document.getElementById("result1").innerHTML = "Liczba " + num[y] + " jest liczba ujemna";



		}
		
	else

			{

				document.getElementById("result2").innerHTML = "Liczba " + num[y] + " jest liczba dodatnia";

			}


	
}
	}


</script>
  
</body>
</html>

Szukam i nie mogę znaleźć błędu przez który instrukcja else nie jest wykonywana.

komentarz 8 lutego 2023 przez adrian17 Ekspert (345,160 p.)

for (let x in numbers,y++)

To wygląda mega dziwnie i na 100% robi co innego niż miałeś na myśli.

Wypisz sobie też wartości tych wszystkich zmiennych num1_, num2_, numbers etc - bo też dziwnie nimi żonglujesz.

Nie wiem też czemu masz słownik i for..in, jak by wystarczyło zwykłe for..of na tablicy.

komentarz 8 lutego 2023 przez niezalogowany

Nie wiedziałem że jest coś takiego jak for of

2 odpowiedzi

odpowiedź 8 lutego 2023 przez radek024 Szeryf (77,160 p.)

Hej,

utworzyłem snippet na codepenie z Twoim problemem. Zakładam, że kod który napisałeś ma za zadanie sprawdzić, czy liczba jest ujemna, czy też dodatnia i robi to w następujący sposób:

Po kliknięciu uruchom funkcję my().
Pobierz wartości z formularza
Przejdź po wartościach
Jeżeli liczba jest ujemna - daj infomację do results1.
W przeciwnym razie do results2.

Rzeczy, które nie do końca wyglądają okej:

Dlaczego bierzesz wartość poprzez toString()?
Czy faktycznie potrzebujesz zmiennych num1_ i num2_? Czemu one służą?
Przypisujesz wartość numbers, ale nigdzie nie definiujesz takiego obiektu.
To co się dzieje w pęli też wygląda dość niejasno. Po co tam y?

A co gdyby:

Pobierać wartość inputów, a nie tekst z pola?
Zdefiniować obiekt i dodać elementy w prawidłowy sposób?
Przejść po obiekcie i sprawdzić wartość na podstawie warunku większy/mniejszy od zera?
Podać właściwą wartość.

Trudno dokładnie powiedzieć jaki element JSa chcesz zbadać, ale sporo rzeczy możesz poprawić. Spróbuj przejść przez każde z zadanych pytań i napisz kod od nowa.

komentarz 8 lutego 2023 przez niezalogowany

Dzięki za radę.

odpowiedź 8 lutego 2023 przez VBService Ekspert (253,520 p.)
edycja 8 lutego 2023 przez VBService

Dla wskazanego kodu np. nie prościej będzie

[ on-line ]

function my() {
  const num1 = document.getElementById("num1").value,
        num2 = document.getElementById("num2").value;

  const res1 = (num1 < 0) ? 'ujemna':'dodatnia';
  document.getElementById("result1").textContent =
    "Liczba " + num1 + " jest " + res1;
  const res2 = (num2 < 0) ? 'ujemna':'dodatnia';
  document.getElementById("result2").textContent =
    "Liczba " + num2 + " jest " + res2;
}

lub nawet

[ on-line ]

function my() {
  const numbers = document.querySelectorAll('[id^="num"]'),
        result = document.getElementById('result');

  result.textContent = '';  
  for (number of numbers)
    result.textContent += 'Liczba ' + number.value + 
      ' jest ' + ((number.value < 0) ? 'ujemna':'dodatnia') + '\n';
}