TypeError: Cannot read property 'getValue' of unde...
# suitescriptr
razer456
10/04/2021, 11:44 AMTypeError: Cannot read property 'getValue' of undefined
i know why is the error, hi know where is exists but still
is there a validation to avoid this error?
c
CD
10/04/2021, 11:56 AMWell, you have
something.getValue()somethingr
razer456
10/04/2021, 11:59 AMi have a search, which is empty i know that.. but i just want to avoid this error.. by some way is it possible????
c
CD
10/04/2021, 12:00 PMShare your code
r
razer456
10/04/2021, 12:27 PMsee what i am saying is
razer456
10/04/2021, 12:27 PMi know that search is null and it has no value ... but if it has no value ..i need to avoid this error
razer456
10/04/2021, 12:28 PMUntitled.txt
c
CD
10/04/2021, 12:28 PMif ( /* whatever */)r
razer456
10/04/2021, 12:29 PMi have two searches .. i have to get value from whatever search and set it
razer456
10/04/2021, 12:29 PManyway it wont work
razer456
10/04/2021, 12:29 PMbecause it is still getting value from the search ... in the array
razer456
10/04/2021, 12:30 PMi have given null validation but the getValue will still show error because the search is null
c
CD
10/04/2021, 12:30 PMso use intermediate variables instead of shoving it all on one horrible line
r
razer456
10/04/2021, 12:31 PMi was try to avoid that extra work
razer456
10/04/2021, 12:31 PMbut okay
c
CD
10/04/2021, 12:34 PMbasic error checking and validation is not "extra work", it should be the minimum
r
razer456
10/04/2021, 12:58 PMno , but creating a whole new object is. when existing code is already up and running. i just thought there might me a way. its okay. there isn't.
s
stalbert
10/04/2021, 2:23 PMYou might want to consider refactoring that search to SuiteQL. I'd write that code very differently - using functions to better reflect the meaning of the code and I'd avoid the column indexing for sure.
stalbert
10/04/2021, 2:33 PMI also generally do NOT create a bunch of intermediate objects or variables unless really needed to make code more clear. In general, every time you introduce extra objects or variables it makes code harder to read because each one is an additional mental connection the reader has to make when trying to understand the code.
👍 1
stalbert
10/04/2021, 2:35 PMironically, sometimes the seeming need to create intermediate variables is a sign of a confusing design. Whenever I find myself thinking I need to create intermediate variables I use it as a sign to step back and consider if I can instead simplify the whole thing.
stalbert
10/04/2021, 2:50 PMin short, one thing you can do (if you're not going to refactor) is use the optional chaining operator. e.g.
searchResultTransactn[0]?.getValue(...)c
CD
10/04/2021, 3:27 PMKinda hard to give decent advice when not all of the code was even shared
s
stalbert
10/04/2021, 3:47 PMtrue that.
r
razer456
10/04/2021, 4:17 PMthanks @stalbert
13 Views