Форум программистов, компьютерный форум CyberForum.ru

Проверьте код - C++

Восстановить пароль Регистрация
 
Chernobyl
211 / 211 / 67
Регистрация: 30.09.2012
Сообщений: 526
20.07.2013, 17:36     Проверьте код #1
Не очень хорошо пошли мне класcы, да и сейчас не уверен все хорошенько ли понял.
Так вот было задание простое. Принять имена на вход, потом возрасты и вывести пары.Но реализовать это с помощью класса. Программу я написал и она делает то, что нужно, но мне вот кажеться, что код написан чрезвычайно плохо.
Посмотрите и оцените знающим глазом, нормальный ли код либо паршивый.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
# include <iostream>
# include <string>
# include <vector>
# include <algorithm>
# include <cmath>
# include <windows.h>
#include <fstream>
using namespace std;
int n0;
 
class Pairs{ 
private:
    vector<int> age;
    vector<string> name;
 
public:
void read_names(){
    cout<<"Vvedite imena"<<endl;
    for(int i=1;i<=n0;i++)
    {
        cout<<i<<" chelovek"<<endl;
        string names;
        cin>>names;
        name.push_back(names);
    }
 
}
void read_ages(){
    cout<<"Vvedite vozrasta"<<endl;
        for(int i=1;i<=n0;i++)
    {
        cout<<i<<" chelovek"<<endl;
        int ages;
        cin>>ages;
        age.push_back(ages);
    }
}
void show_all(){
    for(int i=0;i<n0;i++)
    cout<<name[i]<<" - "<<age[i]<<endl;
}
};
 
int main()
{
SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    
    cout<<"Skoko vvedem?"<<endl;
    cin>>n0;
    Pairs man;
    man.read_names();
    man.read_ages();
    man.show_all();
    system("pause");
    return 0;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
20.07.2013, 17:36     Проверьте код
Посмотрите здесь:

Структура, проверьте код C++
Проверьте код C++
Проверьте код C++
Проверьте код программы C++
C++ Проверьте пожалуйста код
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Даниил
67 / 40 / 7
Регистрация: 14.05.2013
Сообщений: 383
20.07.2013, 18:10     Проверьте код #2
Код написан нормально, ошибок вроде бы нет, но мне кажется что библиотека cmath тут ни к чему
Не знаю так ли это, но всё же...)

Добавлено через 37 секунд
Просто не вижу тут каких-то сложных математических действий)
BigLow
55 / 55 / 2
Регистрация: 07.07.2013
Сообщений: 345
20.07.2013, 18:20     Проверьте код #3
в классе используется глобальная переменная
Цитата Сообщение от Chernobyl Посмотреть сообщение
int n0;
это не очень хорошо.

например, захочешь этот класс дать своему другу, то тогда придется передавать вместе с этой глобальной переменной
alig007
1 / 1 / 0
Регистрация: 30.01.2013
Сообщений: 28
20.07.2013, 18:25     Проверьте код #4
заноси ее в класс и делай статической
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11845 / 6824 / 771
Регистрация: 27.09.2012
Сообщений: 16,919
Записей в блоге: 2
Завершенные тесты: 1
20.07.2013, 18:34     Проверьте код #5
C++
1
vector< std::pair < unsigned int , std::string > > MyNameIsBoris;
Chernobyl
211 / 211 / 67
Регистрация: 30.09.2012
Сообщений: 526
20.07.2013, 18:47  [ТС]     Проверьте код #6
Да, помню, что говорили что глобальные переменные это зло. Переправил, работает корректно.
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name; Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
name?
 Аватар для name?
198 / 169 / 18
Регистрация: 01.06.2010
Сообщений: 368
Завершенные тесты: 1
20.07.2013, 18:54     Проверьте код #7
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
# include <iostream>
# include <string>
# include <vector>
# include <windows.h>
 
using namespace std;
 
class Human{
private:
    int age;
    string name;
 
public:
    Human(string name, int age){
        this->name = name;
        this->age = age;
    }
 
    string get_Name(){
        return name;
    }
 
    int get_Age(){
        return age;
    }
 
void print(){
    cout<<name<<" - "<<age<<endl;
    }
};
 
int main()
{
SetConsoleCP(1251);
    SetConsoleOutputCP(1251);
    vector<Human> humans;
    cout<<"Skoko vvedem?"<<endl;
    int n;
    cin>>n;
    for(int i=0; i < n; i++){
        int age;
        string name;
        cout<<"Vvedite imya: ";
        cin>>name;
        cout<<"Vvedite vozrast: ";
        cin>>age;
        humans.push_back(Human(name,age));
    }
 
    for(int i=0; i < n; i++)
        humans[i].print();
 
    system("pause");
    return 0;
}
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
20.07.2013, 21:11     Проверьте код
Еще ссылки по теме:

Написать код для игры НИМ. проверьте пожалуйста тот ли это код C++
Проверьте код C++
C++ Проверьте код пожалуйста

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
nxtech
77 / 59 / 2
Регистрация: 26.06.2013
Сообщений: 198
20.07.2013, 21:11     Проверьте код #8
Цитата Сообщение от Chernobyl Посмотреть сообщение
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name;
Да, это вместо...
Цитата Сообщение от Chernobyl Посмотреть сообщение
Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
Вектор элементами которого являются pair (пара значений).
Вам предлагают хранить имя и возраст вместе: pair<unsigned int , string>.
Yandex
Объявления
20.07.2013, 21:11     Проверьте код
Ответ Создать тему
Опции темы

Текущее время: 00:21. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru