задача С++ масивы работа над ошибками

@Nephilim · Регистрация: 28.03.2012

Студворк — интернет-сервис помощи студентам

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
#include <vector>
#include <iostream>
#include <algorithm>
#include <numeric>
#include <stdlib.h>
 
inline bool IsZero(int in)
{
  return !in;
}
 
inline bool AbsCmp(int f, int s)
{
  return abs(f) < abs(s) ? true : false;
}
 
int main(void)
{
  using namespace std;
  vector <int> v;
 
  // заполним массив
  v.push_back(10);
  v.push_back(20);
  v.push_back(0);
  v.push_back(30);
  v.push_back(-40);
  v.push_back(15);
  v.push_back(0);
  v.push_back(22);
 
  // распечатаем массив:
  for (unsigned int i = 0; i < v.size(); i++)
      cout<<v[i]<<" ";
  cout<<endl;
 
  // посчитаем число нулевых элементов:
  cout<<"amount zero elems = "<<count_if(v.begin(), v.end(), IsZero)<<endl;
 
  // выполним частичную сумму элементов:
  cout<<"total sum after min elem = "<<accumulate(++min_element(v.begin(), v.end()), v.end(), 0)<<endl;
 
  // отсортируем по возрастанию модуля:
  sort(v.begin(), v.end(), AbsCmp);
 
  // распечатаем массив после сортировки:
  for (unsigned int i = 0; i < v.size(); i++)
      cout<<v[i]<<" ";
  cout<<endl;
 
  return 0;
}

помогите исправить ошибки
: error C2105: '++' needs l-value в строке cout<<"total sum after min elem = "<<accumulate(++min_element(v.begin( ), v.end()), v.end(), 0)<<endl;
: error C2374: 'i' : redefinition; multiple initialization в строке for (unsigned int i = 0; i < v.size(); i++)
: see declaration of 'i' в строке for (unsigned int i = 0; i < v.size(); i++)
подскажите кто понимает о чем идет речь... спасибо!

@Kuzia domovenok · 05.04.2012, 22:30

что ты хотел сказать этой строчкой?
accumulate(++min_element(v.begin(), v.end()), v.end(), 0)

Я считаю, нет ничего вреднее для изучающего программирование, чем использование шаблонов без понимания их внутренних механизмов. Сначала самостоятельно напиши функцию нахождения суммы элементов, или что ты там высчитываешь... А потом уже используй STL, когда чувствуешь, что изобретать велосипед не хочется, но только полностью осознавая, что знаешь , как велосипеды работают!
Ну смотри, ну не работает твоё аккумулейт! Ну пойди альтернативным путём! Посчитай сумму в цикле! Неужели не лень регистрироваться на форуме из-за одной ошибки?

Короче, я б написал

C++
1
2
3
4
5
6
7
8
9
int min=v[0];
int  minid=0;
int sum=0;
unsigned int i;
for ( i = 0; i < v.size(); i++)
      if (v[i]<min){ min=v[i]; minid=i;}
for ( i = minid+1; i < v.size(); i++)
      sum+=v[i];
cout<<sum;

Просто потому, что у меня это занимает меньше времени, чем чтение справки по STL.

Добавлено через 11 минут
вторая ошибка

C++
1
for (unsigned int i = 0; i < v.size(); i++)

умеешь по английски читать? Переведи: 'i' : redefinition; multiple initialization
"Переопределение 'i': многократная инициализация"
Это значит, что ты зря инициализируешь i, как unsigned int в своём цикле
в этой области видимости i уже определена, несколько строк выше увидишь первое определение i в предыдущем цикле. Второй раз так делать бесполезно и даже вредно!

C++
1
for (i = 0; i < v.size(); i++)

Так правильно!

@Nephilim · 05.04.2012, 22:39 **[ТС]**

В одномерном массиве выполнить необходимые расчеты и преобразования.
Количество нулевых элементов массива. Сумму
элементов массива, расположенных после
минимального элемента. Расположить элементы массива в
порядке увеличения их модулей.

...если можете подскажите как исправить, благодарю!

Добавлено через 4 минуты
полный текст плис...

@Kuzia domovenok · 05.04.2012, 22:43

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

