C / C++ / MFC

7-Feb-02 7:18

HI Guys,

i am using the CStdioFile ReadString() Function to read several files with the size of 350KB per File. Sometimes it takes to long to load the files and sometimes my app crashes. Is there any solution to save loading time ??? maybe any other function ?

Best Regards

Sonu

Joaquín M López Muñoz7-Feb-02 7:53

Joaquín M López Muñoz

7-Feb-02 7:53

...and sometimes my app crashes. Is there any solution to save loading time ???

Your app crashing is a very likely sign that you've got a bug in your code --this may have to do with the poor performance you're obtaining, also.
Could you post the piece of code in which you're using ReadString? This would help determine what's going on.

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo

Sonu Kapoor7-Feb-02 8:40

7-Feb-02 8:40

My app only crashes cause the files which i am loading are to large. If i
try this with smaller files, everything works fine. So i need a solution for
the loading Problem.

strFile is just a csv file with the size of 350KB.
CString sRow;
CStdioFile inFile(strFileName, CFile::modeRead);
while(inFile.ReadString(sRow) != FALSE)
{
// loading the String in a ListCtrl

}

inFile.Close();

you see i am not doing very much

Best Regards

Sonu

User 66587-Feb-02 8:53

User 6658

7-Feb-02 8:53

what about checking if "inFile" is valid?
could be a problem as well

modified 12-Sep-18 21:01pm.

Nish Nishant7-Feb-02 8:55

Nish Nishant

7-Feb-02 8:55

350 kb is surely not large enough to cause a crash. It must be something in your code. Carefully go through it step by step.

Nish

Sonork ID 100.9786 voidmain
www.busterboy.org
If you don't find me on CP, I'll be at Bob's HungOut

Sonu Kapoor7-Feb-02 8:59

7-Feb-02 8:59

I know. I tryed the same with smaller files and everything works fine. NO CRASH

Best Regards

Sonu

Joaquín M López Muñoz7-Feb-02 9:09

Joaquín M López Muñoz

7-Feb-02 9:09

I don't think CStdioFile is having anything to do with the slowness of your code (you can verify this commenting out the code inside the loop and seeing if the app still goes slow).
More probably it is the process of populating the CListCtrl that is causing your app to go slowmo. You can do two things to accelerate this process:

Rick York pointed out an interesting idea in a previous thread today that applies to your case. Use WM_SETREDRAW to prevent redrawing while you fill the list:
```
list.SendMessage(WM_SETREDRAW,(WPARAM)FALSE,0);
while(...){
...
}
list.SendMessage(WM_SETREDRAW,(WPARAM)TRUE,0);
```
Make sure the styles LVS_SORTASCENDING and LVS_SORTDESCENDING are note set for the list control, to prevent sorting in the middle of the population process. Later, you can sort the data (if this is your wish) with CListCtl::SortItems.

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo

Sonu Kapoor7-Feb-02 9:14

Re: Sams book on visual C++

7-Feb-02 9:14

Thanx I will try this. Maybe this can solve my Prob.

Best Regards

Sonu

Sams book on visual C++

Rajveer7-Feb-02 5:31

Rajveer

7-Feb-02 5:31

I'm trying to learn using "Sams Teach Yourself Visual C++ 6 in 21 Days". Has anybody read that? I'm finding that there are a lot of mistakes in the sample code in the book. For example, I took this code exactly out of the book:

void CDialogsDlg::OnYesnocancel()
{
// TODO: Add your control notification handler code here

int iResults;

iResults = MessageBox("Press the Yes, No, or Cancel button", "Yes, No, Cancel Dialog",
MB_YESNOCANCEL | MB_ICONINFORMATION);

switch(iResults)
{
case IDYES:
m_sResults = "Yes! Yes! Yes!";
break;
case IDNO:
m_sResults = "No, no, no, no, no.";
break;
case IDCANCEL:
m_sResults = "Sorry, canceled.";
break;
}
UpdateData(FALSE);
}

and I got these errors:

--------------------Configuration: Dialogs - Win32 Debug--------------------
Compiling...
DialogsDlg.cpp
C:\Windows\Desktop\Dialogs\DialogsDlg.cpp(185) : error C2065: 'm_sResults' : undeclared identifier
C:\Windows\Desktop\Dialogs\DialogsDlg.cpp(185) : error C2440: '=' : cannot convert from 'char [15]' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast
C:\Windows\Desktop\Dialogs\DialogsDlg.cpp(188) : error C2440: '=' : cannot convert from 'char [20]' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast
C:\Windows\Desktop\Dialogs\DialogsDlg.cpp(191) : error C2440: '=' : cannot convert from 'char [17]' to 'int'
This conversion requires a reinterpret_cast, a C-style cast or function-style cast
Error executing cl.exe.

DialogsDlg.obj - 4 error(s), 0 warning(s)

Am I forgetting to do something, because I've doubled checked the code and its exactly what appears in the book. This is happening to me for every single code in the book. Can anybody recommend a better book?

Jon Hulatt7-Feb-02 6:18

Jon Hulatt

7-Feb-02 6:18

It looks to me like m_sResults is a member variable, probably a CString mapped in classwizard to an edit box or a static control or something.

