Установка данных одного класса из другого класса

Question

Я пытаюсь установить посещаемость студентов на основе ввода пользователем. Там, где я запускаю его, проблема пытается принять этот вход и назначить его моим данным для каждого учащегося.

Все данные, которые у меня есть, считываются из файла в отдельный список. Структура этого типа данных объявлена ​​в моем узле.h.

Так что я спрашиваю, как изменить значения, которые происходят из класса Node, в моем классе меню?

Menu.cpp

void Menu::chooseMenu(ifstream *input, List L1) 
{ 
    data temp; 
    //data plswork[12]; 
    char holder[30]; 
    char junk[30]; 
    int i=1,j=1; 

    do{ 
    system("cls"); 
    cout<<"1. Import"<<endl; 
    cout<<"2. Load Master"<<endl; 
    cout<<"3. Store Master"<<endl; 
    cout<<"4. Mark Absence"<<endl; 
    cout<<"5. Generate Report"<<endl; 
    cout<<"6. Exit"<<endl; 
    cout<<""<<endl; 
    cin>>selection; 

    if(selection == 1)//import 
    { 
     while(!input->eof()) 
     { 
      memset(holder,0,30); 

      if(i==3 && j != 1) 
      { 
       input->getline(holder,30,'"'); 
       input->getline(holder,30,'"'); 
       input->getline(junk,30,','); 
      } 
      else 
      { 
       input->getline(holder,30,','); 
      } 

      if(i==1) 
      { 
       strcpy(temp.record, holder); 
       //cout<<temp.record<<endl; 
       i++; 
      } 
      else if(i==2) 
      { 
       strcpy(temp.ID, holder); 
       //cout<<temp.ID<<endl; 
       i++; 
      } 
      else if(i==3) 
      { 
       strcpy(temp.name, holder);//read between quotes 
       //cout<<temp.name<<endl; 
       i++; 
       j=2; 
      } 
      else if(i==4) 
      { 
       strcpy(temp.email, holder); 
       //cout<<temp.email<<endl; 
       i++; 
      } 
      else if(i==5) 
      { 
       strcpy(temp.units, holder); 
       //cout<<temp.units<<endl; 
       i++; 
      } 
      else if(i==6) 
      { 
       strcpy(temp.major, holder); 
       //cout<<temp.major<<endl; 
       i++; 
      } 
      else if(i==7) 
      { 
       strcpy(temp.grade, holder); 

       //cout<<temp.grade<<endl; 
       L1.insertOrder(temp); 
       i=1; 
      } 

     } 
    } 

    else if(selection == 2)//load master 
    { 

    } 

    else if(selection == 3)//store master 
    { 

    } 

    else if(selection == 4)//mark absence 
    { 
     while(L1.nextPtr() != NULL) 
     { 
     char name; 
      cout<<"Bruce :"<<L1.getDataL().absent<<endl; 
      cout<<"Is "<<L1.getDataL().name<<" present? (Y/N)"<<endl; 
      cin>>name; 
      if(name == 'y' || name == 'Y') 
      { 
       L1.setAtt(L1.getDataL(),1); // This is where I try to set attendance to 0 or 1. 
       cout<<L1.getDataL().absent<<endl; 
      } 
      else 
      { 
       L1.setAtt(L1.getDataL(),0); 
      } 

      L1.nextPtr(); 
      system("pause"); 
     } 
    } 

    else if(selection == 5)//gen report 
    { 

    } 

    }while(selection != 6); 

} 

node.cpp

#include "Node.h" 

ListNode::ListNode (ListNode &copyObject) 
{ 
    this->mData = copyObject.mData; 
    this->mpNext = copyObject.mpNext; 
} 

ListNode::~ListNode() 
{ 
    // does nothing 
    cout << "exiting listnode object - going out of scope" << endl; 
} 

ListNode * ListNode::getNextPtr() const 
{ 
    return mpNext; 
} 

data ListNode::getData() const 
{ 
    return mData; 
} 

