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

Как усовершенствовать программу? - C++

Восстановить пароль Регистрация
 
555shiro
 Аватар для 555shiro
8 / 8 / 1
Регистрация: 01.12.2009
Сообщений: 226
07.12.2011, 01:40     Как усовершенствовать программу? #1
Доброго времени суток!

Начал изучать с++,написал простенькую прогу чтобы попрактиковаться,хотелось бы узнать как можно ещё более усовершенствовать прогу или усложнить немного?

Программа вычисляет максимальное и минимальное число,причём пользователь может выбирать сколько чисел он хочет сравнить и показывает какие числа он вводил.

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
#include<iostream.h>
#include<conio.h>
//---------------------------------------------------------------------------
 
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
int main(int argc, char* argv[])
{      int a[100];
      int min;
      int t;
      int n;
     while(t!=1)
     {
     int max=0;
    cout<<"--------skolko chisel vi hotite sravnit?--------"<<endl;
    cin>>n;
   cout<<"vvedite chisla :"<<endl;
   for(int i=0;i<n;i++)
   cin>>a[i];
   cout<<"vi vveli chisla: ";
   for(int i=0;i<n;i++)
   cout<<a[i]<<" ";
   cout<<endl;
     min=a[0];
   for(int i=0;i<n;i++)
   if(a[i]<min)    //Г*Г*õîæäåГ*ГЁГҐ min
       {
           min=a[i];
        }
    for(int i=0;i<n;i++)
 if(a[i]>max)       // Г*Г*õîæäåГ*ГЁГҐ max
   {
   max=a[i];
   }
    cout<<"max chislo= "<<max<<" "<<", min chislo= "<<min;
    cout<<endl;
    cout<<"dlya prodolgenia raboty press-0, dlya exit-1"<<endl;
    cin>>t;
    if(t==1)
    {
    exit(0);
    }
    }
          getch();
        return 0;
 
}
//---------------------------------------------------------------------------
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.12.2011, 01:40     Как усовершенствовать программу?
Посмотрите здесь:

C++ усовершенствовать код
немного усовершенствовать... C++
Предлагаю людям как усовершенствовать IDE Dev-Cpp 4.9.9.2 C++
Усовершенствовать код C++
C++ усовершенствовать граф редактор
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Bers
Заблокирован
07.12.2011, 01:46     Как усовершенствовать программу? #2
Цитата Сообщение от 555shiro Посмотреть сообщение
хотелось бы узнать как можно ещё более усовершенствовать прогу или усложнить немного?
например - писать по-русски вместо всяких "dlya prodolgenia raboty press-0, dlya exit-1"
alkagolik
 Аватар для alkagolik
1510 / 616 / 79
Регистрация: 15.07.2011
Сообщений: 3,552
07.12.2011, 01:53     Как усовершенствовать программу? #3
555shiro, для начала приведите в читабельный вид. не скупитесь на пробелы, табы и перевод строки. Например, объявили все переменные \перевод на 2 строки, отработаем блок \перевод на 2 строки. Это чтобы разделять программу на условные "блоки" по функционалу. А то читать невозможно, а копипастить и выравнивать лень.
Paporotnik
383 / 227 / 7
Регистрация: 06.07.2011
Сообщений: 512
07.12.2011, 02:38     Как усовершенствовать программу? #4
если уж строго судить, то ты пишешь на С++ в С-стиле. это конечно не страшно, но и не совсем верно.
воспользуйся ООП. создай класс, отвечающий за абстракцию списка чисел. в его методах можно производить проверку на то, что вводятся действительно числа и эти числа влезут в определенный тобой int. кроме того можно убрать глупое ограничение в максимальную размерность списка (100).
поиск максимального и минимального можно организовать либо методами класса, что проще всего. Либо создать дружественные ф-ции поиска максимальных и минимальных элементов, что более правильно.
Для общения с пользователем (вывод всяких сообщений и т.п.) тоже можно создать отдельный класс. Но это уже посложнее.

ну, и форматирование кода тоже соблюдайте.
555shiro
 Аватар для 555shiro
