The problem is that your macro starts with the first selected row and moves to the last selected row. following these steps,the first row is evaluated, if it is DELETED,then the second row moves to the first postionand the third row moves to the second, etc. at the next iteration of the loop,the row evaluated moves to the next row.But what WAS the next row had been moved to the FIRST row!so, effectively, it was skipped. What you need to do in this case is:either:- set up two row counters, only incrementing the "eval_Row" if the "current" row is NOT deleted.- or: loop from the last row to the first. following is an example processing the selection in the reverse order: Sub rr() Dim LastRow Dim R As Long, strArray, Ext LastRow = Selection.Row + Selection.Rows.Count If (LastRow > ActiveSheet.Rows.Count) Then LastRow = ActiveSheet.Rows.Count For R = LastRow To Selection.Row Step -1 If (Cells(R, "A").Value & "X" <> "X") Then strArray = Split(ActiveSheet.Cells(R, "A").Value, ".") Ext = strArray(UBound(strArray)) If ((Not LCase(Ext) = "xlsx") _ And (Not LCase(Ext) = "docx") _ And (Not LCase(Ext) = "xlsm") _ And (Not LCase(Ext) = "xls") _ And (Not LCase(Ext) = "doc") _ ) Then ActiveSheet.Cells(R, "A").EntireRow.Delete End If End If Next R End Sub Paul----------------------------------------- “Do all the good you can, By all the means you can, In all the ways you can, In all the places you can, At all the times you can, To all the people you can, As long as ever you can.” - John Wesley -----------------------------------------
On Friday, February 5, 2016 9:47 AM, Izhar <izharra...@gmail.com> wrote: look at the attachment, in this file I've a macro when i run it, the macro does not remove the jpeg extension URL at once but we have to run it again and again then it removes the jpg. REQUIRED I need a macro when i run once it removes all the jpeg extension URL except others thanks in advance -- Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @ https://www.facebook.com/discussexcel FORUM RULES 1) Use concise, accurate thread titles. Poor thread titles, like Please Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice will not get quick attention or may not be answered. 2) Don't post a question in the thread of another member. 3) Don't post questions regarding breaking or bypassing any security measure. 4) Acknowledge the responses you receive, good or bad. 5) Jobs posting is not allowed. 6) Sharing copyrighted material and their links is not allowed. NOTE : Don't ever post confidential data in a workbook. Forum owners and members are not responsible for any loss. --- You received this message because you are subscribed to the Google Groups "MS EXCEL AND VBA MACROS" group. To unsubscribe from this group and stop receiving emails from it, send an email to excel-macros+unsubscr...@googlegroups.com. To post to this group, send email to excel-macros@googlegroups.com. Visit this group at https://groups.google.com/group/excel-macros. For more options, visit https://groups.google.com/d/optout. -- Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @ https://www.facebook.com/discussexcel FORUM RULES 1) Use concise, accurate thread titles. Poor thread titles, like Please Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice will not get quick attention or may not be answered. 2) Don't post a question in the thread of another member. 3) Don't post questions regarding breaking or bypassing any security measure. 4) Acknowledge the responses you receive, good or bad. 5) Jobs posting is not allowed. 6) Sharing copyrighted material and their links is not allowed. NOTE : Don't ever post confidential data in a workbook. Forum owners and members are not responsible for any loss. --- You received this message because you are subscribed to the Google Groups "MS EXCEL AND VBA MACROS" group. To unsubscribe from this group and stop receiving emails from it, send an email to excel-macros+unsubscr...@googlegroups.com. To post to this group, send email to excel-macros@googlegroups.com. Visit this group at https://groups.google.com/group/excel-macros. For more options, visit https://groups.google.com/d/optout.