Подскажите пожалуйста какие ошибки здесь есть - или что не соблюдено.

@Aliru777 · Регистрация: 03.11.2011

Доброе утро!
Я начинаю только осваивать программирование.Читаю - решаю задачи - пытаюсь разбираться в ошибках.
Написал программу с циклами - идея такая - вводят оценки от 0 до 100 включительно,затем высчитывают средний балл.Если хотят закончить ввод вводят -1.
Я постарался исключить все проблемы из серии: ввели число отрицательное или больше 100,деление на 0...
Знаю что не соблюдены "правила хорошего програмирования" - буду стараться когда сильнее освою базу.
Что кроме этого сделано в программе неправильно или так что можно было бы сделать легче или быстрее?
Очень буду рад всем советам.

Код.

C++

#include "stdafx.h"
#include <conio.h>
#include <math.h>
#include <iostream>
 
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
{
    float total=0, mean;
    int grade,count=0;
    cout<<"Enter grade\n";
    cin>>grade;
    if (grade == -1)
    {
    cout<<"there are no grades!\n";
    }
    else 
    {
    if (grade <= 100 && grade>=0)
    {
    for ( ; grade != -1; )
    {
    total+=grade;
    count++;
    {
    do
    {
    cout<<"ENTER grade from 0 to 100 or -1 to END\n";
    cin>>grade;
    }
    while ((grade > 100 || grade < 0) && grade != -1 );
    }
    }
    }
    else
    {
    for ( ; ; )
    {
    do
    {
    cout<<"enter grade from 0 to 100 or -1 to END\n";
    cin>>grade;
    }
    while ((grade > 100 || grade < 0) && grade != -1 );
    if (grade == -1)
    break;
    total+=grade;
    count++;
    }
    }
    mean=total/count;
    cout<<total<<"\n";
    cout<<count<<"\n";
    cout<<mean<<"\n";
    }
    getch();
    return 0;
}

Комментарий модератора

Используйте теги форматирования кода!

@soon · 03.11.2011, 09:04

Сообщение от Aliru777

while ((grade > 100 || grade < 0) && grade != -1 );

Введите -2.
Может есть еще, я не вчитывался.

C++

//Принимает оценку, для выхода необходимо ввести оценку вне диапазона [0;100], либо символ.
//После окончания ввода выводит средний балл.
#include <iostream>
 
int main()
{
    int mark, count = 0;
    float sum = 0;
    std::cout << "enter mark(0 <= mark <= 100), incorrect mark to exit" << std::endl;
    while((std::cin >> mark) && (mark >= 0) && (mark <= 100))
    {
        sum += mark;
        ++count;
    }
    if(count)
        std::cout << sum / count << std::endl;
    return 0;   
}

MrGluck · 03.11.2011, 09:50

C++

#include <iostream>
#include <conio.h>
#include <clocale>
using namespace std;
 
int main()
{
    setlocale(LC_ALL,"Russian");
    int sum = 0, count = 0, el;
    while(cin>> el)
    {
        if (el == -1) break;
        else
        if (el > 100 || el < -1)
            cout<< "Вы ввели неверное значение\n";
        else
        {
            sum += el;
            count++;
        }
    }
    if (count != 0) // дабы не делить на 0
    {
        float ave = sum/count;
        cout<< "Среднее значение равно "<< ave;
    }
    getch();
    return 0;
}

@Dar101 · 03.11.2011, 12:30

Буквы еще можно ввести, что нехорошо для вашей программы.

@Aliru777 · 03.11.2011, 15:01 **[ТС]**

C++

#include <iostream>
#include <conio.h>
#include <clocale>
using namespace std;
 
int main()
{
    setlocale(LC_ALL,"Russian");
    int sum = 0, count = 0, el;
    while(cin>> el)
    {
        if (el == -1) break;
        else
        if (el > 100 || el < -1)
            cout<< "Вы ввели неверное значение\n";
        else
        {
            sum += el;
            count++;
        }
    }
    if (count != 0) // дабы не делить на 0
    {
        float ave = sum/count;
        cout<< "Среднее значение равно "<< ave;
    }
    getch();
    return 0;
}

Это проще чем у меня и , наверное, быстрее.
Спасибо!
Про буквы - я еще так далеко не читал книг - я просто не знаю как это делать вообще - но это мне на будущее - так что Спасибо!
Про -2 действительно ступил - еще и делал условие,а потом убрал его.
А что лучше почитать и где задачи брать для практики?

@soon · 03.11.2011, 15:04

Есть прикрепленная тема в разделе c/c++, под названием литература. Почитайте.
Задания есть на форуме, для начинающего самое то, руку набьете. Также есть сайты с олимпиадными задачами, к примеру acmp.

@Aliru777 · 04.11.2011, 10:02 **[ТС]**

Программа которая пересчитывает из 2ой системы в 10чную.
Не могу понять почему в цикле ФОР не считает переменную b?
Подскажите пожалуйста.

