327
ManuLG
7y

Quality code I made, as always

Comments
  • 14
    I usually do these things too. They're nice because you get the structure of the code done but have to fill in the blanks. Remember, a TODO without any code is much better than no TODO at all.
  • 9
    It's a good approach for fast development for other parts, but I'd really really recomment to at least add a warning message (in most projects they use "NotImplementedExceptions") like this:

    console.warn( "validateForm() not implemented" );

    It really helps to not forget about implementing it. 😉
  • 2
    Wouldn't it be better to start method, which return a bool, with 'is'?
  • 1
    That's called "Getting to green."
  • 0
    Why not skip this then and it when you're ready? Would make for nice commits and there is chance you're going to change naming/calls/signatures when you implement validation functionality anyway.
  • 1
    If you're doing EDD, couldn't you do this to test other components before validation is ready?
  • 1
    form.checkValidity() ftw 😋
  • 0
    Well, you can't go wrong with that 😂
Add Comment