Форум программистов, компьютерный форум CyberForum.ru

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

Восстановить пароль Регистрация
 
reidaigo
6 / 6 / 0
Регистрация: 16.12.2012
Сообщений: 31
29.10.2013, 12:10     Преждевременный выход из цикла #1
Здраствуйте! Спасибо за внимание. Стояла задача:
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;
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.10.2013, 12:10     Преждевременный выход из цикла
Посмотрите здесь:

Автоматический выход из цикла. C++
Выход из цикла C++
Выход из цикла C++
Преждевременный выход из цикла for на некотором промежутке C++
C++ Выход из двойного цикла
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Alexxx7
97 / 33 / 5
Регистрация: 24.04.2012
Сообщений: 151
29.10.2013, 12:20     Преждевременный выход из цикла #2
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
146 / 112 / 2
Регистрация: 15.01.2013
Сообщений: 266
29.10.2013, 12:21     Преждевременный выход из цикла #3
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
Антикодер
577 / 478 / 23
Регистрация: 15.09.2012
Сообщений: 2,429
29.10.2013, 12:25     Преждевременный выход из цикла #4
Цитата Сообщение от 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 уже высказал правильную мысль.
Yandex
Объявления
29.10.2013, 12:25     Преждевременный выход из цикла
Ответ Создать тему
Опции темы

Текущее время: 08:11. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru