Removing empty text frames

Question

trying to remove any text frames with empty content but it doesn't seem to be removing correct textFrame layers, any idea what I might be doing wrong?
var numberOfEmptyTextBoxes = 0;
var layersWithNoText = new Array();
if ( app.documents.length > 0 ) {
    for ( i = 0; i < app.activeDocument.textFrames.length; i++ ) {
        text = app.activeDocument.textFrames[i].textRange;
        numWords = app.activeDocument.textFrames[i].words.length;
        if (numWords == 0){
                  layersWithNoText.push(i);
            numberOfEmptyTextBoxes++;
    if(numberOfEmptyTextBoxes > 0){
              alert("WARNING you have " + numberOfEmptyTextBoxes + " empty text boxes");
    alert("layers with no text:" +layersWithNoText);
    removeTextLayersWithNoContent(layersWithNoText);
function removeTextLayersWithNoContent(layersWithNoText) {
          var layersWithNoText = layersWithNoText;
          for (var i = 0; i < layersWithNoText.length; i++) {
                    var currentIndex = layersWithNoText[i];
                    alert("current index: "+currentIndex)
                    app.activeDocument.textFrames[currentIndex].remove();

Answer 1

yes, to do it like that, you would have to loop backwards to remove the textframes and not mess up with the indexes,
or, push the textframes into your array like this
var numberOfEmptyTextBoxes = 0;
var layersWithNoText = new Array();
if ( app.documents.length > 0 ) {
    for ( i = 0; i < app.activeDocument.textFrames.length; i++ ) {
        text = app.activeDocument.textFrames[i].textRange;
        numWords = app.activeDocument.textFrames[i].words.length;
        if (numWords == 0){
                  //layersWithNoText.push(i); // don't push the indexes
                  layersWithNoText.push(app.activeDocument.textFrames[i]); // push the actual text frames
            numberOfEmptyTextBoxes++;
    if(numberOfEmptyTextBoxes > 0){
              alert("WARNING you have " + numberOfEmptyTextBoxes + " empty text boxes");
    alert("layers with no text:" +layersWithNoText);
    removeTextLayersWithNoContent(layersWithNoText);
function removeTextLayersWithNoContent(layersWithNoText) {
          var layersWithNoText = layersWithNoText;
          for (var i = 0; i < layersWithNoText.length; i++) {
                    var currentIndex = layersWithNoText[i];
                    alert("current index: "+currentIndex)
                    //app.activeDocument.textFrames[currentIndex].remove(); // removing 1 item re-indexes the remaining items
                    layersWithNoText[i].remove(); // remove the textframes in the array, regardless of actual index