По с++ преподаватель пишет, что программа не соответствует "функции вводится последовательность из n целых чисел. найти среднее арифметическое его цифр." в документе ворд код и тут оставлю: // f1.cpp: определяет точку входа для консольного приложения. // #include "stdafx.h" #include using namespace std; bool pr ( int& s , int& i) { bool p = true; int n , a; cout < < "n = "; while (cin > > n) { for ( a = 2 ; a< =n ; a++) { if ((n%a ! = 0) || (n == 2)) { cout < < "простое" < < endl < < endl; s=s+n; i=i+1; } else { p=false; cout < < "не простое" < < endl < < endl; } break; } } return p; } float arif (int s , int i) { cout < < i < < endl; cout < < s < < endl; return ( s/i ); } int _tmain(int argc, _tchar* { setlocale (lc_all , "rus"); int s=0 , i=0; bool p = pr (s ,i); if (p=true) cout < < "среднее арифметическое = " < < arif ( s , i ); system ("pause"); return 0; }
/*Код соответствует тексту задания из файла. Последовательность вводится в строку через знаки пробела, нажатие клавиши Enter заканчивает ввод последовательности. Программа НЕ проверяет корректность введенных данных*/
#include <iostream>
bool is_prime(const int n);
int main(int argc, char* argv[])
{
int numb, n_of_pr_numb = 0;
long int sum = 0;
do
{
std::cin >> numb;
if(is_prime(numb))
{
sum += numb;
n_of_pr_numb++;
}
} while (std::cin.peek() != '\n');
std::cout << "The arithmetic mean: " << (n_of_pr_numb == 0 ? 0 : sum * 1. / n_of_pr_numb);
}
bool is_prime(const int n)
{
if (n <= 1)
{
return false;
}
for (int i = 2; i <= pow(n, 1/2); i++)
{
if (n % i == 0)
{
return false;
}
}
return true;
}
Основные ошибки/недочеты Вашей реализации:
В функции pr строка for ( a = 2 ; a<=n ; a++) неверна, т.к. на последнем шаге число всегда будут сравнивать с самим собой, правильно for ( a = 2 ; a<n ; a++). Ну и нет смысла проверять до a=n-1, достаточно проверить до √n(об этом есть целая теорема).По условию никакого вывода слов "Простое/Не простое" не требуетсяСтрока cout << "n = "; лишняя, т.к. даже если на ввод будет подано несколько чисел, на экран "n = " будет выведено лишь в начале. Лучше уточнить для пользователя формат ввода.Смысл функции arif неясен, она выводит на консоль какие-то непонятные пользователю данные, затем возвращает s/i[ВНИМАНИЕ! Так как s и i целочисленные, то s/i не является средним арифметическим, это будет лишь целая часть от него. Правильнее s * 1./i или static_cast<double>(s)/i], а затем выводит это значение. Зачем разрывать cout на две функции - неясноВвод данных не должен находиться в функции проверки числа на простоту(да и смысл разбиения программы на функции пропадает)