• Status: Solved
• Priority: Medium
• Security: Public
• Views: 160

# Excel showform modification

Ken Butters formulated a solution to my previous question:
http://www.experts-exchange.com/Software/Office_Productivity/Office_Suites/MS_Office/Excel/Q_28411230.html

The new example shows 29 columns on sheet 2 when there are many more. I've moved media to column AC. I'd like to know how to capture the data and blank spaces between column D and column AC of sheet2 using Mr. Butters showform macro. I am using Excel 2003, is this a limitation in this version of Excel?
TestBook1-Test.xls
showform1-detail-rev.txt
0
mfrax
• 4
• 2
1 Solution

Author Commented:
Attached is the original workbook with the showform recorded.
TestBook1-Result.xls
0

Commented:
The code that copies data from one sheet to another is this:

j = 2
For i = 2 To lastCellWk2.Row
If SelectedCustomersDict.Exists(CStr(Wk2.Cells(i, 2).Value)) And Wk2.Cells(i, 29).Value = Me.cboMediaType.Value Then
wk3.Range("B" & j & ":J" & j).Value = Wk2.Range("D" & i & ":AC" & i).Value
j = j + 1
End If
Next

The specific line that actually does the copy is just this one:
wk3.Range("B" & j & ":J" & j).Value = Wk2.Range("D" & i & ":AC" & i).Value

in the above single line of code the lower case 'j' is a variable that represents the rownumber of the destination sheet.

The lowercase 'i' is a variable that represents the rownumber of the source sheet.

So when a single row of data is copied the line of code would be the equivalent of something like this:

wk3.Range("B4:J4").Value = Wk2.Range("D2:AC2").Value

Now when you look at a specific example the problem becomes more evident... you are indicating that the data in columns D2 through AC2 get copied to columns B4 to J4.

D2 through AC2 contain 26 cells, where B4 through J4 only contain 9 cells.

If you make the destination of the copy contain the same number of cells as the source of the copy you should be good to go.

This modification should do it:

wk3.Range("B" & j & ":AA" & j).Value = Wk2.Range("D" & i & ":AC" & i).Value
0

Author Commented:
Thank you for pointing out that I missed the beginning of that portion of the instruction. My mind missed that it was referring to wk3. When I modified the previous macro, it hadn't had that portion of the instruction.
0

Author Commented:
What does the instruction look like if it is similar to

Else if "all" or "All" then
End if

for example all media or all dates but only once per ID?
0

Commented:
It would not be as simple as updating the If statement.

The current solution is not designed to be able to show all media types or all dates.

The current solution requires The selection of a single Media Type, and a Single date.

The way the dates/Media types in are being chosen in this solution, is via the combo boxes.  If you wanted to change the solution to be able to select multiple dates / multiple media types, several things would have to change:

1) Change from comboBox to Listbox
2) change property of listbox to allow multiple entries to be selected
3) change each step of logic to not just check for the single option selected, but for all selected options.
0

Author Commented:
Clear and concise. Thanks.
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.