2013-06-05 2 views
0

Когда я нажимаю кнопку генерации, она отображается неопределенно, она должна получить уникальный идентификационный номер. от guidGenerator. Что-то не так с функцией generateID, где guidGenerator, но я не уверен, что туда положить. Мне нужно значение, которое возвращает guidGenerator. Если я сам document.write (guidGenerator), он отобразит код, но не в текстовом поле.Случайное числовое значение не отображается в текстовом поле

function guidGenerator() { 
     var S4 = function() { 
     return (((1+Math.random())*0x10000)|0).toString(16).substring(1); 
     }; 
     return (S4()+S4()+S4()); 
    } 

    function generateID(guidGenerator) { 
     var TheTextBox = document.getElementById("generateidtxt"); 
     TheTextBox.value = TheTextBox.value + guidGenerator; 
    } 

enter image description here

+0

Как вы называете функцию 'generateID'? Прямо сейчас у вас есть параметр 'guidGenerator', поэтому, когда вы используете этот идентификатор внутри функции, он будет использовать значение параметра, а не функцию' guidGenerator', которую вы определили ранее. – Guffa

+0

Покажите, как вы вызываете 'generateID' –

ответ

1

Вы должны вызвать функцию

TheTextBox.value = TheTextBox.value + guidGenerator(); // added parentheses 

EDIT

Согласовано с @Ian, что это не единственная проблема. Я могу сказать больше, когда вижу, как вы вызываете generateID, но, возможно, вы хотите удалить параметр функции и вызвать guidGenerator, как показано выше.

+0

Не работает. Параметр называется 'guidGenerator', который, вероятно,' undefined' и ничего не делает – Ian

+0

@ Возможно, ему просто нужно удалить аргумент 'guidGenerator'. – basilikum

+0

Я забыл поставить круглую скобку ... глупая ошибка .. спасибо! – Cloud

Смежные вопросы