שלום,
קצת הסתבכתי באיך לבנות destructor למבנה שלי בסי++.(מחלקה של סטודנטים)
מה בדיוק אני אמורה למחוק.
אשמח אם אוכל להעזר במישו , אשלח לו במייל את העבודה שלי.
זה קובץ ההדר שלי:

#pragma once
#include <string>
using namespace std;

struct Course {
int CourseNum;
string CourseName;

void course() {
this->CourseNum=0;
this->CourseName="Non";
}

void course(int CourseNum){
if (CourseNum<1||CourseNum>6) { course();}
if (CourseNum==1) { this->CourseName="C"; this->CourseNum=1;}
if (CourseNum==2) { this->CourseName="Cpp"; this->CourseNum=2;}
if (CourseNum==3) { this->CourseName="Data_Structure"; this->CourseNum=3;}
if (CourseNum==4) { this->CourseName="Algorithms"; this->CourseNum=4;}
if (CourseNum==5) { this->CourseName="Java"; this->CourseNum=5;}
}

};


struct student {
string name, address;
long id;

struct student(){
this->id=0;
this->name=" ";
this->address=" ";

}

struct student(long id, string name,string address){
this->id=id;
this->name=name;
this->address=address;
}

};



class ClassStudent
{

private:
int size;
Course Courses[3];

public:
struct student st;

ClassStudent(void);
ClassStudent(struct student &st);

string getName() const;
string getAddress() const;
long getID() const;

void setName(string name);
void printName() const;
bool setCourse(Course c1, Course c2, Course c3);
bool delCourse(int CourseNum);
void printStudent() ;
int FindStudnt(long id, ClassStudent *sArr, int size);
void ResetCourses( ClassStudent *sArr, int size);

struct student operator=(struct student &st){
struct student temp;
temp.id = st.id;
temp.address =st.address;
temp.name=st.name;
return temp;
}

~ClassStudent(void);
};

ממה שראיתי לא השתמשת בכלל בהקצאה דינמית של שום דבר, ואין לך איזה שהוא מערך של מצביעים שצריך להפעיל עליהם delete - במקרה כזה הדיסטרקטור הדיפולטי יספיק (כלומר - אתה אפילו לא צריך להגדיר אותו).
ד"א - למה הגדרת את האופרטור = של struct student בתוך classStudent ולא בתוך ה-struct עצמו? זה בכלל מתקמפל ככה?


ו-2 הערות כלליות לגבי פרסום קוד בפורום:
1. תשתמש בתגית code (הכפתור שיש לו צורה שלו סולמית)
2. גם אינדנטציה תעזור

תסכים איתי שהרבה יותר קל לקרוא את הקוד שהוא נראה ככה:

קוד:

#include <string>
using namespace std;

struct Course {
	int CourseNum;
	string CourseName;

	void course() {
		this->CourseNum=0;
		this->CourseName="Non";
	}

	void course(int CourseNum){
		if (CourseNum<1||CourseNum>6) { course();}
		if (CourseNum==1) { this->CourseName="C"; this->CourseNum=1;}
		if (CourseNum==2) { this->CourseName="Cpp"; this->CourseNum=2;}
		if (CourseNum==3) { this->CourseName="Data_Structure"; this->CourseNum=3;}
		if (CourseNum==4) { this->CourseName="Algorithms"; this->CourseNum=4;}
		if (CourseNum==5) { this->CourseName="Java"; this->CourseNum=5;}
	}

};


struct student {
	string name, address;
	long id;

	struct student(){
		this->id=0;
		this->name=" ";
		this->address=" ";

	}

	struct student(long id, string name,string address){
		this->id=id;
		this->name=name;
		this->address=address;
	}

};



class ClassStudent
{

private:
	int size;
	Course Courses[3];

public:
	struct student st;

	ClassStudent(void);
	ClassStudent(struct student &st);

