but had hoped to learn how to free up memory that might otherwise be used by large structures in the manner usually deemed to be correct in programming circles

<Fixed code tags: [code][/code].>

Help with Code Tags

C++ Syntax (Toggle Plain Text)

struct movies
{
   char title[10];
   int year;
};
 
int main()
{
   movies *films = new movies;
   // use films
   movies.title = "Harts War";
   delete films;
   return 0;
}

struct movies { char title[10]; int year; }; int main() { movies *films = new movies; // use films movies.title = "Harts War"; delete films; return 0; }

The compiler responds with an error on the line that tries to assign a string to the title. I believe it is complaining about a pointer being on one side but no pointer on the other.

If anyone can provide a means of both creating a structure and using its components, so that I can learn how to delete a structure in Builder 6.0 if I wish, I would be very grateful.

Automatic memory is freed when an object goes out of scope.

First, movies is a type, not an object. So this

Help with Code Tags

C++ Syntax (Toggle Plain Text)

movies.title = "Harts War";

movies.title = "Harts War";

should be this

Help with Code Tags

C++ Syntax (Toggle Plain Text)

films->title = "Harts War";

films->title = "Harts War";

But that is still wrong because you cannot assign C-style strings that way. Use strcpy or std::string for assignment.

What is the bigger picture? What do you want to do? (Don't solve the wrong problem with a perceived solution.)

As for the bigger picture, I had the functions working before I submitted my first post to this forum. I had perceived the deletion of a structure as a mere detail...until now. The reason follows and is based upon differences in the way in which one must code to use the structure after declaring it using a pointer reference.

When I created the structure using "movies films;", and referenced the data using "films.title" and "films.year", my file input/output (read/write to disk) functions worked well. When I create the structure using "movies *films = new movies;", and reference the contents using "films->title" and "films->year", I can no longer display the title. What is the difference between these two approaches, and why must I use two different means of referencing the variables that comprise the structure depending on how the structure is declared?

I am saving and reading structures from files using an iostream function and am referencing the structure to be saved using a char pointer as "(char*)(&films)". The problem seems to arise if I use the pointer means of declaring the structure ("movies *films = new movies;") when I read from the file and try to display "films->title" in a RichEdit text control using "RichEdit2->Text = films->title". I had this problem previously when I declared "title" to be an AnsiString, but worked through it and used char[10] to declare "title", StrCopy() to load data into "films.title", and "RichEdit2->Text = films.title" to display it.

I had hoped to create a general structure I/O function for my future use, but now seem to need to create two differently coded structure I/O functions to accomodate the pointer means of declaring the structure, particularly if I wish to comply with the current C++ ANSI standard in my coding efforts in terms of the means by which I declare a structure. Using the iostream method, a pointer to a location in memory, and the size of the data I need to store, doesn't seem to enable me to display the title using "RichEdit2->Text = films->title;". I had previously been able to load the title as part of the structure that was loaded using the iostream method, then used "RichEdit2->Text = films.title" to display the title when I declared the structure using "movies films;".

What I meant by the bigger picture was, "This program reads and writes movie information from/to a text file. Movie information is then displayed ..." The minutae are the details of structures, dynamic/automatic allocation, etc.

For example, loading a particular structure with information: why do you need dynamic allocation? Is the structure several thousand bytes in size? Is it part of a linked list? If the information is relatively small and the data is displayed (and discarded) as it is being read, then why bother?

Are you asking about the difference between accessing structure members with the . and the -> operator? You use . with objects and -> with pointers to objects.

As I mentioned, this is not really what I was getting at with the 'big picture'. If you are going into such detail, you might as well post the code. Paraphrasing code leads to debugging paraphrasing, which may be neither here nor there.

Again, I think you may need to try to see the forest from the trees (or at least help me do so).

For example, the following is a short snippet that uses a couple of functions that do very much the same thing in two different ways. The mywrite function uses dynamic allocation, the myread function uses automatic allocation. Both essentially do the same thing, but mywrite has excess overhead for the (in my opinion needless) dynamic allocation.

Help with Code Tags

C++ Syntax (Toggle Plain Text)

#include <iostream>
#include <fstream>
#include <cstring>
using namespace std;
 
struct movie
{
   char title[30];
   int  year;
};
 
movie collection[] =
{
   { "Star Wars",           1978},
   { "Empire Strikes Back", 1980},
   { "Return of the Jedi",  1983},
};
 
void mywrite(const char *filename)
{
   ofstream file(filename);
   movie *film = new movie;
   for ( size_t i = 0; i < sizeof collection / sizeof *collection; ++i )
   {
      strcpy(film->title, collection[i].title);
      film->year = collection[i].year;
      file << film->title << ',' << film->year << '\n';
   }
   delete film;
}
 
void myread(const char *filename)
{
   ifstream file(filename);
   movie film;
   while ( file.getline(film.title, sizeof film.title, ',') &&
           file >> film.year && file.get() )
   {
      cout << film.title << ',' << film.year << '\n';
   }
}
 
int main()
{
   const char filename[] = "file.txt";
   mywrite(filename);
   myread(filename);
   return 0;
}
 
/* my output
Star Wars,1978
Empire Strikes Back,1980
Return of the Jedi,1983
*/

#include <iostream> #include <fstream> #include <cstring> using namespace std; struct movie { char title[30]; int year; }; movie collection[] = { { "Star Wars", 1978}, { "Empire Strikes Back", 1980}, { "Return of the Jedi", 1983}, }; void mywrite(const char *filename) { ofstream file(filename); movie *film = new movie; for ( size_t i = 0; i < sizeof collection / sizeof *collection; ++i ) { strcpy(film->title, collection[i].title); film->year = collection[i].year; file << film->title << ',' << film->year << '\n'; } delete film; } void myread(const char *filename) { ifstream file(filename); movie film; while ( file.getline(film.title, sizeof film.title, ',') && file >> film.year && file.get() ) { cout << film.title << ',' << film.year << '\n'; } } int main() { const char filename[] = "file.txt"; mywrite(filename); myread(filename); return 0; } /* my output Star Wars,1978 Empire Strikes Back,1980 Return of the Jedi,1983 */