Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
0 / 0 / 0
Регистрация: 05.10.2018
Сообщений: 22
1

Random

25.11.2018, 19:57. Показов 967. Ответов 8

Здравствуйте. Столкнулся с таким: если использую такой код
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++;
            }
то код работает как надо. В чём тут хитрость?
0
Лучшие ответы (1)
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
25.11.2018, 19:57
Ответы с готовыми решениями:

Random и объекты класса (pseudo random)
Всем привет. Есть класс: typedef unsigned int ui; class Player { private: ui health;

Random
Помогите,пожалуйста, подкорректировать текст проги так,что был массив задавался рандомно,а не...

random
Нужно сгенерировать случайное число двух значное как это сделать ? Visual C++ Гуглил но только...

random
R=random(101)/100.; Это генератор случайных чисел (0;1)?

8
0 / 0 / 0
Регистрация: 05.10.2018
Сообщений: 22
25.11.2018, 20:38  [ТС] 2
На самом деле, если вывести массив ещё раз, то там будут другие значения
0
Миниатюры
Random  
14150 / 7605 / 1808
Регистрация: 30.01.2014
Сообщений: 12,728
25.11.2018, 20:44 3
Natatem00, Полностью код не хотите показать?
0
Продавец времени
5022 / 2780 / 637
Регистрация: 12.03.2015
Сообщений: 13,504
25.11.2018, 20:56 4
Цитата Сообщение от Natatem00 Посмотреть сообщение
В чём тут хитрость?
А чем отличаются эти 2 куска кода?
0
0 / 0 / 0
Регистрация: 05.10.2018
Сообщений: 22
25.11.2018, 21:08  [ТС] 5
Цитата Сообщение от 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;
}
0
1354 / 992 / 314
Регистрация: 28.07.2012
Сообщений: 2,750
25.11.2018, 21:58 6
Строка 22. Думаю, что планировалось вот так:
Цитата Сообщение от Natatem00 Посмотреть сообщение
arr[i] = &temp[i * LINES];
Строки 25-31 лучше удалить. В чем их смысл?
0
0 / 0 / 0
Регистрация: 05.10.2018
Сообщений: 22
25.11.2018, 22:04  [ТС] 7
Цитата Сообщение от nonedark2008 Посмотреть сообщение
Строки 25-31 лучше удалить. В чем их смысл?
Наверно в том, чтобы не выходить за размер хипа, если строк больше столбцов?
Я спрашивал совсем про другое.
0
14150 / 7605 / 1808
Регистрация: 30.01.2014
Сообщений: 12,728
25.11.2018, 22:54 8
Лучший ответ Сообщение было отмечено Natatem00 как решение

Решение

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];
    }
1
0 / 0 / 0
Регистрация: 05.10.2018
Сообщений: 22
25.11.2018, 23:10  [ТС] 9
DrOffset, спасибо! Вы действительно объяснили, а то, что было выше - называется вброс
0
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
25.11.2018, 23:10

Заказываю контрольные, курсовые, дипломные работы и диссертации здесь.

Random
для чего пишем srand(time(0)); ? srand(time(0)); int a = rand() % N;

random
Доброго времени суток! Вопрос прост: для генерации случайных чисел всегда пользовался rand(), но...

Random
Всем привет! Столкнулся с проблемой: по программе должен нажать S, и после этого из списка 1.cpp...

Работа с random
Доброго времени суток! У меня есть массив int a={10,20,30,40,50}. Можно сделать так, чтобы...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
9
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2021, vBulletin Solutions, Inc.