Форум PHP программистов > Взаимное скрытие и отображение блоков

[ Поиск ] - [ Пользователи ] - [ Календарь ]

Полная Версия: Взаимное скрытие и отображение блоков

djar

3.05.2011 - 05:16

Это мой первый скрипт на js, хочу сделать переключение между формами для поиска.


var type = '1';
        function change_search_type()
                 {
                 if (type = '1')
                    {
                    document.getElementById('kvartira').style.display = 'none';
                    document.getElementById('dom').style.display = 'block';
                    type = '2';
                    }
                 else
                    {
                    document.getElementById('dom').style.display = 'none';
                    document.getElementById('kvartira').style.display = 'block';
                    type = '1';
                    }
                 }

Чтобы можно было нажать на "Дом", и появилась другая формочка, потом нажимаем на "Квартира" - и появляется прежняя.
Вызываю функцию

<a href="#" onClick="change_search_type()">Дом</a>
...
<a href="#" onClick="change_search_type()">Квартира</a>

Первое переключение - на Дом - срабатывает, обратно уже не переключается.
Почему?

Спустя 2 часа, 42 минуты, 53 секунды (3.05.2011 - 06:59) ИНСИ написал(а):

Цитата

if (type = '1')
{
document.getElementById('kvartira').style.display = 'none';
document.getElementById('dom').style.display = 'block';
type = '2';
}

Ты сам присваиваешь type = 1. Надо так:

var type = '1';
        function change_search_type()
                 {
                 if (type == '1')
                    {
                    document.getElementById('kvartira').style.display = 'none';
                    document.getElementById('dom').style.display = 'block';
                    type = '2';
                    }
                 else
                    {
                    document.getElementById('dom').style.display = 'none';
                    document.getElementById('kvartira').style.display = 'block';
                    type = '1';
                    }
                 }

Спустя 8 минут, 55 секунд (3.05.2011 - 07:08) nugle написал(а):

velbox
зачем весь код переписывать из-за одного = ?
djar
Такого рода ошибки часто встречаются, будьте внимателнее

if (type == '1')

Спустя 14 минут, 29 секунд (3.05.2011 - 07:22) ИНСИ написал(а):

Цитата

velbox
зачем весь код переписывать из-за одного = ?

Я не переписывал

Просто выделил, нажал на кнопку и вставил еще один знак

Спустя 12 минут, 7 секунд (3.05.2011 - 07:34) nugle написал(а):

а ну тогда ладно, я и имел ввиду копи паст!просто я различия 5 минут искал, не вникая в код

Спустя 6 минут, 41 секунда (3.05.2011 - 07:41) djar написал(а):

:) спасибо.
Я поначалу проверял условие

if (document.getElementById('kvartira').style.display == 'none')

Но почему-то не работало..наверное где-то ещё косяки были.

Быстрый ответ:

Здесь расположена полная версия этой страницы.