What does that mean? well, you can add an edit box to your dialog in the dialog editor. You've probably already learned that in VC/MFC, a dialog is represented by a class. well, each control on the dialog box is represented by a member variable in that class. you use the class wizard tool to control the mappings between controls on the dialog, and memebr variables in your class.

The code snippet you've typed in relies on this class wizard mapping being setup: if it was, you wouldnt be getting those errors.

I think therefore that the code you are reading is meant to be indicative of the point in discussion and not directly intended to be typed in and tried. It probably isn't necessary for the book to explain how to make every classwizard mapping for every control in every example.

If there's a CD in the book, it probably has VC projects to open of all these code snippets, so you can see the code in action. These projects will include all of the dependant configuration that is omitted from the sample.

Stick at it, and if you get stuck, post here... people will help.

You'll feel very rewarded when you write your first working MFC app on your own. But I warn you now, the 21 days thing is a gimmick... it takes years to learn VC++ properly. The really knowledgeable people on this forum have all been using VC for years, and I bet if you asked any of them they'd tell you they're still learning.

Jon

Sorry to dissapoint you all with my lack of a witty or poignant signature.

7-Feb-02 4:52

just forgot to say thank you

Jon Hulatt7-Feb-02 5:04

Jon Hulatt

7-Feb-02 5:04

Please stop including your whole project every time you have a problem.

And spending some more time describing what the problem is would help too Smile | :)

:\Windows\Desktop\DataStru\main.cpp(639) : error C2664: 'binarySearch' : cannot convert parameter 1 from 'class Stock *(void)' to 'class Stock *'        There is no context in which this conversion is possibleError executing cl.exe.main.obj - 1 error(s), 0 warning(s)

on line 639, cast your parameter 1 to (Stock *).

Giles7-Feb-02 9:40

Giles

7-Feb-02 9:40

You missed a ; - or did you. Use MSDN, and then you would leard the following :

'function' : cannot convert parameter number from 'type1' to 'type2'

The following is an example of this error:

class A {} a;
func( int, A );
void main()
{
   func( 1, 1 );  // error, no conversion from int to A
}

It just gets on my tits. Do some investigation and try an narrow the problem down by say - commenting out code!!!!!! And seeing if it still recompiles, unless you wrote all of this with out one build.

Don't let me see you post a stupid question with all this code again.
Eek! | :eek:

Giles

Re: iCListView and CTreeView

Christian Graus7-Feb-02 9:51

Christian Graus

7-Feb-02 9:51

I'll just add my voice to the people saying you should take a closer look yourself before posting, and at the least show you've done that by only posting the offending code instead of half your project, and describe your problem a bit.

However,

return -(first +1); //failed to find

why are you returning a range of numbers to indicate failure ? Usual practice is to return -1. what if a client of this function accidentally sends a range that starts at -3 ? You're not doing any range validation ( i.e. checking for valid values to be passed in ), which makes all of this code very fragile.

Christian

I have come to clean zee pooollll. - Michael Martin Dec 30, 2001

Picture the daffodil. And while you do that, I'll be over here going through your stuff.

eVC Image Zoom Problem

7-Feb-02 4:36

I have an embedded Visual C++ program, which uses the Imgdecmp.dll to load and display JPEG images.
Zooming is achieved through a variable contained in the Decompress Image Structure (an int between 1 and 100). This works fine, however the map needs to be reloaded (via the open file dialog everytime the user zooms in or out)to show the zoom effect.
Does anyone know a way around this...any suggestions greatly appreciated.
Cheers,
Colm

iCListView and CTreeView

7-Feb-02 3:07

Hello,
I am not able to include any class derived from CListView or CTreeView in my project for say a child view of some sort.
I get error saying the class is undefined.
I can however make a new project with either of the views by selecting the right choice in the app wizard.
Why?

Jon Hulatt7-Feb-02 4:42

Jon Hulatt

7-Feb-02 4:42

You need to include:-

#include <afxcview.h>

Personally, i'd add that to StdAfx.h

This is a frequent problem: many of the MFC classes are defined in header files not included by default. Whenever you choose a new class or function or whatever, if you look at the class overview page in MSDN, it tells what what header file to use. Make sure you add suitable includes for this header file.

how to compile win32 program in bcb 5.0

Johndotcom7-Feb-02 3:03

Johndotcom

7-Feb-02 3:03

as title, my win32 program(just use API) can run in VC 6
but can't in BCB 5.0, why?
just a simple api function MessageBox() also can't show anything

hi

Carlos Antollini7-Feb-02 3:12

Carlos Antollini

7-Feb-02 3:12

Compile?
How do you are compiling the project?

Regards....

The programmer's national anthem is 'AAAAAAAARRRRGHHHHH!!'.

Carlos Antollini.
Sonork ID 100.10529 cantollini

Johndotcom7-Feb-02 16:21

Johndotcom

7-Feb-02 16:21

i'm sorry , may be there is something wrong with my expression,
that is, my code can't run in bcb 5.0 buy can in vc 5.0.
for example, the MessageBox() can't work!

hi

Carlos Antollini8-Feb-02 2:35

Carlos Antollini

8-Feb-02 2:35

But Does your code Compile, or not?

Regards...

"The three most dangerous things in the world are a programmer with a soldering iron, a hardware type with a software patch and a user with an idea."

Carlos Antollini.
Sonork ID 100.10529 cantollini