Почему не работает событие js кода?

Question

[Arbitr]

На html странице есть иконка(работает как кнопка), при нажатии на нее, должна вылетать форма авторизации, но выдает сообщение: Uncaught ReferenceError: avtorization(это функция) is not defined at HTMLInputElement.onclick. Я все перепробовал, но эта ошибка как появлялась, так и появляется, в чем проблема?
п.с. извиняюсь за говнокод....

Сама кнопка:

<input type="image" src="img/house.png" alt="" onclick="avtorization()">

Окно:

<div class="black_bg" name="avtoriz">
   <div class='avtoriz linkChange'>
    <p>Авторизация</p>
     <form action="index.php" method="Post"> 
      <input name="login" type="text" placeholder="Login..."><br>
      <input name="passw" type="password" placeholder="Password..."><br>
      <a href="reg.html">Регистрация</a><br>
      <input type="submit" src="../img/voit.png" name="voity" value="Войти">
     </form>
   </div>
  </div>

Функция события:

setTimeout(function avtorization() {
	var avt = document.getElementsByName('avtoriz');
	if (avt.style.display != "block") { 
        avt.style.display = "block";
    }
    else avt.style.display = "none";
},1000);

Стили:

.black_bg{
    display: none;
    background-color: rgba(0,0,0,.8);
    width: 100%;
    height: 100%;
    position:fixed;
    z-index: 99;
    margin: 0;
    top: 0
}
.avtoriz{
    width: 300px;
    height:auto;
    position: fixed;
    border: solid 2px #AEC4CA;
    border-radius: 8px;
    background: #102C3D;
    left: 40%;
    top: 40%;
    z-index: 100;
    text-align: center;
    font-size: 1.2em
}

Answer 1

[Kovalsky]

setTimeout(function avtorization() {
  var avt = document.getElementsByName('avtoriz');
  if (avt.style.display != "block") { 
        avt.style.display = "block";
    }
    else avt.style.display = "none";
},1000);

Эта конструкция выполняет именованную функцию avtorization минимум через 1000мс после собственного выполнения. Она не объявляет эту функцию в области видимости, про специфику работы function expression/declaration читайте в интернетах. Собственно, по этой причине avtorization is not defined.

К тому же обработчики гораздо удобней устанавливать с помощью element.addEventListener( event_name, handler );

Comments

[Arbitr]

я переделал, теперь ругается на свойcтво display:

Uncaught TypeError: Cannot read property 'display' of undefined
at avtorization (js.js:4)
at HTMLInputElement.onclick (header.html:21)

Теперь вот так:

function avtorization(){

	var avt = document.getElementsByName('avtoriz');
    	if (avt.style.display != 'block') { 
        	avt.style.display = 'block';
	    }
	    else {
	    	avt.style.display = 'none';
	    }
}

А в html так:

<input type="image" src="img/house.png" alt="" onclick="setTimeout(avtorization(),1000);">

[Kovalsky]

ArBitr_exe: вам уже ниже ответили, у вас в avt лежит не элемент, а массив с элементами, у которого нет свойства style.