Solved

Excel Macro is missing some strings.

Posted on 2011-09-10
6
280 Views
Last Modified: 2012-05-12
Hi Experts:

I have a macro attched fetching data from txt file, it works mostly fine except some issues:
After I checked manully the txt total of accounts is not matching;
1- Follwing accounts are missed I don't know why: account are
     [ 798026 - 798039 - 798020 - 798040 - 798016 - 798017 - 208028 ].
2- in some customers if there is FX line should be add to TOTAL FINANCED, the issue is macro missing that so    GRAND TOTAL becomes wrong.


Thanx.



<<byundt removed MatLab and VBScript Zones 9-10-11>>



Master-Filtering.xlsm
CBG-Aug--11.txt
0
Comment
Question by:obad62
  • 3
  • 3
6 Comments
 
LVL 17

Expert Comment

by:andrewssd3
ID: 36516445
Your vba project is password protected - could you free it up and repost pls?
0
 

Author Comment

by:obad62
ID: 36517237
macro password :  pingo000
0
 

Author Comment

by:obad62
ID: 36518886
no-pass version
Master-Filtering.xlsm
0
Live: Real-Time Solutions, Start Here

Receive instant 1:1 support from technology experts, using our real-time conversation and whiteboard interface. Your first 5 minutes are always free.

 
LVL 17

Expert Comment

by:andrewssd3
ID: 36522062
Hi

The problem with the missing accounts is that the account name in each case began with a number, and the macro was only looking for alphabetic characters.  You can fix this by changing in clsOfficers the line the identifies the account as follows:
        'ElseIf TheLine Like "* ###### [A-Z]*" Then
        ' andrewssd3 replace this line
        ElseIf TheLine Like "* ###### [A-Z0-9]*" Then

Open in new window


The exception is 208028 which is missed because its account name is all blanks.  This is more difficult to pick up - can you confirm that you need it?

I don't understand exactly what you require with the FX line - there are two different sorts of FX lines in the file - can you please explain exactly what you need to identify, and which total they need to be added to?

Thanks

Stuart
0
 

Author Comment

by:obad62
ID: 36523302
Dear Stuart,

Many thanks for your reply.

1- Missing account is solved, 208028 issue i asked IT to add the name and worked fine.

2- FX , issue ,   Yes there are 2 types of FX and I need both of them to be checked by the macro
      ( FX Spot, FX forward ) if any or both is there macro will add it to TOTAL FINANCED.

Obad

0
 
LVL 17

Accepted Solution

by:
andrewssd3 earned 500 total points
ID: 36528069
Hi

I have made some changes to the clsOfficers Import code, which I've marked with my id as the previous people did.  I think this does what you want now.  

Stuart Master-Filtering.xlsm
0

Featured Post

Gigs: Get Your Project Delivered by an Expert

Select from freelancers specializing in everything from database administration to programming, who have proven themselves as experts in their field. Hire the best, collaborate easily, pay securely and get projects done right.

Question has a verified solution.

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

Introduction While answering a recent question (http:/Q_27311462.html), I created an alternative function to the Excel Concatenate() function that you might find useful.  I tested several solutions and share the results in this article as well as t…
This article descibes how to create a connection between Excel and SAP and how to move data from Excel to SAP or the other way around.
The view will learn how to download and install SIMTOOLS and FORMLIST into Excel, how to use SIMTOOLS to generate a Monte Carlo simulation of 30 sales calls, and how to calculate the conditional probability based on the results of the Monte Carlo …
This Micro Tutorial demonstrate the bugs in Microsoft Excel for Mac with Pivot Charts.

813 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

Need Help in Real-Time?

Connect with top rated Experts

19 Experts available now in Live!

Get 1:1 Help Now