ListNode::ListNode(data newData) 
{ 
    mData = newData; 
    this->mpNext = NULL; 
} 

list.h

#include "Node.h" 

#ifndef LinkedList_H 
#define LinkedList_H 
using std::ostream; 

class List 
{ 
    friend ostream & operator << (ostream &lhs, List &rhs); 
    friend ListNode; 

    public: 
     List(); 
     List (List &copyObject); 
     ~List(); 

     List & operator = (List &rhs); 

     ListNode *makeNode (data newNode); 
     bool insertOrder (data Node); 
     bool deleteNode (data Node); 
     data getDataL(); 
     ListNode *nextPtr(); 
     void setAtt(int atten); 




    private: 
     ListNode *pHead; 
}; 
#endif 

ListNode.h

#ifndef ListNode_H 
#define ListNode_H 

#include <iostream> 
#include <string> 

using std::cin; 
using std::cout; 
using std::endl; 
using std::string; 

struct data 
    { 
     char record[8]; 
     char ID[12]; 
     char name[30]; 
     char email[30]; 
     char units[6]; 
     char major[8]; 
     char grade[14]; 
     int absent; 
     char dateabsent[32]; 
    }; 


class List; 


class ListNode 
{ 
    friend class List; 
    friend struct data; 
    public: 
     ListNode(); 
     ListNode(data newData); 
     //ListNode(char newrecord[], string newID, string newname, string newemail, string newunits, string newmajor, string newgrade); 
     ListNode(ListNode &copyObject); 
     ~ListNode(); 

     data getData() const; 
     ListNode *getNextPtr() const; 

     ListNode & operator = (ListNode &rhs); 

    private: 
     data mData; 
     ListNode *mpNext; 
}; 
#endif 

Answer 1

Я понятия не имею, как ваш setAtt метод на List предполагается работать. Таким образом, без использования setAtt, одним из решений является сделать getDataL return data * (указатель на структуру данных). Теперь ваш главный хранит указатель на некоторые данные в связанном списке и может изменять его по мере необходимости.

Answer 2

Итак, у вас будет очень трудное время получить хорошие ответы на этот код, потому что ваша структура данных не очень хорошо построена, вам нужен только один класс, возможно, два, чтобы сделать это. Пыльник это два объявление класса:

struct data 
{ 
    char record[8]; 
    char ID[12]; 
    char name[30]; 
    char email[30]; 
    char units[6]; 
    char major[8]; 
    char grade[14]; 
    int absent; 
    char dateabsent[32]; 
}; 
class listNode 
{ 
    listNode* next; 
    data data; 

public: 
    listNode(data& d); //Will copy d into data, you will need to define 
    listNode();   //Will initialize a blank data, you will need to define 
    ~listNode();   //Definitely need a destructor 

    data& getMutableData(); 
    const data& getImutableData() const; 

    void insert(listNode* n); 
    bool delete(listNode* n); 
    listNode* next(); 

}; 

Класс обертки у вас есть на самом деле нет необходимости для связного списка, и вы хотите передать структуры больших данных либо в качестве указателей или по ссылке, вы никогда не должны возвращать «данных», только «данные *» и «данные &». «данные» должны быть зарезервированы только для создания новых объектов. Реализация несколько важных функций здесь:

void listNode::insert(listNode* n) 
{ 
    if(next!=NULL) 
     next->insert(n); 
    else 
     next = n; 
} 
bool listNode::delete(listNode* n) 
{ 
    if(next==NULL) 
     return false; 
    if(next == n) 
    { 
     listNode* temp = next; 
     next = next->next(); 
     delete temp; 
     return true; 
    } 
    return next->delete(n); 
} 

Причина у вас возникли проблемы, в основном, что ваши структуры данных очень слабо реализованы, они нуждаются в редизайн. То, что у меня выше, ни в коем случае не является единственной или даже самой лучшей реализацией (в частности, существует небольшая инкапсуляция или защита данных), но, похоже, она близка к функциональности с односвязным списком, которую вы ищете.