8 / 8 / 1
Регистрация: 01.12.2009
Сообщений: 226
07.12.2011, 10:40  [ТС]     Как усовершенствовать программу? #5
Цитата Сообщение от alkagolik Посмотреть сообщение
555shiro, для начала приведите в читабельный вид. не скупитесь на пробелы, табы и перевод строки. Например, объявили все переменные \перевод на 2 строки, отработаем блок \перевод на 2 строки. Это чтобы разделять программу на условные "блоки" по функционалу. А то читать невозможно, а копипастить и выравнивать лень.
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
58
59
60
61
62
63
64
#include<iostream.h>
#include<conio.h>
//---------------------------------------------------------------------------
 
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#pragma argsused
int main(int argc, char* argv[])
{  
      int a[100];
      int min;
      int t;
      int n;
 
 
      while(t!=1)
               {
                   int max=0;
 
                   cout<<"--------skolko chisel vi hotite sravnit?--------"<<endl;
                    cin>>n;
                     cout<<"vvedite chisla :"<<endl;
 
 
                     for(int i=0;i<n;i++)
                      cin>>a[i];
                      cout<<"vi vveli chisla: ";
 
                        for(int i=0;i<n;i++)
                        cout<<a[i]<<" ";
                         cout<<endl;
 
                          min=a[0];
                        for(int i=0;i<n;i++)
                          if(a[i]<min)                        //нахождение min
                              {
                                  min=a[i];
                              }
 
 
                               for(int i=0;i<n;i++)
                             if(a[i]>max)                        // нахождение max
                                   {
                                            max=a[i];
                                    }
 
 
                            cout<<"max chislo= "<<max<<" "<<", min chislo= "<<min;
                              cout<<endl;
                            cout<<"dlya prodolgenia raboty press-0, dlya exit-1"<<endl;
                              cin>>t;
 
                              if(t==1)
                                  {
                                       exit(0);
                                   }
              }
          getch();
        return 0;
 
}
//---------------------------------------------------------------------------
Код теперь читаем?
amor1k
Студент
 Аватар для amor1k
147 / 147 / 24
Регистрация: 18.01.2011
Сообщений: 469
07.12.2011, 13:59     Как усовершенствовать программу? #6
а если все числа будут отрицательны? то ваш максимум среди них будет 0. хотя бы так сделайте
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
#include<iostream>
#include<conio.h>
using namespace std;
int main(int argc, char* argv[])
{  
    int a[100];
    int min;
    int t;
    int n;
    while(t!=1)
    {
        cout<<"--------skolko chisel vi hotite sravnit?--------"<<endl;
        cin>>n;
        cout<<"vvedite chisla :"<<endl;
 
        for(int i=0;i<n;i++)
            cin>>a[i];
        cout<<"vi vveli chisla: ";
        for(int i=0;i<n;i++)
            cout<<a[i]<<" ";
        cout<<endl;
        min=a[0];
        for(int i=0;i<n;i++)
        {
            if(a[i]<min)                        //нахождение min
            {
                min=a[i];
            }
        }
        int max = a[0];
        for(int i=0;i<n;i++) {
            if(a[i]>max)                        // нахождение max
            {
                max=a[i];
            }
        }
        cout<<"max chislo= "<<max<<" "<<", min chislo= "<<min;
        cout<<endl;
        cout<<"dlya prodolgenia raboty press-0, dlya exit-1"<<endl;
        cin>>t;
 
        if(t==1)
        {
            exit(0);
        }
    }
    getch();
    return 0;
 
}
NNN777
299 / 157 / 19
Регистрация: 27.02.2010
Сообщений: 317
07.12.2011, 14:23     Как усовершенствовать программу? #7
555shiro, а если, когда будет выделена память под t в выделяемой ячейке случайно будет записано значение 1...
C++
1
2
3
4
int t;
        int n;
        while(t!=1)
        { //......
ваша программа даже не отработает...Проинициализируйте t явно
C++
1
2
3
4
int t = 0;
        int n;
        while(t!=1)
        { //......
Хотите усложнить?.. Попробуйте для начала разбить программу на функции (поиск min и поиск max)...
Bers
Заблокирован
07.12.2011, 17:27     Как усовершенствовать программу? #8
Цитата Сообщение от NNN777 Посмотреть сообщение
Хотите усложнить?.. Попробуйте для начала разбить программу на функции (поиск min и поиск max)...
Дробление задачи на под-задачи, эт не усложнение, а понижение сложности алгоритмов.
К тому же, повышает читабельность.
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
07.12.2011, 23:47     Как усовершенствовать программу?
Еще ссылки по теме:

C++ как усовершенствовать студ.актив при помощи С++
C++ Усовершенствовать метод пузырьковой сортировки

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

Или воспользуйтесь поиском по форуму:
555shiro
 Аватар для 555shiro
8 / 8 / 1
Регистрация: 01.12.2009
Сообщений: 226
07.12.2011, 23:47  [ТС]     Как усовершенствовать программу? #9
Цитата Сообщение от amor1k Посмотреть сообщение
а если все числа будут отрицательны? то ваш максимум среди них будет 0. хотя бы так сделайте
С отрицательными числами тоже всё правильно работает...или я просто не понял суть твоего предложения..

Добавлено через 11 минут
Цитата Сообщение от Paporotnik Посмотреть сообщение
кроме того можно убрать глупое ограничение в максимальную размерность списка (100).
C++
1
a[100];
попробовал сделать так..чтобы пользователь сам мог задавать размерность...
C++
1
2
3
4
5
6
int main(int argc, char* argv[])
{
     int size;
     cin>>size;
      int a[size];   
}
но компилятор выдаёт ошибку типа..константе требуется выражение...
Yandex
Объявления
07.12.2011, 23:47     Как усовершенствовать программу?
Ответ Создать тему
Опции темы

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