jQuery: Each Loop Oddity

Question

I want to go through all fields of a form and determine if the fields are populated with data. When all fields are given, I'd like a button save to appear.

This is the Fiddle with code, that works (!) ...

This is the same in JavaScript.

The jQuery-Object to iterate through:

formElements = $ 'form input[type="text"], form input[type="number"], form textarea'

The Function:

formFilled = ->
  filled = true
  formElements.each ->
    if $(this).val().length is 0
      filled = false
  filled

The Event-Handler:

formElements.on 'keyup', (keyup) ->
  keyup.preventDefault()
  if formFilled()
    save.show()
  else
    save.hide()

Any suggestions?

the save button save never shows up (is hidden all the time) ... — Timo
– Timo, Commented Nov 15, 2016 at 0:00
There's nothing that's asynchronous here. Does your keyup handler get called at all? Does formElements actually have anything in it? — mu is too short
– mu is too short, Commented Nov 15, 2016 at 2:35
@Timo I see you've thrown a lot of questions up in the last 24 hours on this. Could you create a [JS fiddle|jsfiddle.net/] including your coffeescript and html? We really need to see the whole code to figure this out. And you can insert coffeescript into the fiddle by clicking on the javascript text in the bottom left text area — caffeinated.tech
– caffeinated.tech, Commented Nov 15, 2016 at 9:22

Timo · Accepted Answer · 2016-11-15 11:58:59Z

1

# I know the answer:

JavaScript validates the number-input fields and accepts only values that are numbers.

I stupidly tested those fields with text input ... silly.

The code is fine. Just the testing was bad. Sorry for wasting your time.

This works:

formInputs = $ 'input[type="text"], input[type="number"], textarea'

formFilled = ->
  filled = true
  formInputs.each ->
    if $(this).val().length is 0
      filled = false
  filled

formInputs.on 'keyup', (keyup) ->
  if formFilled()
    saveButton.show()
  else
    saveButton.hide()

answered Nov 15, 2016 at 11:58

Timo

2615 silver badges19 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

jQuery: Each Loop Oddity

This is the Fiddle with code, that works (!) ...

This is the same in JavaScript.

1 Answer 1

# I know the answer:

This works:

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

This is the Fiddle with code, that works (!) ...

This is the same in JavaScript.

1 Answer 1

# I know the answer:

This works:

Comments

Your Answer

Sign up or log in

Post as a guest

Related