Массивы, критика кода

@Learn · Регистрация: 30.09.2012

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

Дорогие друзья, такой вопрос что не так в моем коде?

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include <iostream>
#include <stdlib.h>
 using namespace std;
 
 
int main()
{
int Matrix[255][255];
cout << "Enter number: "<< endl;
int n;
cin >> n;
 
for(int i =0; i<n; i++)
{
for(int j = 0; j<n; j++){
    Matrix[i][j] = rand()%100;
    cout << Matrix[i][j]<<endl;}
 
}
system("pause");
return 0;
}

P.s: побольше критики.

@OhMyGodSoLong · 23.11.2012, 01:59

Самое важное: как думаете, что будет, если ввести число, большее чем 255?

Мелочи:

<cstdlib> вместо <stdlib.h> (не критично, но хороший тон для Си++);
больше отступов же ;)
если уж это матрица, то выводите её квадратиком, а не верёвкой;
<личное> нормальная консоль или перенаправление потоков лучше непереносимого system("pause") </личное>

MrGluck · 23.11.2012, 02:15

Мне кажется, главная ошибка тут как раз в том, что ты создаешь массив размерностью 255*255, а потом заполняешь как n*n. А если n > 255? Либо запихни размер массива в const и в циклах индексы до этого числа меняй, либо используй динамические массивы (не забудь освободить после них память).
Если добавлять по-мелочам, где srand (time (0)) ?
Почему переменная с большой буквы? Так обычно называют классы. Вообщем, рефакторинг. Ах да, код платформозависимый

<личное> нормальная ~~ОС, типо Linux~~ консоль или перенаправление потоков лучше непереносимого ~~Windows~~ system("pause") </личное>

@ForEveR · 23.11.2012, 09:20

MrGluck, С какого бока он платформозависимый?

C++
1
2
3
4
5
6
7
8
9
#include <iostream>
#include <cstdlib>
 
int main()
{
   std::cout << "Hello" << std::endl;
   system("pause");
   std::cout << "Bye" << std::endl;
}

Bash
1
2
3
4
5
forever@pterois:~/My_pro1/cpp_pro$ g++ -o new new.cpp
forever@pterois:~/My_pro1/cpp_pro$ ./new 
Hello
sh: 1: pause: not found
Bye

Ну ругнулся bash и что далее?

MrGluck · 23.11.2012, 19:25

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

MrGluck, С какого бока он платформозависимый?

Да, громко сказано. Но ведь реализация будет отличаться. Я не увижу эникей. И терминал не будет ждать нажатия клавиши.

@David Sylva · 23.11.2012, 19:28

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

Массивы,начало

Не по теме:

название темы звучит, как название блокбастера

@ForEveR · 23.11.2012, 19:32

MrGluck, В данном случае соглашусь конечно.

Новые блоги и статьи Все статьи Все блоги /
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Programma_Boinc 26.12.2025 Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США. Нашел на реддите интересную статью под названием «Кто-нибудь знает, где получить бесплатный компьютер или. . .	Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Programma_Boinc 23.12.2025 Рецензия / Мнение/ Перевод Нашел на реддите интересную статью под названием The Thinkpad X220 Tablet is the best budget school laptop period . Ниже её машинный перевод. Thinkpad X220 Tablet —. . .	PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .

@OhMyGodSoLong ~ Эврика! ~ 1258 / 1007 / 74 Регистрация: 24.07.2012 Сообщений: 2,002
	23.11.2012, 01:59
	Самое важное: как думаете, что будет, если ввести число, большее чем 255? Мелочи: <cstdlib> вместо <stdlib.h> (не критично, но хороший тон для Си++); больше отступов же ;) если уж это матрица, то выводите её квадратиком, а не верёвкой; <личное> нормальная консоль или перенаправление потоков лучше непереносимого system("pause") </личное> 0

MrGluck Форумчанин 8216 / 5047 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	23.11.2012, 02:15
	Мне кажется, главная ошибка тут как раз в том, что ты создаешь массив размерностью 255255, а потом заполняешь как nn. А если n > 255? Либо запихни размер массива в const и в циклах индексы до этого числа меняй, либо используй динамические массивы (не забудь освободить после них память). Если добавлять по-мелочам, где srand (time (0)) ? Почему переменная с большой буквы? Так обычно называют классы. Вообщем, рефакторинг. Ах да, код платформозависимый <личное> нормальная ~~ОС, типо Linux~~ консоль или перенаправление потоков лучше непереносимого ~~Windows~~ system("pause") </личное> 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	23.11.2012, 19:32
	MrGluck, В данном случае соглашусь конечно. 0