Преждевременный выход из цикла

@reidaigo · Регистрация: 16.12.2012

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

Здраствуйте! Спасибо за внимание. Стояла задача:
1. построить двумерный динамический массив. (выполнено)
2. нахождение количества строк, не содержащих элемента со значением ноль. (выполнено)
3. нахождение максимального элемента, который есть в массиве более одного раза. Вот тут у меня проблемы. Для начала преобразовал двумерный массив в одномерный, затем провел сортировку по убыванию.
Далее мой алгоритм действий:
если в отсортированном массиве, первый элемент равен второму, то он и есть искомый максимальный элемент(так как сортировка по убыванию - первыми стоят наибольшие числа). Если нет, то в цикле дальше идет перебор элементов. И всё работает, но программа ищет все повторяющиеся элементы, а не только самый максимальный, хоть и после выполнения условия я пытался и break, и булевую переменную. Необходимый код помечен комментарием "//поиск максимального повторяющегося элемента" (внизу)

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
#include "stdafx.h"
#include <iostream>
#include <ctime>
#include <iomanip>
using namespace std;
 
int main(int argc, char* argv[])
{
    setlocale(LC_ALL, "");
    int m,n;
    cout << "Введите количество строк матрицы m = "; cin >> m; "\n";
    cout << "Введите количество столбцов матрицы n = "; cin >> n; "\n";
    srand(time(0)); 
    int **ptrarray = new int* [m]; 
    for (int count = 0; count < m; count++)
        ptrarray[count] = new int [n]; 
 
    for (int count_row = 0; count_row < m; count_row++)
        for (int count_column = 0; count_column < n; count_column++)
            ptrarray[count_row][count_column] = rand()%100-50;
 
    for (int count_row = 0; count_row < m; count_row++)
    {
        for (int count_column = 0; count_column < n; count_column++)
            cout << setw(4) <<setprecision(2) << ptrarray[count_row][count_column] << "   ";
        cout << endl;
    }
 
    // подсчет количества строк, не содержащих нулей
    int kolvo=0;
for(int i=0; i<m; i++) {
        bool mb_zero=false;
        for(int j=0; j<n; j++) {
                if(!ptrarray[i][j]) mb_zero=true;
            
        }
        if(!mb_zero) kolvo++;
}
cout << "Количество строк, не содержащих ни одного нулевого элемента " << kolvo << "\n"; 
 
//преобразование двумерного массива в одномерный
int* pmas2 = new int[m*n]; int k=0;
    for (int i=0;i<m; i++) {
        for (int j=0;j<n;j++) {
            
            pmas2[k]=ptrarray[i][j];
                k++;
    
        }}
    for (int i=0;i<k;i++) {cout << pmas2[i] << " ";}
    cout << "\n";
    
    //сортировка массива
     for (int i = 0; i < k; i++)
       for (int j = i+1; j < k; j++)
          if (pmas2[i] < pmas2[j])
          {
             int buf = pmas2[i];
             pmas2[i] = pmas2[j];
             pmas2[j] = buf;
          }
    cout << endl << endl;
    for (int i = 0; i < k; i++)
       cout << pmas2[i] << "  ";
 
    //поиск максимального повторяющегося элемента
    bool flag=false; 
     for (int i = 0; i < k; i++) {
       for (int j = i+1; j < k; j++) 
          if (pmas2[i] == pmas2[j])
              cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl; flag=true;
          
          if (!flag) break;
     }
 
 
    // удаление двумерного динамического массива
    for (int count = 0; count < m; count++)
        delete []ptrarray[count];
    system("pause");
    return 0;
}

@Alexxx7 · 29.10.2013, 12:20

C++
1
2
if (pmas2[i] == pmas2[j])
              cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl; flag=true;

может стоит попробовать так

C++
1
2
3
4
if (pmas2[i] == pmas2[j]){
              cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl;
              flag=true;
}

@Rivory · 29.10.2013, 12:21

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    bool flag=false;
    for (int i = 0; i < k; i++)
    {
    if(flag) break;
        for (int j = i+1; j < k; j++)
        {
        if(flag) break;
            if (pmas2[i] == pmas2[j])
            {
                cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl;
                flag = true;
                break;
            }
        }
    }

Не по теме:

Что за дурацкая привычка не писать {...} в циклах и if'ах? Вот как по вашему,

C++
1
2
          if (pmas2[i] == pmas2[j])
              cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl; flag=true;

flag=true присваевается внутри if'a или вне его?

Вообще намного легче вынести это в отдельную функцию и сделать return в нужный момент.

XRuZzz · 29.10.2013, 12:25

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

//поиск максимального повторяющегося элемента bool flag=false; for (int i = 0; i < k; i++) { for (int j = i+1; j < k; j++) if (pmas2[i] == pmas2[j]) cout << "\n Максимальный повторяющийся элемент = " << pmas2[i] << endl; flag=true; if (!flag) break; }

в C++ стиле обычно оперируют классами и объектами, но если уж пишите в C стиле то хотя бы вынесли бы поиск максимума в отдельную функцию.

Ну вот, пока писал пост, товарищ Rivory уже высказал правильную мысль.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .	Установка Emscripten SDK (emsdk) и CMake на Windows для сборки C и C++ приложений в WebAssembly (Wasm) 8Observer8 30.01.2026 Чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. Система контроля версиями Git. . .	Подключение Box2D v3 к SDL3 для Android: физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .	Загрузка PNG с альфа-каналом на SDL3 для Android: с помощью SDL3_image 8Observer8 27.01.2026 Содержание блога SDL3_image - это библиотека для загрузки и работы с изображениями. Эта пошаговая инструкция покажет, как загрузить и вывести на экран смартфона картинку с альфа-каналом, то есть с. . .	Влияние грибов на сукцессию anaschu 26.01.2026 Бифуркационные изменения массы гриба происходят тогда, когда мы уменьшаем массу компоста в 10 раз, а скорость прироста биомассы уменьшаем в три раза. Скорость прироста биомассы может уменьшаться за. . .	Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android 8Observer8 26.01.2026 Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .