Почему в массив записываются числа не и указанного диапазона?

Question

[Ogurchik-007]

Написал код, элементы массива должны задаваться рандомно в указанном диапазоне, но при выводе массива числа рандомные.

#include<iostream>
#include <cstdlib>
#include <ctime>
#include <iomanip>
using namespace std;

void inputArray(int *p, int k); // Функция инициализации массива

void printArray(int B[14][24], int i, int j) { // Функция для вывода массива
	for (i = 0; i < 24; i++) {
		for (j = 0; j < 14; j++) {
			cout<<setw(5)<<B[i][j];
		} 
		cout<<endl;
	}
}

void processingArray(int *B[0], int k, int i, int j) { // Функция для обработки массива 
	for (i = 0; i < 24; i++) {
		for (j = 0; j < k; j++) {
			if (B[i][j] > 0) {
				B[i][j] = 0;
			}
			else {
				B[i][j] = B[i][j] * -10;
			} 
		} 
	};
}

int main() {
  setlocale(LC_ALL, "Russian");
  const int n = 14;
  const int m = 24;
  int B[n][m], i=0, j=0;


	
	
  for(i = 0; i < n; i++){
  	inputArray(B[i], m);
  	for (j = 0; j < m; j++){
  		cout<<setw(12)<<B[i][j];
	  }
	cout<<endl;
  }

  cout<<endl<<endl;
  

  
  

}

void inputArray(int *p, int k) {
	srand(time(0));
	*p = (rand( )%100 - 50);
	for (int i = 1; i < k; i++) {
		*(p + i) = -i * * (p + i - 1);
	}
}

Comments

[Александр Ананьев]

Почему не используете std::vector ?

Answer 1

[Wataru]

*p = (rand( )%100 - 50);
for (int i = 1; i < k; i++) {
  *(p + 1) = -i * * (p + i - 1);
}

Можете вообще объяснить, что этот код делает? Зачем он это делает? Что вы хотели, чтобы он делал?

Помимо этого, вместо *(p+X) можно писать p[X]. Так понятнее. У вас тут явно не к месту арифметика указателей приплетена.

А так, код работает так, как написан. Первое число в массиве заполняется случайным от -50 до 49, второе число много раз переписывается чем-то умноженным на какое-то другое число массива, остальные числа не заполняются вообще.

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

Comments

[Ogurchik-007]

Wataru, писал код с учебника и там сказано что функциия inputArray должна заполнить двумерный массив.

[Wataru]

Ogurchik-007, Перечитайте внимательно учебник. Вы с ошибками перепечатали оттуда и у вас получилась хрень.

[Ogurchik-007]

Wataru, уже несколько раз перепроверил запись и вызов цункции и ее тело написал как в учебнике

[Wataru]

Ogurchik-007, Значит, опечатка в учебнике. Как в другом ответе верно заметили, если поменять p+1 на p+i, то уже будут хотя бы заполнятся все элементы массива. Правда там только первый будет случайным числом в промежутке от -50 до 50, остальные будут от этого первого зависить - и могут быть очень большими. Там формула забита - домножать первое число на факториал и менять знак у каждого второго.

[Ogurchik-007]

Wataru, а как поправить что бы все элементы были случайными?

[Wataru]

Ogurchik-007, А что в учебнике по этому поводу написано?

[Ogurchik-007]

Wataru, там только это

[Wataru]

Ogurchik-007, я думаю - нет. Я думаю это вы откуда-то списали, но ничего не поняли и теперь просите интернет решить задание за вас. Если я ошибаюсь - то извините. Загрузите фотографию учебника куда-нибудь и я вам помогу в этом разобраться.

[Ogurchik-007]

Wataru, Дело в том что это видео в закрытом доступе, могу только приложить скриншот данного фрагмента в видео

[Wataru]

Ogurchik-007, Ладно. Вот то, что у вас перед циклом написано, с rand(), надо сделать для всех остальных элементов массива (p[i] или *(p+i))

[Ogurchik-007]

Wataru,

[Ogurchik-007]

Wataru, я так тоже делал только оно заполняет элементы в столбце одинаковыми значениями

[Wataru]

Ogurchik-007, Этот приведенный вами код сильно отличается от того, что в вопросе. где у вас там есть "что у вас перед циклом написано, с rand()"?

[Ogurchik-007]

Wataru, без s rand не работает счетчик случайных чисел

[Ogurchik-007]

Wataru, Вот так сейчас выглядит код

#include<iostream>
#include <cstdlib>
#include <ctime>
#include <iomanip>
using namespace std;

void inputArray(int *p, int k); 

int main() {
  setlocale(LC_ALL, "Russian");
  const int n = 10;
  const int m = 10;
  int B[n][m], i, j;
	
  for (i = 0; i < n; i++) {
  	inputArray(B[i], n);
		for (j = 0; j < m; j++) {
			cout<<setw(5)<<B[i][j];
		} 
		cout<<endl;
	}
}

void inputArray(int *p, int k) {
	srand(time(0));
	  for (int i = 0; i < k; i++)
	  	*(p+i)=(rand()%100 - 50);
}

[Wataru]

Ogurchik-007, переместите srand в main. Это инициализация генератора случайных чисел. Если вызывать его для каждой строки, то с большой вероятностью время системное будет таким же для соседних строк (точность не такая большая, что бы каждые несколько инструкций тикать), а значит генераторы будут одинаково иниуиализированны и будут авдавать лдинаковые последовательности.

[Ogurchik-007]

Wataru, спасибо, это помогло.

Answer 2

[Rsa97]

Вы заполняете только p[0] и p[1], остальные ячейки массива остаются неинициализированными.
Судя по всему, вы криво набрали текст с какого-то видеоурока, и, вместо *(p + i) написали *(p + 1).

Comments

[Ogurchik-007]

Rsa97, да ошибся в этом месте, но если исправить на i, то все равно ничего не меняется.