2016-04-26 3 views
-2

Я делаю проект, в котором пользователь добавит информацию о машине, и одна функция покажет все доступные автомобили. Но я не могу копировать имена автомобилей с одного массив для другого здесь - это код.Динамический массив символов из класса не печатает нужный результат

#include <fstream> 
#include <iostream> 
#include <stdio.h> 
#include <conio.h> 
#include <string.h> 
using namespace std; 

const int SIZE=10; 

class car 
{ 
    public: 
int car_no=0; 
char car_name[SIZE][20]; 
char car_model[SIZE][20]; 
char car_colour[SIZE][20]; 
char reg_id[SIZE][20]; 
int rate_per_day[SIZE]; 
char cars_in_lot[SIZE][20]; 
}; 

addcar() 
{ 
    char choice; 
    do{ 

    car*newcar=new car; 

    cout<<"\t\t\t\t(->) Name : "; 
    cin>>newcar->car_name[newcar->car_no]; 
    strcpy(newcar->cars_in_lot[newcar->car_no],newcar->car_name[newcar->car_no]); 
    cout<<endl; 
    cout<<"\t\t\t\t(->) [email protected]!0n number : "; 
    cin>>newcar->reg_id[newcar->car_no]; 
    cout<<endl; 
    cout<<"\t\t\t\t(->) c0l0ur : "; 
    cin>>newcar->car_colour[newcar->car_no]; 
    cout<<endl; 
    cout<<"\t\t\t\t(->) Model : "; 
    cin>>newcar->car_model[newcar->car_no]; 
    cout<<endl; 
    cout<<"\t\t\t\t(->) Rate of Rent Per Day : "; 
    cin>>newcar->rate_per_day[newcar->car_no]; 
    newcar->car_no++; 
    cout<<"Want to add another car [y/n]"<<endl; 
    cin>>choice; 
    }while(choice == 'y' || choice == 'Y'); 

} 
void show_cars_in_lot() 
{ 
     cout<<"Avaialable Cars in Lot are : "<<endl; 
     car*newcar=new car; 
     for(int i=0;i<SIZE;i++) 
     cout<<i+1<<"\t"<<newcar->cars_in_lot[i]<<"\t"<<newcar->car_colour[i]  <<"\t"<<newcar->car_model[i]<<"\t"<<newcar->reg_id[i]<<endl; 
getch(); 
} 

void display1() 
    { 

cout<<"\t\t\t(1) Show Cars in Lot"<<endl; 
cout<<"\t\t\t(2) Add Cars"<<endl; 
}// end of display1 

int main() 
{ 
    char option; 
    int desire_car; 
do 
{ 
    int choice; 
    display1(); 
    cout<<"Enter Your Choice : "; 
    cin>>choice; 
    switch(choice) 
    { 
    case 1: 
     show_cars_in_lot();break; 
    case 2: 
     addcar();break; 
    default: 
    cout<<"You Entered Wrong Input"<<endl; 
    } 
cout<<"Go To Main Menu [y/n]";cin>>option; 
}while(option == 'y' || option == 'Y'); 

return 0; 
    } 
+5

Используйте 'std :: string', чтобы сделать вашу жизнь проще. –

+2

Также используйте отладчик, чтобы сделать вашу жизнь еще проще. – edmz

+0

Потому что вы добавляете автомобили к одному объекту ('car * newcar = new car;' в 'addcar()'), но пытаетесь отобразить из другого ('car * newcar = new car;' в 'show_cars_in_lot()'). Используйте 'new' только для выделения памяти для нового экземпляра. Вероятно, вам не нужно использовать 'new' в' show_cars_in_lot() ' – kay27

ответ

0

Я рекомендую вам изменить свою структуру данных на контейнер свойств автомобиля, а не на единую структуру массивов.

struct Car_Properties 
{ 
    public: 
    int   number; 
    std::string name; 
    std::string model; 
    std::string colour; 
    std::string reg_id; 
    int   rate_per_day; 
    std::string cars_in_lot; 
}; 
std::vector<Car_Properties> car_inventory; 
// Or 
Car_Properties car_lot[SIZE]; 

В целом вышеизложенное более эффективно, поскольку свойства автомобиля (для одного автомобиля) смежны в памяти.

Примечание: При использовании std::vector, нет необходимости использовать оператор new. std::vector сделает копию. Вы захотите реализовать конструктор копирования и операторы присваивания. Напоминание: также операторы перегрузки == и <.

Edit 1: Создание Car_Properties динамически
Самый простой способ, это просто объявить экземпляр переменной:

int main() 
{ 
    Car_Properties das_auto; // This creates an instance in local memory. 
    //... 
    return EXIT_SUCCESS; 
} 

Если вы хотите прочитать свойства автомобиля из файла, вы можете перегрузить объект для чтения своих членов:

struct Car_Properties 
{ 
    public: 
    void load_by_prompting(std::istream& input, 
          std::ostream& output); 
    //... 
} 

void 
Car_Properties :: 
load_by_prompting(std::istream& input, 
        std::ostream& output) 
{ 
    output << "Enter car number: "; 
    input >> number; 
    output << "Enter car name: "; 
    std::getline(input, name); 
    // ... 
} 

int main() 
{ 
    Car_Properties das_auto; 

    // Input data into the variable 
    das_auto.load_by_prompting(std::cin, std::cout); 

    return EXIT_SUCCESS; 
} 
+0

не могли бы вы рассказать мне, как я могу сделать объект автомобиля динамически –

+0

@AhsanAli: См. Мой раздел 1. –