[SOLVED] if you have specific field name

[ted_chou12]

What if you have field name called delete or var?

document.photocomment.delete.style.visibility = ...

what can you do to prevent syntax?

Thanks,

Ted

[priti]

It's not a good practice to name variable like this

 

if its a delete button then you should name it as - btnDelete

if some Variable is there then it should be - varName

 

do u get some error if you use document.photocomment.delete.style.visibility ... something like this ?

[Adam]

It's not a good practice to name variable like this

 

if its a delete button then you should name it as - btnDelete

if some Variable is there then it should be - varName

 

Hmm that's kinda what my Visual Basic tutor taught me at college, but to be honest I don't follow it. I find it a much bigger ball ache than it's worth; meaningful variable names should avoid any confusion. As for 'delete', I don't even think it's a prototype is it? What errors do you get?

 

EDIT: I've found a 'delete operator'; but that uses a different syntax so shouldn't cause any problems...

[ted_chou12]

Hi, thanks for the advice, I am using this code in ajax, inserting this code in prevents the code to work, but im not exactly sure what the error is, can you let me know how to show the errors?

Thanks,

Ted

[Adam]

If you're using FF it has a built in error console (tools > error console) - you should be able to find something to point you in the right direction there. If not perhaps look at installing Firebug or a similar FF extension.

[Psycho]

Ok, I am in TOTAL agreement with those above - do not name elements "delete" or anything else that would cause a conflict. But, just for educational purposes, there is a workaround.

 

document.photocomment['delete'].style.visibility = ...

 

I only pose this because if you have fields named so they will be processed as arrays (i.e. name="foo[]") then you need to use this method to reference the fields.

 

var fieldsArrayObj = document.formName['foo[]'];

 

[ted_chou12]

thanks, I changed the field name to "del", the code works after that

Ted.