...если можете подскажите как исправить, благодарю!

А я тебе, что не исправил? Или у тебя больше двух ошибок?

Не по теме:

я б на месте учителя ученикам, использующим STL, устраивал бы мощный расспрос по всем алгоритмам, которые они шаблонно используют. Легко же определить, кто не просто халявил, а выдрал кусок из чужого кода, без понимания его работы. Поэтому советую хорошо подумать прежде чем показывать это: легче даже списать чей-нибудь чужой вариант с простыми циклами и массивами, чем наводить на себя подозрение использованием каких-либо шаблонов

@antoha398 · 05.04.2012, 22:44

Сообщение от Kuzia domovenok

вторая ошибка

C++
1
for (unsigned int i = 0; i < v.size(); i++)

умеешь по английски читать? Переведи: 'i' : redefinition; multiple initialization
"Переопределение 'i': многократная инициализация"
Это значит, что ты зря инициализируешь i, как unsigned int в своём цикле
в этой области видимости i уже определена, несколько строк выше увидишь первое определение i в предыдущем цикле. Второй раз так делать бесполезно и даже вредно!

C++
1
for (i = 0; i < v.size(); i++)

Так правильно!

В с++ должно прокатывать объявление переменной в условии цикла. Ведь она определяется локально в цикле и уничтожается при завершении цикла.

@Nephilim · 05.04.2012, 22:54 **[ТС]**

общими усилиями я понял свои ошибки) спасибо всем участвующим.

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
#include <vector>
#include <iostream>
#include <algorithm>
#include <numeric>
#include <stdlib.h>
 
inline bool IsZero(int in)
{
   return !in;
}
 
inline bool AbsCmp(int f, int s)
{
   return abs(f) < abs(s) ? true : false;
}
 
int main(void)
{
int i;
   using namespace std;
   vector <int> v;
 
   // заполним массив
   v.push_back(10);
   v.push_back(20);
   v.push_back(0);
   v.push_back(30);
   v.push_back(-40);
   v.push_back(15);
   v.push_back(0);
   v.push_back(22);
 
   // распечатаем массив:
   for (i = 0; i < v.size(); i++)
       cout<<v[i]<<" ";
   cout<<endl;
 
   // посчитаем число нулевых элементов:
   cout<<"amount zero elems = "<<count_if(v.begin(), v.end(), IsZero)<<endl;
   
   // выполним частичную сумму элементов:
   int min=v[0];
int  minid=0;
int sum=0;
 
for ( i = 0; i < v.size(); i++)
      if (v[i]<min){ min=v[i]; minid=i;}
for ( i = minid+1; i < v.size(); i++)
      sum+=v[i];
cout<<"total sum after min elem = "<<sum;
cout<<endl;
 
   // отсортируем по возрастанию модуля:
   sort(v.begin(), v.end(), AbsCmp);
 
   // распечатаем массив после сортировки:
   for (i = 0; i < v.size(); i++)
       cout<<v[i]<<" ";
   cout<<endl;
 
   return 0;
}

@Kuzia domovenok · 05.04.2012, 23:08

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

В с++ должно прокатывать объявление переменной в условии цикла. Ведь она определяется локально в цикле и уничтожается при завершении цикла.

ну я соглашусь с тобой, но тогда скажи, что ещё могло вызвать "i redifinition" ??

@antoha398 · 05.04.2012, 23:12

Сообщение от Kuzia domovenok

ну я соглашусь с тобой, но тогда скажи, что ещё могло вызвать "i redifinition" ??

Для меня это тоже загадка)
Наверное компилятор бредит

@Nephilim · 06.04.2012, 14:11 **[ТС]**

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

Для меня это тоже загадка)
Наверное компилятор бредит

это скорее всего.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

@Nephilim 0 / 0 / 0 Регистрация: 28.03.2012 Сообщений: 10
	05.04.2012, 22:39 [ТС]
	В одномерном массиве выполнить необходимые расчеты и преобразования. Количество нулевых элементов массива. Сумму элементов массива, расположенных после минимального элемента. Расположить элементы массива в порядке увеличения их модулей. ...если можете подскажите как исправить, благодарю! Добавлено через 4 минуты полный текст плис... 0