Want to win a PS4? Go Premium and enter to win our High-Tech Treats giveaway. Enter to Win

x
?
Solved

Use VBA to change cell values in recordset

Posted on 2009-06-29
7
Medium Priority
?
1,476 Views
Last Modified: 2013-11-27
Access 2007 using VBA:  Have a table built with a cross-tab query with qty values in columns whose names change constantly.  The ROW always contains in the first column, Column(0), the Customer.  The second column, Column(1) is always the total shipped and each column thereafter represents a city to which the qty is shipped, i.e....

CUSTOMER         TTL SHIPPED          CITY_1       CITY_2      CITY_3      CITY_(i)
Customer_1            1200                                     600                            600
Customer_2              300                    300
Customer_3            1000                      25                            975
Customer_(x)              10                                       5                                 5

OBJECTIVE:  Cycle through the recordset by column and change the current value from what it is to current value/total shipped to get a percentage value relative to the ROW TTL_SHIPPED not the sum of the column.  I am using the following code.  I get an "Invalid use of NULL" error.  The (i) does increment but I can't get by the value change.      

Private Sub Form_Load()
 
    '////// For recalculating quantities as a percent of Total Shipped.  ///////
    Dim rs As Object, varPctShip As Single, varCurrVal As Single
    
    varPctShip = 0
    varCurrVal = 0
 
    Set rs = CurrentDb.OpenRecordset("tbl_temp_freight_sort_cost")
    
    With rs
        
        rs.MoveFirst
 
            '////// Cycle through columns, regardless of name and number.  //////////
            '       1st Column is referenced as column zero.
            
            For i = 2 To rs.Fields.Count - 1
            
                varCurrVal = rs.Fields(i).Value
                
                If varCurrVal <> 0 Then
                
                varPctShip = varCurrVal / (rs.Fields(1).Value)
                
                .Edit
                !rs.Fields(i) = varPctShip
                .Update
                
                End If
            
            Next
 
        rs.MoveNext
        
        End With
 
End Sub

Open in new window

dynamic-columns.jpg
0
Comment
Question by:jdfuller
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 5
  • 2
7 Comments
 

Author Comment

by:jdfuller
ID: 24740455
Maybe this is available in the cross-tab query that builds the table in the first place.  I couldn't get that to work because I needed the total calculated prior to running the query i nthe first place.
0
 
LVL 85
ID: 24744064
You need to check for NULL or 0 values:

For i = 2 To rs.Fields.Count - 1
  '/Fields(i) and Fields(1) should be both NOT NULL and <> 0
  If Nz(rs.Fields(i),0) <> 0 AND Nz(rs.Fields(1),0) <> 0 Then
    .Edit
    !rs.Fields(i) = rs.Field(i) / rs.Fields(1)
    .Update
  End If

Next i
           
0
 

Author Comment

by:jdfuller
ID: 24748099
The code fails at line 20 before the valuation of the variable.

The Bang after the .Edit gives an error of item not found so I removed it and this worked BUT for only one record.  

I followed the code for the first record and it cycles fine until it gets to the last column then goes to rs.MoveNext and fails to move to the next record and cycle again.  

I have added a check on the recordset to make sure it doesn't run out until it's done but this has sent it into an endless loop for some reason.  I got it to stop and it doesn't seem to like the 'method' referenced in 'recordset_2'.  I have no idea where it came up with that comment.

New code follows...
Private Sub Form_Load()
 
    '////// For recalculating quantities as a percent of Total Shipped.  ///////
    Dim rs As Object, varPctShip As Single, varCurrVal As Single
    
    varPctShip = 0
    varCurrVal = 0
 
    Set rs = CurrentDb.OpenRecordset("tbl_temp_freight_sort_cost")
    
    With rs
        
        Do While (Not (rs.EOF))
        
        rs.MoveFirst
 
            '////// Cycle through columns, regardless of name and number.  //////////
            '       1st Column is referenced as column zero.
            
            For i = 2 To rs.Fields.Count - 1
            
                If IsNull(rs.Fields(i).Value) = False Or rs.Fields(i).Value <> 0 Then
            
                varCurrVal = rs.Fields(i).Value
                
                varPctShip = varCurrVal / (rs.Fields(1).Value)
                
                .Edit
                rs.Fields(i) = varPctShip
                .Update
                
                End If
            
            Next
 
        rs.MoveNext
        
        Loop
        
     End With
 
End Sub

Open in new window

0
Nothing ever in the clear!

This technical paper will help you implement VMware’s VM encryption as well as implement Veeam encryption which together will achieve the nothing ever in the clear goal. If a bad guy steals VMs, backups or traffic they get nothing.

 

Author Comment

by:jdfuller
ID: 24748203
HOLD THE BUS!  I have it running backa nd forth to the first record!  Doh!!  Let me try moving the MoveFirst outside the loop!

0
 
LVL 85

Accepted Solution

by:
Scott McDaniel (Microsoft Access MVP - EE MVE ) earned 2000 total points
ID: 24748280
I just noticed that also ...
0
 

Author Comment

by:jdfuller
ID: 24748347
Swap line 13 and 15 in the second code submission and it works fabulously!!  Thanks LSM for the push over the edge!

JDF
0
 

Author Closing Comment

by:jdfuller
ID: 31598133
Sometimes it takes another pair of eyes.  Thanks LSM.  Your answer to check the null prior to setting the value worked.  I had to get the Bang (!) out of the item reference in line 27 for the field to get the edit to work properly but you get the full credit for the NULL thing.  The BORG is alive!!
0

Featured Post

Concerto's Cloud Advisory Services

Want to avoid the missteps to gaining all the benefits of the cloud? Learn more about the different assessment options from our Cloud Advisory team.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Did you know that more than 4 billion data records have been recorded as lost or stolen since 2013? It was a staggering number brought to our attention during last week’s ManageEngine webinar, where attendees received a comprehensive look at the ma…
We live in a world of interfaces like the one in the title picture. VBA also allows to use interfaces which offers a lot of possibilities. This article describes how to use interfaces in VBA and how to work around their bugs.
With Microsoft Access, learn how to specify relationships between tables and set various options on the relationship. Add the tables: Create the relationship: Decide if you’re going to set referential integrity: Decide if you want cascade upda…
Add bar graphs to Access queries using Unicode block characters. Graphs appear on every record in the color you want. Give life to numbers. Hopes this gives you ideas on visualizing your data in new ways ~ Create a calculated field in a query: …

609 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question