Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.50/2: Рейтинг темы: голосов - 2, средняя оценка - 4.50
224 / 224 / 69
Регистрация: 30.09.2012
Сообщений: 545
1

Проверьте код

20.07.2013, 17:36. Просмотров 470. Ответов 7
Метки нет (Все метки)

Не очень хорошо пошли мне клас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;
}
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
20.07.2013, 17:36
Ответы с готовыми решениями:

Написать код для игры НИМ. проверьте пожалуйста тот ли это код
#include &lt;string&gt; using namespace std; int compChoose(int markers); int userChoose(int...

Проверьте код
#include &lt;iostream&gt; #include &lt;cmath&gt; #include &lt;iomanip&gt; using namespace std; ...

Проверьте код
#include &lt;stdio.h&gt; #include &lt;math.h&gt; #include &lt;conio.h&gt; int main() { int a,b,res; ...

Проверьте код
Есть задача. Её нужно решить в консольном приложении. Написал к ней код, ну не вычисляет и всё...

7
68 / 41 / 1
Регистрация: 14.05.2013
Сообщений: 383
20.07.2013, 18:10 2
Код написан нормально, ошибок вроде бы нет, но мне кажется что библиотека cmath тут ни к чему
Не знаю так ли это, но всё же...)

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

например, захочешь этот класс дать своему другу, то тогда придется передавать вместе с этой глобальной переменной
0
1 / 1 / 0
Регистрация: 30.01.2013
Сообщений: 28
20.07.2013, 18:25 4
заноси ее в класс и делай статической
0
Грамотный. Безпорно.
16841 / 9738 / 1875
Регистрация: 27.09.2012
Сообщений: 24,139
Записей в блоге: 2
20.07.2013, 18:34 5
C++
1
vector< std::pair < unsigned int , std::string > > MyNameIsBoris;
0
224 / 224 / 69
Регистрация: 30.09.2012
Сообщений: 545
20.07.2013, 18:47  [ТС] 6
Да, помню, что говорили что глобальные переменные это зло. Переправил, работает корректно.
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name; Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
0
201 / 172 / 52
Регистрация: 01.06.2010
Сообщений: 371
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;
}
1
78 / 60 / 9
Регистрация: 26.06.2013
Сообщений: 198
20.07.2013, 21:11 8
Цитата Сообщение от Chernobyl Посмотреть сообщение
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name;
Да, это вместо...
Цитата Сообщение от Chernobyl Посмотреть сообщение
Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
Вектор элементами которого являются pair (пара значений).
Вам предлагают хранить имя и возраст вместе: pair<unsigned int , string>.
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
20.07.2013, 21:11

Заказываю контрольные, курсовые, дипломные и любые другие студенческие работы здесь.

Проверьте код
# include &lt;stdio.h&gt; void main() { int a; printf(&quot;Vvedit a\n&quot;); scanf(&quot;%d&quot;, &amp;a); int...

Проверьте код, не компилируется
Говорит: &quot; 'S' cannot be used as a function &quot;. Для х, изменяющегося от a до b с шагом h=(b-a)/10...

Проверьте маленький код
Хотел создать тему по поводу ошибки, но когда писал то нашел решение) #include &lt;iostream&gt; double...

Кодеры, проверьте код!
Задание: отсортировать массив из 5 элементов методом выбора. #include &lt;cstdlib&gt; #include...


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

Или воспользуйтесь поиском по форуму:
8
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2020, vBulletin Solutions, Inc.