excel - Copy data from one sheet to another with the cell value -


i have sheet "data". sheet, looking column k. if red in colour extract complete row , copy them sheet " delay".

i following below code. code not have error but, copies 4 rows of red while have 12 rows.

could me find wrong , changes need? 

sub delay() dim cell range dim nextrow long dim double  application.screenupdating = false = application.worksheetfunction.counta(sheets("data").range("k:k")) each cell in sheets("data").range("k5:k" & a) if cell.displayformat.interior.color = vbred nextrow = application.worksheetfunction.counta(sheets("delayed").range("k:k")) rows(cell.row).copy destination:=sheets("delayed").range("a" & nextrow + 1) end if  next application.screenupdating = false end sub

first of all:
worksheetfunction.counta counts number of cells not empty , values within list of arguments, can't use count total number of rows or find number of last row (unless cells not empty).
might need is: 

nmbrows = workbook("workbookname").worksheet("sheetname").range("k" & rows.count).end(xlup).row

or less blunt.
using counta can result in constriction of range of search lead in data loss or, in case, inserting row in incorrect place.
example:
result of 

option explicit sub test()     dim long     = application.worksheetfunction.counta(thisworkbook.sheets("Ëèñò1").range("a:a"))     thisworkbook.sheets("Ëèñò1").range("b1").value = end sub

is
enter image description here
second:
cautious, add reference each range, sheet as

thisworkbook.sheets("data").range("k5:k" & nmbrows)

by doing can you're refering correct range want check.

another note:
i'm not vba expert if count number of rows each sheet respectively @ start of macro, in loop use nextrow=nextrow + 1 construction instead of calling function each time.


Comments

Post a Comment

Popular posts from this blog

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m
Read more

javascript - Z-index in d3.js -

i have map , when hover effect on region of map below yellow line , circle, last ones disappear. how can solve problem? http://jsfiddle.net/ot21n9qx/ var arc = g.append("path") .style("fill", "none") .style("stroke", "yellow") .style("stroke-width", 2) .attr("d", "m" + pathorigin[0] + "," + pathorigin[1] + " q" + svgpoint[0] + "," + pathorigin[1] + " " + svgpoint[0] + "," + svgpoint[1]); var circlesize = d3.scale.linear() .domain([0, 0.5, 1]) .range([4, 10, 4]); svg not have z order draws objects in order created. need create path , circle after create highlight regions. here forked fiddle: http://jsfiddle.net/o00fzgaf/1/
Read more