C++

#include "stdafx.h"
#include <conio.h>
#include <math.h>
#include <iostream>
 
using namespace std;
 
int _tmain(int argc, _TCHAR* argv[])
 
{
    int b,x,c=0,d=0;
    double w=2;
    cout<<"Enter the number\n";
    cin>>x;
    while (x > 0)  //нахождение кол-ва знаков во введенном числе
    {
        x/=10;
        c++;
    }
    cout<<c<<"\n";
    for (int a=1;a<=c;a++)
    {
        b=x%10;
        cout<<b<<"\n"; // вывод значения 1 или 0
        x=(x-b)/10;
            if (b == 1)
            {
                d=d+pow(w,a);
            }
            else 
            {
                d+=0;
            }
    }
    cout<<d<<"\n";
    getch();
    return 0;
}

@soon · 04.11.2011, 10:22

C++

#include <iostream>
#include <cstring>
 
int main()
{
    std::string str;
    std::cin >> str;
    int lngth = str.length();
    int num = 0;
    for(int i = lngth; i >= 0; --i)
    {
        if(str[i] == '1')
            num += (1 << lngth - i - 1);
    }
    std::cout << num << std::endl;
    return 0;
}

@Aliru777 · 04.11.2011, 10:27 **[ТС]**

Это массив,наверное - я их еще подробно не учил - запишу себе как вариант - Спасибо!
я всю ночь долбился с ней и через 20 минут после того как сюда написал решил ее,по своему правда - хотел удалить а нельзя(((.
качал пресс,играл с ребенком и читал дискретную мат-ку)))
вот где озарение приходит)))

@soon · 04.11.2011, 10:42

Все понятно, или может прояснить некоторые моменты в коде?

@Aliru777 · 04.11.2011, 10:55 **[ТС]**

Я их не выучил - а объяснять тому кто не знает элементарного бессмысленно.
Но из того что я знаю не пойму что такое std i endl это как "/n",а стд чтобы не задействовать юзинг найм спейс?
и не совсем понял эту сторку num += (1 << lngth - i - 1).

@soon · 04.11.2011, 11:16

Про std::endl все верно, это перевод на следующую строку.
Касательно

C++

1	num += (1 << lngth - i - 1)

<< это битовый сдвиг. Т.е. единицу можно представить в виде 00000001. Соответственно про сдвиге влево мы получим вот это 00000010. В десятеричной системе это двойка. будем сдвигать и дальше, будем получать num * 2^i, где i - показатель степени. В данном примере lngth - i - 1 определит номер символа с конца.
Ps/ lngth вводится для того, чтобы не вычислять постоянно str.length(). Так, на всякий случай пояснил

silent_1991 · 04.11.2011, 14:36

std::endl, в отличие от '\n', кроме перевода на новую строку вызывает функцию flush, т.е. выталкивает всю накопленную в буфере информацию на выход, т.о. буфер остаётся пустым, что бывает необходимо для, например, синхронизации.

silent_1991 5041 / 3102 / 271 Регистрация: 11.11.2009 Сообщений: 7,047
	04.11.2011, 14:36	13
	std::endl, в отличие от '\n', кроме перевода на новую строку вызывает функцию flush, т.е. выталкивает всю накопленную в буфере информацию на выход, т.о. буфер остаётся пустым, что бывает необходимо для, например, синхронизации. 2

@Dar101 40 / 40 / 7 Регистрация: 12.05.2011 Сообщений: 109
	03.11.2011, 12:30	4
	Буквы еще можно ввести, что нехорошо для вашей программы. 0

@soon 2551 / 1316 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	03.11.2011, 15:04	6
	Есть прикрепленная тема в разделе c/c++, под названием литература. Почитайте. Задания есть на форуме, для начинающего самое то, руку набьете. Также есть сайты с олимпиадными задачами, к примеру acmp. 1

@Aliru777 1 / 1 / 0 Регистрация: 03.11.2011 Сообщений: 71
	04.11.2011, 10:27 [ТС]	9
	Это массив,наверное - я их еще подробно не учил - запишу себе как вариант - Спасибо! я всю ночь долбился с ней и через 20 минут после того как сюда написал решил ее,по своему правда - хотел удалить а нельзя(((. качал пресс,играл с ребенком и читал дискретную мат-ку))) вот где озарение приходит))) 0

@soon 2551 / 1316 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	04.11.2011, 10:42	10
	Все понятно, или может прояснить некоторые моменты в коде? 0

@Aliru777 1 / 1 / 0 Регистрация: 03.11.2011 Сообщений: 71
	04.11.2011, 10:55 [ТС]	11
	Я их не выучил - а объяснять тому кто не знает элементарного бессмысленно. Но из того что я знаю не пойму что такое std i endl это как "/n",а стд чтобы не задействовать юзинг найм спейс? и не совсем понял эту сторку num += (1 << lngth - i - 1). 0