Random

@Natatem00 · Регистрация: 05.10.2018

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

Здравствуйте. Столкнулся с таким: если использую такой код

C++
1
2
3
4
5
6
7
8
9
10
for (int j = 0; j < COLOUMNS; j++)
            {
                arr[i][j] = -7 + rand() % 13;
                if (arr[i][j] < 0)
                {
                    otriz++;
                    continue;
                }
                dodat++;
            }

то проверка условия срабатывает не верно, но если написать так:

C++
1
2
3
4
5
6
7
8
9
10
11
for (int j = 0; j < COLOUMNS; j++)
            {
                arr[i][j] = -7 + rand() % 13;
                cout << arr[i][j] << " ";
                if (arr[i][j] < 0)
                {
                    otriz++;
                    continue;
                }
                dodat++;
            }

то код работает как надо. В чём тут хитрость?

@Natatem00 · 25.11.2018, 20:38 **[ТС]**

На самом деле, если вывести массив ещё раз, то там будут другие значения

@DrOffset · 25.11.2018, 20:44

Natatem00, Полностью код не хотите показать?

Verevkin · 25.11.2018, 20:56

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

В чём тут хитрость?

А чем отличаются эти 2 куска кода?

@Natatem00 · 25.11.2018, 21:08 **[ТС]**

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

Natatem00, Полностью код не хотите показать?

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
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
int main()
{
  srand(time(0));
  setlocale(LC_ALL, "ru");
  int LINES, COLOUMNS;
 
  cout << "Введите колличество строк и столбцов\n";
  cin >> LINES >> COLOUMNS;
  if (LINES <= 0 || COLOUMNS <= 0)
  {
    cout << "Ошибка! Введите значение > 0\n";
    system("pause");
    return 1;
  }
 
  int *temp = new int[LINES * COLOUMNS];
  int ** arr = new int*[COLOUMNS];
  if (LINES < COLOUMNS)
  {
    for (int i = 0; i < COLOUMNS; i++)
    {
      arr[i] = &temp[i];
    }
  }
  else
  {
    for (int i = 0; i < LINES; i++)
    {
      arr[i] = &temp[i];
    }
  }
  
  int key = 0;
  int otriz = 0;
  int current = 0;
  int dodat = 0;
  bool first = true;
  int numLine = 0;
  cout << "Создать автоматически массив - 1, самостоятельно - 0 \n";
  cin >> key;
  if (key == 1)
  {
    for (int i = 0; i < LINES; i++)
    {
      for (int j = 0; j < COLOUMNS; j++)
      {
        arr[i][j] = -7 + rand() % 13;
        cout << arr[i][j] << " ";
        if (arr[i][j] < 0)
        {
          otriz++;
          continue;
        }
        dodat++;
      }
      cout << endl;
      if (first)
      {
        numLine = i;
        current = otriz;
        first = false;
      }
      else if (current < otriz)
      {
        current = otriz;
        numLine = i;
      }
      otriz = 0;
    }
  }
  else
  {
    for (int i = 0; i < LINES; i++)
    {
      for (int j = 0; j < COLOUMNS; j++)
      {
        cout << "Введите [" << i << "][" << j << "] элемент: ";
        cin >> arr[i][j];
        if (arr[i][j] < 0)
        {
          otriz++;
          continue;
        }
        dodat++;
      }
      cout << endl;
      if (first)
      {
        numLine = i;
        current = otriz;
        first = false;
      }
      else if (current < otriz)
      {
        current = otriz;
        numLine = i;
      }
      otriz = 0;
    }
    for (int i = 0; i < LINES; i++)
    {
      for (int j = 0; j < COLOUMNS; j++)
      {
        cout << arr[i][j] << " ";
      }
      cout << endl;
    }
    cout << endl;
  }
 
  cout << endl;
 
  cout << "Введите множитель: \n";
  cin >> key;
 
  cout << "Dodatni - " << dodat << endl;
  for (int i = 0; i < LINES; i++)
  {
    for (int j = 0; j < COLOUMNS; j++)
    {
      if (i == numLine)
      {
        cout << arr[i][j] * key << " ";;
      }
      else
      {
        cout << arr[i][j] << " ";;
      }
    }
    cout << endl;
  }
 
  delete[] temp;
  system("pause");
  return 0;
}

@nonedark2008 · 25.11.2018, 21:58

Строка 22. Думаю, что планировалось вот так:

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

arr[i] = &temp[i * LINES];

Строки 25-31 лучше удалить. В чем их смысл?

@Natatem00 · 25.11.2018, 22:04 **[ТС]**

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

Строки 25-31 лучше удалить. В чем их смысл?

Наверно в том, чтобы не выходить за размер хипа, если строк больше столбцов?
Я спрашивал совсем про другое.

@DrOffset · 25.11.2018, 22:54

Natatem00, у вас неверно распределяется память под строки и столбцы. Вы просто назначаете по порядку указатели на элементы массива LINES*COLOUMNS, а надо с кратностью LINES или COLOUMNS в зависимости от того, что является первым индексом. Следовательно при записи туда элементов у вас последующая запись перетирает предыдущую, поэтому получаете такой эффект как на скрине.
Вообще, вам надо определиться, какой индекс у вас первый, а то вот вы выделяете COLOUMNS для массива указателей, а используете из него только LINES элементов (первый цикл). Если LINES < COLOUMNS, то все в порядке, а если больше, то начинаются интересности, с коими вы боретесь условием, по поводу которого вам выше сделали замечание. Очень похоже на лечение симптома, а не проблемы.
Надо было сделать так, и нужда в доп. условиях отпала бы:

C++
1
2
3
4
5
    int ** arr = new int*[LINES];
    for (int i = 0; i < LINES; i++)
    {
        arr[i] = &temp[i * COLOUMNS];
    }

@Natatem00 · 25.11.2018, 23:10 **[ТС]**

DrOffset, спасибо! Вы действительно объяснили, а то, что было выше - называется вброс

Новые блоги и статьи Все статьи Все блоги /
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. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@Natatem00 0 / 0 / 0 Регистрация: 05.10.2018 Сообщений: 22
	25.11.2018, 20:38 [ТС]
	На самом деле, если вывести массив ещё раз, то там будут другие значения Миниатюры 0

@DrOffset 19500 / 10105 / 2461 Регистрация: 30.01.2014 Сообщений: 17,816
	25.11.2018, 20:44
	Natatem00, Полностью код не хотите показать? 0

@Natatem00 0 / 0 / 0 Регистрация: 05.10.2018 Сообщений: 22
	25.11.2018, 23:10 [ТС]
	DrOffset, спасибо! Вы действительно объяснили, а то, что было выше - называется вброс 0

Random

Решение