	string getName() const;
	string getAddress() const;
	long getID() const;

	void setName(string name);
	void printName() const;
	bool setCourse(Course c1, Course c2, Course c3);
	bool delCourse(int CourseNum);
	void printStudent() ;
	int FindStudnt(long id, ClassStudent *sArr, int size);
	void ResetCourses( ClassStudent *sArr, int size);

	struct student operator=(struct student &st){
		struct student temp;
		temp.id = st.id;
		temp.address =st.address;
		temp.name=st.name;
		return temp;
	}

	~ClassStudent(void);
};

זה מתקמפל לי
לגבי הפרסום מסכימה איתך, לא ידעתי שיש את זה , תודה על הערה!
אגב איך מכניסים את זה לקטע קוד? :-9

זה באמת מתקמפל, אבל אני לא חושב שזה עושה את מה שאת חושבת שזה עושה.
את כתבת שם אופרטור השמה ל struct student, אבל הוא נמצא בתוך classStudent, מה שאומר שמבחינת הקומפיילר בעצם כתבת אופרטור השמה ל-classStudent שבכלל מחזיר struct student!

אני לא יודע באיזה תכנה את מקמפלת, אבל כדאי לך מאוד להוסיף את הדגל [tex]-Wall[/tex] כשאת מקמפלת, יש לך שם הרבה אזהרות של הקומפיילר שחלקם די משמעותיות.

בvisual studio 2010
לא הכי הבנתי את ההערה שלך, אני רוצה לשבת יותר על כל עניין של המבנים.יש לי עוד מה ללמוד.
אבל הרצתי את זה , וזה עובד לי טוב. שיחקתי עם זה , זה רץ מצוין.
יש אפשרות שאשלח לך את התוכנית שלי? תריץ אותה, ותגיד אם אני צריכה להוסיף משו בדיסטראקטור, כי על פניו הבנתי ממכם שלא אצטרך להגדיר בו כלום כי לא הקצאתי זכרון בהדר.
אבל אם הקצאתי זיכרון בmain שלי, למערך דינאמי של סטודנטים. אז אני אצטרך להשתמש בדיסטראקטור שישחרר את הזכרון של המערך הזה?

תשלחי לי קישור להורדה של הקובץ ואני אסתכל

קישור לתוכנית
אגב- לוקח לדף קצת זמן לעלות.
תודה!

זו החלטה של המתכנת מה הוא רוצה שה destructor יעשה... אם זה היה משהו מובן מאליו, המערכת הייתה יודעת לעשות את זה לבד...

ההגיון הבריא אומר למחוק כל מה שלא משתמשים בו יותר - כמובן שצריך להתחשב תמיד בכך שייתכן ו"מישהו אחר" מסתכל על אותו מידע (רפרנס שהועבר למקום אחר, למשל) - ואז אסור למחוק אותו בלי לוודא שה"מישהו האחר" לא ינסה לקרוא מרגע זה והלאה... אחרת המידע הזה נשאר "בשימוש", ואם הזיכרון שמכיל אותו ישוחרר בחזרה למערכת ההפעלה, בפעם הבאה שינסו לגשת לאותו מקום, תהיה שגיאת Access Violation / SIGSEGV

נמאס לכם לזכור סיסמאות? לחצו כאן!

כן זיהו הבנתי את זה שזה משחרר את הזכרון לאחר סיום השימוש בו.
רק שבקוד שלי, בקובץ main יש את התפריט עצמו , מה שאתה רוצה לעשות למערך הסטודנטים שהקצאתי שם.
ClassStudent * students=new ClassStudent [studentsNum];
בין אם זה להוסיף סטודנט, לעדכן אצלו קורסים, למחוק קורסים שלו להציג את פרטי הסטודנט.
הבחירה מה שאתה רוצה לעשות זה באמצעות קייסים (לא כזה חשוב).
אז אני צריכה לעשות דיסטראקטור להקצאה students שלי?