1 回答

TA贡献1963条经验 获得超6个赞
解释:
迭代删除工作表中行的一种方法是创建一个向后 for
循环。
替换:
for (var i = 0; i < data.length; i++)
与:
for (var i = data.length - 1; i >= 0; i--)
这样,每删除一行,data
仍然会对应到正确的行。
另一个问题是你得到var range = ss1.getRange(1, 1, lr, lc)
。这意味着,您从第一行(包括标题)开始迭代,然后您使用这样的解决方法:
if (i == 0) {
ss1.deleteRow(i+2);
} else {
ss1.deleteRow(i)
}
但实际上,您不需要首先包含标头。请改用:var range = ss1.getRange(2, 1, lr, lc)并将if/else语句替换为:ss1.deleteRow(i+2)。
因为data索引从 开始0,但您range从 row 开始2。
解决方案:
var etfar = ["Cash File"] //This is a string because I have multiple sheets I'm looping through
function cashCopy(etf) {
var ss = SpreadsheetApp.getActiveSpreadsheet();
var ss1 = ss.getSheetByName(etf);
var ss2 = ss.getSheetByName("Target Sheet");
var lr = ss1.getLastRow();
var lc = ss1.getLastColumn();
// gets the data in Cash File
var range = ss1.getRange(2, 1, lr, lc); // <- modification
var data = range.getValues();
// loops through rows in data
for (var i = data.length - 1; i >= 0; i--)
{
var check = data[i][2] // ith row, 3rd column
if (check.includes("USD")) {
var rowToCopy = data[i];
ss2.appendRow(rowToCopy);
ss1.deleteRow(i+2); // <- new code
};
}; // end i
}
for (var i = 0; i < etfar.length; i++) {
cashCopy(etfar[i])
}
添加回答
举报