|
Budric B. wrote: The problem is when I call "delete Voxels;"...
Which should be:
delete [] Voxels;
"A good athlete is the result of a good and worthy opponent." - David Crow
"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne
|
|
|
|
|
I see...even though it's an object? I know I'm casting an array to an object, but I would have thought things would behave as they would for any object created with new.
|
|
|
|
|
Well, it wouldn't hurt to try. If it doesn't work, you're no worse off.
"A good athlete is the result of a good and worthy opponent." - David Crow
"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne
|
|
|
|
|
I'm probably wrong, but at first glabce something seems fishy here....
When you write you do this
file.write((char *)voxelData->m_pTexture, size); When you read you do this
Voxels * voxelData = (Voxels *) memblock;
voxelData->m_pTexture = (unsigned short *) (memblock + sizeof(Voxels));
Is there a discrepancy there?
Mark
Mark Salsbery
Microsoft MVP - Visual C++
![Java | [Coffee]](https://codeproject.global.ssl.fastly.net/script/Forums/Images/coffee.gif)
|
|
|
|
|
No,
I first write the object, which contains the data members and a pointer at the end. The size of the object is sizeof(Voxels).
I then write the array of stuff.
When I read, I read everything. I know that the first sizeof(Voxels) bytes is the object including a pointer with a value that's meaningless. I then assign the pointer to point to the data, which just happens to be right after the object (because that's where I read it into).
|
|
|
|
|
What is the reason that you do not have save/load as members of the Voxel class that then manages the internal char*
??
|
|
|
|
|
I get the impresion that Voxels is an object with actual behavior, and therefore shouldn't be written to a file like that. Things like virtual function table pointers can't be saved and restored that way. Also, it looks like Voxels is a relatively small object with a pointer to a huge array, so you wouldn't lose much performance by saving the simple members of Voxels individually. I would save the simple members of Voxels first and then save the array with one file.write call. To restore it, I'd create a Voxels object, read the simple members, and then allocate and read the array.
Also, I noticed you are using unsigned int for the size to write to the file. If you use size_t, the type will be adjusted properly if you migrate to a 64 bit system.
Nathan
|
|
|
|
|
It doesn't have that much behavior other than a constructor and a destructor to free the memory. The reason I was doing it this way was because I would be adding more data members to describe the data later. But you're probably right that it wouldn't add that much performance loss to write each data member. I'll have to time it and see.
I've made a workaround where I allocate 2 buffers, 1 for the object and one for the data, the rest of the code stays the same. That doesn't trigger the assertion. However I don't know if the object's memory gets freed or not since I'm still calling delete Voxels and not delete [] voxels;
|
|
|
|
|
Budric B. wrote: It doesn't have that much behavior other than a constructor and a destructor to free the memory. The reason I was doing it this way was because I would be adding more data members to describe the data later. But you're probably right that it wouldn't add that much performance loss to write each data member. I'll have to time it and see.
Watch out for the copy constructor and assignment operator, since the compiler will generate invalid forms and call them freely.
Budric B. wrote: I've made a workaround where I allocate 2 buffers, 1 for the object and one for the data, the rest of the code stays the same. That doesn't trigger the assertion. However I don't know if the object's memory gets freed or not since I'm still calling delete Voxels and not delete [] voxels;
I think that should work. The main difference between delete and delete[] is the way destructors are called, which shouldn't be a problem if it's just data.
Nathan
|
|
|
|
|
Is it possible to trap DDE messages from the shell?
I mean when a file is opened and it's open method is a DDE command rather than a command line. I have been trying to use SetWindowsHookEx() with no success so far, just wanted to see if I was fighting a lost battle.
|
|
|
|
|
I dont know about "trapping" them but you can monitor them. See the DDESpy Tool, it should have been installed in your Visual Studio/Common/Tools directory, the sources are available as a sample also.
|
|
|
|
|
I have tried DDESpy and I can see the conversation when I load something like Excel document but what I am trying to do is see the messages when something like a .GIF's 'open' verb is invoked which uses DDE to open it with 'OIS.EXE'.
The ultimate goal is to create a ShellExecuteHook which also works for DDE verbs.
|
|
|
|
|
I'm having the compiler give me
error C2327: 'ABC::m_strErrorMessageText' : is not a type name, static, or enumerator
And this is likely because I'm not putting the code together just the way it wants it. 
Here's the code
class ABC
{
public:
CString m_strErrorMessageText;
const char* GetErrorMessageText(){ return ( (const char*)m_strErrorMessageText ); }
class DEF
{
public:
enum tagSource
{ INIT,
TERMINATE,
OPEN,
SQL
} Source;
char m_lpErrorText[256];
DEF(tagSource src){Source = src;}
};
class GHI
{
public:
error-> GHI() { m_strErrorMessageText = "Registry Information is missing. Please run BATCH_CONFIG.EXE to correct."; }
};
This code is to house various exceptions that may occur within a library and I'm trying to let each of the different exception types, DEF, GHI put text into the ABC::m_strErrorMessageText member in anyway they want to. The consumers of the library can then decide what exception types they wish to catch and simply use the ABC::GetErrorMessageText in order to get meaningful information. Any ideas on what I might still need to be doing here? Thanks.
Chris Meech
I am Canadian. [heard in a local bar]
|
|
|
|
|
Chris Meech wrote: GHI() { m_strErrorMessageText = "Registry Information is missing. Please run BATCH_CONFIG.EXE to correct."; }
Well GHI either needs an instance of ABC or you need to make that CString a static member of ABC. However based on your last comments in the post I have doubts about your design.
|
|
|
|
|
Thanks. Based upon your's and Mike's reply, I am going to have rethink my design. As well as remember a little more about nested classes. 
Chris Meech
I am Canadian. [heard in a local bar]
|
|
|
|
|
You don't have a reference to an ABC object inside GHI's constructor. m_strErrorMessageText is an instance variable, not a static variable, so you need an ABC pointer/reference/object. Nested classes are only a definition scope - you don't automatically get a GHI instance member of ABC, for example.
DoEvents: Generating unexpected recursion since 1991
|
|
|
|
|
Mike, thanks for your excellent response. It explains things exactly and coupled with led mike's response, I'm going to have to re-design things. Not sure why the original designer used the nested classes. It seems superflous considering the original design had no members for the ABC class. Appreciate the quick explanation and help.
Chris Meech
I am Canadian. [heard in a local bar]
|
|
|
|
|
If you are using nested class then consider using 'friend' to control acess to members of the other classes. But you will still need a pointer/reference to access the in the first place.
|
|
|
|
|
I have added a List control as a resource on a dialog box. It is multicolumn and i have been able to set its column headers and width.
Now in my OnIinitDialog function i want to insert first row where the second column should have a check box instead of having any value.
Dont know how to add a check box as a item to a multilist column.
Please help me.
Dhiraj Kumar Saini
-- modified at 9:53 Wednesday 19th September, 2007
|
|
|
|
|
Dhiraj kumar Saini wrote: Dont know how to add a check box as a item to a multilist column.
One solution is here.
"A good athlete is the result of a good and worthy opponent." - David Crow
"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne
|
|
|
|
|
I want to add such a control in my dialog box which can house several pictures in it and can decrease and incease in size dynamically as i add pictures in it and also pictures can be of any type.
One thing more which i want is that each picture should have a caption at the bottom as a title of the picture and on taking mouse over it the title hsould change its colour from black to blue.
I f u have problem in understanding what i want to do just see webshot desktop which i am also trying to develop exactly same.
Please help me.
Thanks In Advance.
Dhiraj Kumar Saini
|
|
|
|
|
Dhiraj kumar Saini wrote: Please help me.
Help you what? Create your application? Read the first thread in this forum titled "How to get an answer to your question". Pay attention to item #2.
|
|
|
|
|
Dhiraj kumar Saini wrote: I want to add such a control in my dialog box which can house several pictures in it...
Use a static control with the SS_BITMAP style.
"A good athlete is the result of a good and worthy opponent." - David Crow
"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne
|
|
|
|
|
Heh. The OP had more specs. I think led mike is writing it for him
Mark Salsbery
Microsoft MVP - Visual C++
![Java | [Coffee]](https://codeproject.freetls.fastly.net/script/Forums/Images/coffee.gif)
|
|
|
|
|
Whew! That's a relief. I'd hate for the OP to have to go home this evening without having accomplished anything.
"A good athlete is the result of a good and worthy opponent." - David Crow
"To have a respect for ourselves guides our morals; to have deference for others governs our manners." - Laurence Sterne
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
