По с++ преподаватель пишет, что программа не соответствует "функции вводится последовательность из 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 на две функции - неясноВвод данных не должен находиться в функции проверки числа на простоту(да и смысл разбиения программы на функции пропадает)