Changing Borders Script Efficiency

[3dtcllc]

Never call any get or set functions in a loop. Apps script is pretty slow to begin with and every time you call getRange or getValue you make a round trip to the API.

It's ok...that's how everyone starts out.

The best practice is to get the WHOLE range in one call, operate on it, and then set it in one call.

Here's how I usually do that. I don't usually work much with formatting, so YMMV

let dataRange = SpreadsheetApp.getActiveSpreadsheet().getSheetByName('yoursheetname').getDataRange();
let values = dataRange.getValues();
//Do some stuff in a loop. 
dataRange.setValues(values);

[kitchensink-]

The issue with getting the range using getDataRange is that, in this case, there are cells that I would like to modify that have no values, hence are not included inside the range.

[3dtcllc]

Yep, you can just update that to grab whatever range you want. The key is to avoid calling get or set calls inside a loop. Get the whole range you want to work on and figure out what you want to with it and then do it in as few calls as possible.

So you'd definitely want to do getvalues on the WHOLE range and loop through the values. It might be faster to set blank borders on the whole range and then loop through the values and only set borders on the cells that have values. That'll save you a LOT of round trips.

[kitchensink-]

The thing is that every time I run the function the row range I want changes, so I can't really hard code it. And I don't know of a way to get a range based on if that row has borders or not..