Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.75/4: Рейтинг темы: голосов - 4, средняя оценка - 4.75
232 / 232 / 69
Регистрация: 30.09.2012
Сообщений: 545

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

20.07.2013, 17:36. Показов 777. Ответов 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
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
20.07.2013, 17:36
Ответы с готовыми решениями:

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

Проверьте код
#include &lt;iostream&gt; #include &lt;cmath&gt; #include &lt;iomanip&gt; using namespace std; int main() { long long n,z; ...

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

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

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

например, захочешь этот класс дать своему другу, то тогда придется передавать вместе с этой глобальной переменной
0
1 / 1 / 0
Регистрация: 30.01.2013
Сообщений: 28
20.07.2013, 18:25
заноси ее в класс и делай статической
0
Неэпический
 Аватар для Croessmah
18149 / 10731 / 2067
Регистрация: 27.09.2012
Сообщений: 27,035
Записей в блоге: 1
20.07.2013, 18:34
C++
1
vector< std::pair < unsigned int , std::string > > MyNameIsBoris;
0
232 / 232 / 69
Регистрация: 30.09.2012
Сообщений: 545
20.07.2013, 18:47  [ТС]
Да, помню, что говорили что глобальные переменные это зло. Переправил, работает корректно.
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name; Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
0
 Аватар для name?
201 / 172 / 52
Регистрация: 01.06.2010
Сообщений: 371
20.07.2013, 18:54
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
Цитата Сообщение от Chernobyl Посмотреть сообщение
Croessmah С трудом понимаю то,что вы написали. Это вместо vector<int> age;
vector<string> name;
Да, это вместо...
Цитата Сообщение от Chernobyl Посмотреть сообщение
Вы обьявляете один вектор у которого элементы типов интеджер и стринг?
Вектор элементами которого являются pair (пара значений).
Вам предлагают хранить имя и возраст вместе: pair<unsigned int , string>.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
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 b; printf(&quot;Vvedit b\n&quot;); ...

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

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

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


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

Или воспользуйтесь поиском по форуму:
8
Ответ Создать тему
Новые блоги и статьи
Ритм жизни
kumehtar 27.02.2026
Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
[В процессе разработки] SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten
8Observer8 27.02.2026
Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru