Оцените работу, пожалуйста.

[Tobyan]

(Нажмите, чтобы показать/скрыть)

Код: [Выделить]

#include<iostream>
#include<fstream>
#include<sstream>
#include<cmath>
#include<cstdlib>
#include<string>
#include<list>
#include<vector>
#include<algorithm>
#include<stdexcept>
#include<map>
using namespace std;
// Практическая работа по применению основных элементов языка С++
//
int main() {
    vector<double> varcan;
    double arc;
    arc = 0;
    cout << "Пожалуйста, введите число и еденицу измерения и символ '.'.\n";
    cout << "m - метры\ni - дюймы\nf - фунты\ns - сантиметры\n";

    int i = 0;   // индекс последнего числа в векторе для выявления наибольшего значения
    int rex = 0; //Проверка условия true false
    int arc2 = 0;//Значение для сравнения чисел из вектора

    char eden = 'r';    //Еденица измерения
    char end_for = 'r'; //Завершающие символы
    char end_wh = 'r';  //

    const double ft = 12;  // 1 ft == 12 in
    const double in = 2.54;// 1 in == 2.54 cm
    const double cm = 100; // 1 m  == 100 cm

    double sum = 0; //сумма всех значений вектора
    int url = 0;    //индекс числа в векторе для вывода

    while(true){
//----------Начало цикла ввода значений
            while(true) {
//--------------Цикл операций ввода.
                while(true){
//------------------Ввод числа, еденицы измерения, и завершающего символа
                    for(;;){
                        cin>>arc;
                        cin >> eden;
                        cin >> end_for;
                        if (end_for == '.'){
                            break;
                        }
                        else if (end_for != '.'){
                            cout << "Некорректная команда!\n";
                        }
                    }
//----------------------Выбор введенной еденицы измерения, и применение к ней соответствующих опций
                        switch(eden){
                            case('m'):
                               cout << "Число : " << arc << " метров" <<" == "<< arc * cm <<" сантиметров == "
                                    << (arc * cm)/in << " дюймов == "<< ((arc * cm)/in)/ft << " футов." << endl;
                                    rex = 0;
                                    arc = arc;
                            break;
                        case('i'):
                            cout << "Число : " << arc << " дюймов" <<" == "<< arc/ft <<" футов"
                                 << " == "<< arc*in <<"  сантиметров == " << (arc * in)/cm << " метров." << endl;
                                 rex = 0;
                                 arc = (arc * in)/cm;
                        break;
                        case('f'):
                            cout << "Число : " << arc << " футов" <<" == "<< arc*ft <<" дюймов == "
                                 << (arc*ft)*in <<" сантиметров == "<< ((arc*ft)*in)/cm << " метров." << endl;
                                 rex = 0;
                                 arc = ((arc*ft)*in)/cm;
                        break;
                        case('s'):
                           cout << "Число : " << arc << " сантиметров" <<" == "<< arc/cm <<" метров == "
                                << arc/in <<" дюймов == "<< (arc/in)/ft << " Футов." << endl;
                                rex = 0;
                                arc = arc/cm;
                        break;
                        default:
                            cout << "Введено некорректное наименование."<<endl;
                            rex = 1;
                        break;
                    }
                if (rex == 0){
                    break;
                }
            }
//----------Добаление значения в вектор и сортировка значений
            varcan.push_back(arc);
            sort(varcan.begin(),varcan.end());
//----------Выбор и вывод наименьшего или наибольшего значения.
            if (arc < varcan[1]) {
                cout << arc <<" метров : Самое маленькое значение из введенных.\n";
            }
            if (arc > arc2) {
                cout << arc <<" метров : Самое большое значение из введенных.\n";
            }
            arc2 = varcan[i];
//----------Параметрическая информация о размере вектора и числе i
            cout << "Размер варкана "<< varcan.size() << endl;
            cout << "число i : " << i << endl;
            if (varcan.size()> 100) {
                break;
            }
//----------Завершение или продолжения цикла ввода значений.
            cout << "Для завершения программы программы ввода введите символ '|'.\n Для продолжения любой символ. " << endl;
            cin >> end_wh ;
            ++i;
            if (end_wh == '|'){
                    break;
                }
            else {
                cout << "Вы решили продолжить программу :\n";
            }

        }
//------Определение суммы всех значений вектора и вывод их на экран в отсортированном порядке.
        for (int ii = 0;ii < varcan.size();++ii,++url) {
            sum += varcan[url];
        }
        cout << "Cумма всех введенных значений : " << sum <<endl;
        for (int ii = 0;ii < varcan.size();++ii){
            cout << varcan[ii]<< " ";
        }
    }
}



Закончил наконец мурыжиться с задачей. Прошу взглянуть и оценить, как оно вам, а именно в качестве решения. :)Прилагаю файл.цпп.
Если нужно могу написать условия задачи.

[yorik1984]

Если нужно могу написать условия задачи.

Как бы с этого и начинать надо было) откуда знать что ты там решали?

[JustAnother]

отвратительно, с любой точки зрения.

[Tobyan]

Если нужно могу написать условия задачи.

Как бы с этого и начинать надо было) откуда знать что ты там решали?

Обязательно выложу, просто сейчас я не дома и не имею такой возможности.

отвратительно, с любой точки зрения.

Простите, но ваши комментарии неконструктивны. Пожалуйста проявите компетентность и соизвольте сообщить, с каких сторон вы расматривали код и на каких основания сделали данное утверждение. Если влом, то можно было и вовсе не комменрировать. Лень не приветствуется. А насрать, извините, и в толчке можно.

[Lifewalker]

"Пастернака не читал, но осуждаю" - народная мудрость.

Кто вас учил писать так:

Код: [Выделить]

cout << "Пожалуйста, введите число и еденицу измерения и символ '.'.\n";Стукните его в лоб и скажите, что это был я. Если уж берётесь писать на Си++, так следуйте его идеологии. Например, вот так:

Код: [Выделить]

cout << "Пожалуйста, введите число и еденицу измерения и символ '.'." << endl;

[andrew_bye]

Вставлю и свои 5 копеек.

Фраза "Пожалуйста, введите число и еденицу единицу измерения и символ '.'." хороша в поэзии. В прозе же лучше писать: "Пожалуйста, введите число, единицу измерения и символ '.'."

[Yurror]

докопались до опечаток =)
хорошо что не написал

Код: [Выделить]

printf("Vvedite chislo:");это уже радует

ну а вообще, обычное решение обычной студенческой лабы.

[di-erz]

А на сайте нет правила, что учебные задания не рассматриваются?!

[Yurror]

не знаю. прочитай, расскажи.

[Tobyan]

докопались до опечаток =)
хорошо что не написал

Код: [Выделить]

printf("Vvedite chislo:");это уже радует

ну а вообще, обычное решение обычной студенческой лабы.

Особенно если учесть что я вообще еще не студент, и еще даже не факт что им буду. Мне б для начала 11 класс закончить

Вставлю и свои 5 копеек.

Фраза "Пожалуйста, введите число и еденицу единицу измерения и символ '.'." хороша в поэзии. В прозе же лучше писать: "Пожалуйста, введите число, единицу измерения и символ '.'."

Бывают опечатки, чего уж там. Да и насчет многократного "и" (ага, бугога) это я заметил,  но почему то поленился исправить От этого то суть программы не страдает
Пользователь решил продолжить мысль 29 Октября 2011, 21:31:25:

А на сайте нет правила, что учебные задания не рассматриваются?!

Это конечно не моё дело, но было бы довольно глупо вводить такое правило, от этого страдает контент, затем, как следствие, посетительность и затем, как следствие, информативность. Форумы они для того и нужны, что бы снабжать полезной информаций.
 Да и с кем еще посоветоваться, если вокруг тебя люди которые выпучиваю глаза от слова " программирование " нервно вздыхают при "информатике" и ядерно блюют и бьются головой об пол при "математике."
Пользователь решил продолжить мысль 29 Октября 2011, 21:35:53:

"Пастернака не читал, но осуждаю" - народная мудрость.

Кто вас учил писать так:

Код: [Выделить]

cout << "Пожалуйста, введите число и еденицу измерения и символ '.'.\n";Стукните его в лоб и скажите, что это был я. Если уж берётесь писать на Си++, так следуйте его идеологии. Например, вот так:

Код: [Выделить]

cout << "Пожалуйста, введите число и еденицу измерения и символ '.'." << endl;

Это вышло случайно, в связи с мои довольно кривым началом. Сначала я прочел что надо начать с Си, начал с Си. Потом прочел что лучше все же С++ для старта сразу, давай юзать С++. Но тот небольшой фрагментик заний от С случайно отпечатался в головенке и наблюдается привычка юзать "\n.". Акстюсь

[serg..2000]

Критика:
Поверхностный анализ:

(Нажмите, чтобы показать/скрыть)

1.Слишком большая функция main (см. структурное программирование)
2.Нет единообразия:

Код: [Выделить]

..while(true)..
..for(;;)..3.

Код: [Выделить]

              for(;;){
                        cin>>arc;
                        cin >> eden;
                        cin >> end_for;
                        if (end_for == '.'){
                            break;
                        }
                        else if (end_for != '.'){
                            cout << "Некорректная команда!\n";
                        }
                    }можно заменить на (например):

Код: [Выделить]

do{
    cin>>arc>> eden >> end_for;
    if (end_for != '.')  cout << "Некорректная команда!\n";
}while(end_for == '.');
тоже самое    для   if (rex == 0){ break; }
Злоупотребление "break";
4. неиспользование T accumulate( input_iterator start, input_iterator end, T val );

Код: [Выделить]

        for (int ii = 0;ii < varcan.size();++ii,++url) {
            sum += varcan[url];
        }//если url!=0; varscan[url];++url ошибка!!!5. зачем #include<fstream> и прочее
Общее впечатление: все запутано и трудно читается. Возможно с ошибками.

про cout<< endl; cout<<"\n";
endl сбрасывает буфер, а cout<<"\n" нет. Поэтому их стоит различать и использовать там где нужно.

[andrey_p]

Дуй дальше, не зацикливайся сильно на (ПОКА!) на качестве кода. Учебные программы все равно мало что имеют общего с настоящими.

ЗЫ Stdout по умолчанию буферизируется построчно в стандартной С библиотеке. В С++ по другому? 
Или это POSIX? Или это не стандарт вообще?

[JustAnother]

Простите, но ваши комментарии неконструктивны. Пожалуйста проявите компетентность и соизвольте сообщить, с каких сторон вы расматривали код и на каких основания сделали данное утверждение. Если влом, то можно было и вовсе не комменрировать. Лень не приветствуется. А насрать, извините, и в толчке можно.

• Огромный метод Main
• Константами заменяются магические числа для того, что бы было понятно, что это такое, но название констант ft, in, cm жизнь не облегчает
• Впрочем как и названия остальных переменных
• При вводе единицы измерения пользователь не знает что вводит
• Алгоритм выхода из цикла по if (rex == 0). Неужели по другому никак?
• Огромный метод Main

[serg..2000]

Дуй дальше, не зацикливайся сильно на (ПОКА!) на качестве кода. Учебные программы все равно мало что имеют общего с настоящими.

Да. Но я бы переписал программу - переделал циклы. Как-то ненормально:
while(true){
...код...
if(isBreak) break;
}
Один час разобраться с циклами (понять для чего while, for, while(true), переписать одно и тоже разными способами ), и это  сэкономит массу времени и строк в будущем. А главное - программы станут проще.

ЗЫ Stdout по умолчанию буферизируется построчно в стандартной С библиотеке. В С++ по другому? 
Или это POSIX? Или это не стандарт вообще?

У Страуструпа endl - запись '\n' и очистка
у меня в ostream

Код: [Выделить]

  template<typename _CharT, typename _Traits>
    inline basic_ostream<_CharT, _Traits>& endl(basic_ostream<_CharT, _Traits>& __os)
    { return flush(__os.put(__os.widen('\n'))); }Хотя выигрыша у меня нет, но кто знает как на других системах.   
ЗЫ Занятная опечатки - метры в фунты переводить.