Почему в переменную не приходит значение функции?

Question

[Sizoider]

всем привет.
что то я не могу понять почему в переменную не приходит значение из функции
ребят помогите

<form>
  Band name <input type="text" name="name" id="bandname"><br>
  <input type="submit" value="Submit" onclick="submitValue();">
</form>

function submitValue(){
  var startStr = 'http://api.bandsintown.com/artists/';
  var endStr = '/events.json?app_id=sizoider';
  var a = document.getElementById("bandname").value;
  a = a.replace(/\s/g, "%20");
  var query = startStr + a + endStr;
  return query;
}
var val = submitValue();
console.log(val);

Answer 1

[Алексей]

А что вы собственно хотите сделать?
В какую переменную вы хотите чтобы "пришли" значения из функции?
Вы никуда не записываете результат работы функции.

Ваш код сейчас работает так.
При загрузке страницы в консоль выводится результат функции при пустом поле ввода - "api.bandsintown.com/artists//events.json?app_id=si..."
При нажатии на кнопку, происходит вызов функции, который в общем-то ничего не меняет, и сразу за этим идет сабмит формы методом GET по текущему адресу, что перезагружает страницу. При этом, разумеется, очищается поле.

Comments

[Sizoider]

мне нужно сначала получить значение из поля ввода(при клике на кнопку)
его вклеить между startStr и endStr
и чтобы значение переменной var val была эта строка

[Алексей]

Попробуйте так - codepen.io/anon/pen/WwJOBE?editors=1111

[Sizoider]

Алексей:
спасибо!
а как мне теперь значение запихнуть в переменную и чтобы к неи был доступ отовсюду???

[Алексей]

Оно и так помещается в глобальную переменную val и доступно отовсюду.
Однако есть ощущение что вы пытаетесь получить ее значение еще до того как кликнули на кнопку.

[Sizoider]

отбой!я разобрался

Answer 2

[Alastor]

мой код работает

function submitValue(){
  var url = 'http://api.bandsintown.com/artists/{{param}}/events.json?app_id=sizoider';
  return url.replace('{{param}}', encodeURIComponent(document.getElementById("bandname").value));
}

попробуй обернуть в setTimeout(function() { }, 3000); код, видимо HTML не успевает рисоваться или на body событие onLoad повесить, после которого запускать код

Comments

[Sizoider]

plnkr.co/edit/esFHWkhDUhYl4HVyQlaY?p=streamer
вот ваш .он не работает

[Alastor]

ну конечно не работает, его там нет

[Sizoider]

извиняюсь,я не ту ссылку вставил

[Alastor]

plnkr.co/edit/rOU8o0Z6yfJ99LflYFmC?p=preview я же писал, HTML не успевает рисоваться

[Sizoider]

Alastor:
мне уже самому интересно стало сделать те варианты которые вы предложили.
вот посмотрите,по вашему совету сделал немного.теперь не знаю как значение запихнуть в переменную и чтобы к неи был доступ отовсюду???
output.jsbin.com/bamizeh