Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/3: Рейтинг темы: голосов - 3, средняя оценка - 4.67
0 / 0 / 0
Регистрация: 30.09.2012
Сообщений: 37

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

23.11.2012, 01:46. Показов 692. Ответов 6
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Дорогие друзья, такой вопрос что не так в моем коде?
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: побольше критики.
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
23.11.2012, 01:46
Ответы с готовыми решениями:

Нужна конструктивная критика по оформлению и читабельности кода (Game Trainer C++)
Относительно недавно начал изучать C++. Для обучения решил написать трейнер попутно изучая язык/читая необходимую литературу разного рода. ...

Нужна конструктивная критика: советы, замечания по заданному фрагменту кода
Есть входные данные, которые я кладу в вектор 5 1 2 3 4 5 Ваша критика #include &lt;iostream&gt; #include &lt;vector&gt;

Небольшой шаблонный класс matrix: комментарии/критика относительно кода с точки зрения правильности написания
В плане своего общего развития написал не большой шаблонный класс matrix. Хочу поделиться своей работой с остальными, - может кому то это...

6
~ Эврика! ~
 Аватар для OhMyGodSoLong
1258 / 1007 / 74
Регистрация: 24.07.2012
Сообщений: 2,002
23.11.2012, 01:59
Самое важное: как думаете, что будет, если ввести число, большее чем 255?

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

<личное> нормальная ОС, типо Linux консоль или перенаправление потоков лучше непереносимого Windows system("pause") </личное>
0
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
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 и что далее?
0
Форумчанин
Эксперт CЭксперт С++
 Аватар для MrGluck
8216 / 5047 / 1437
Регистрация: 29.11.2010
Сообщений: 13,453
23.11.2012, 19:25
Цитата Сообщение от ForEveR Посмотреть сообщение
MrGluck, С какого бока он платформозависимый?
Да, громко сказано. Но ведь реализация будет отличаться. Я не увижу эникей. И терминал не будет ждать нажатия клавиши.
0
 Аватар для David Sylva
1321 / 983 / 267
Регистрация: 17.05.2012
Сообщений: 2,687
23.11.2012, 19:28
Цитата Сообщение от Learn Посмотреть сообщение
Массивы,начало

Не по теме:

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

0
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
23.11.2012, 19:32
MrGluck, В данном случае соглашусь конечно.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
23.11.2012, 19:32
Помогаю со студенческими работами здесь

Критика кода на F#
Прошу прошения, что создаю тему без конкретного вопроса, просто знакомство с F# дается с трудом, и мне очень интересно, правильно ли я...

Критика кода
Здравствуйте форумчане, я сейчас в процессе изучения создания сайтов (сейчас учу js/php), понемногу верстаю чтоб набить руку. Но вот...

Требуется критика кода
Доброго времени суток, уважаемые форумачане. Хотел бы вас попросить покритиковать код. Версия python 3.6. Код во вложении. В заранее...

Калькулятор - критика кода
Сплошной примитив и способ реализации уверен не самый удачный, но я новичок в Си и для меня это большой шаг)) Хотелось бы посмотреть на...

Шахматы - критика кода
Думать пока не могут, только доска с возможностью делать ходы за обе стороны. Все правила игры соблюдаются, включая рокировку и взятие на...


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

Или воспользуйтесь поиском по форуму:
7
Ответ Создать тему
Новые блоги и статьи
Кто-нибудь знает, где можно бесплатно получить настольный компьютер или ноутбук? США.
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 - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru