214 похожих чатов

Void Test() { Console.Write("Элементов массива: ");

var col = int.Parse(Console.ReadLine());
var arr = new int[col];

CheckCol(col);

for (var i = 0; i < arr.Length; i++)
{
Console.WriteLine("\nЧисло массива");
arr[i] = int.Parse(Console.ReadLine());

if (arr[i] is < 0 or > 200000)
{
Console.WriteLine("ОШИБКА");
break;
}
}

Console.WriteLine("Элементы массива:");
foreach (var t in arr)
{
Console.WriteLine(t);
}

Console.WriteLine("Что сделать с массивом? (Вызначить минимальное число (min), вызначить максимальное число(max) )");

var ans = Console.ReadLine();

switch (ans)
{
case "min":
{
var resMin = arr.Min();
Console.WriteLine($"Минимальное значение: {resMin}");
break;
}
case "max":
{
var resMax = arr.Max();
Console.WriteLine($"Максимальное значение: {resMax}");
break;
}
default:
Console.WriteLine("Введите либо min, либо max");
break;
}

Console.WriteLine(arr[^1]);
}

static void CheckCol(int col)
{
if (col is < 0 or > 200000)
{
Console.WriteLine("ОШИБКА");
}
}

4 ответов

16 просмотров

void Test() { Console.Write("Элементов массива: "); var value = Console.ReadLine(); if (value == null) return; var countCols = int.Parse(value); var arr = new int[countCols]; if (CheckValue(countCols)) { return; } for (var i = 0; i < arr.Length; i++) { Console.WriteLine("\nЧисло массива"); var valueCell = int.Parse(Console.ReadLine() ?? string.Empty); if (CheckValue(valueCell)) { break; } arr[i] = valueCell; } PrintArray(arr); Console.WriteLine("Что сделать с массивом? (Вызначить минимальное число (min), вызначить максимальное число(max) )"); switch (Console.ReadLine()) { case "min": { var resMin = arr.Min(); Console.WriteLine($"Минимальное значение: {resMin}"); break; } case "max": { var resMax = arr.Max(); Console.WriteLine($"Максимальное значение: {resMax}"); break; } default: Console.WriteLine("Введите либо min, либо max"); break; } Console.WriteLine(arr[^1]); } static void PrintArray(int[] arr) { Console.WriteLine("Элементы массива:"); foreach (var t in arr) Console.WriteLine(t); } static bool CheckValue(int col) { if (col is < 0 or > 200000) { Console.WriteLine("ОШИБКА"); return true; } return false; }

Swatar-Common Автор вопроса
Vlad
void Test() { Console.Write("Элементов массива...

Проверочка у него одинаковая конечно, но наверно лучше в две вынести, так как разные данные проверяет

Много проблем внутри этой функции. static bool CheckValue(int col) { if (col is < 0 or > 200000) { Console.WriteLine("ОШИБКА"); return true; } return false; } 1) Название: CheckValue это действие не имеющее результата. Если мы что-то возвращаем, то название должно об этом говорить. Например, IsValueValid. Кстати, если метод возвращает буль, то обычно его называют вопросом, на который метод должен ответить. 2) Возвращаемые значения. Возвращать true при ошибке - контринтуитивно. 3) Выражение «col is < 0» нету смысла записывать так. Можно написать col < 0. 4) Вкусовщина, но писать or вместо || тоже такое себе. 5) Я бы не делал внутри этой функции Console.WriteLine - получается что функция проверки валидности ещё и ответсвенна за вывод сообщения в консоль. Сумбурненько

Swatar-Common Автор вопроса
Nikolajs Djačenko
Много проблем внутри этой функции. static bool C...

лайк, насчет is or вкусовщина и без is, or не пашет

Похожие вопросы

Обсуждают сегодня

Мужики и девушки, привет) в Вelphi xe7 в настройках во вкладке "Editor Options" далее " Color" есть список: "Elements", открыв который мы можем настраивать отображение разных...
Kraszx
14
Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Я вот подумал. SSE выполняет операции максимум с 64-битной точностью. А FPU - всегда с 80-битной. Разве не должно быть FPU точнее тогда?
The Bird of Hermes
13
как быть с принтером? такой подход прокатит?
zamtmn
12
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
13
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Я не понимаю, это троллинг или что? Швабрика поддерживают, который буквально пишет на ассемблере взаимодействия с винапи. Я это ещё написал загрузчик и хоть что-то изучаю в о...
Shadow Akira
6
А вот это что за конструкция? Вернее, она тут нафига?
Serjone
10
Привет. Подскажите, как правильно сматчить лист фиксированного размера, чтобы компилятор не говорил мне о неполном паттерне? Допустим что-то такое [x', y'] = sort [x, y]?
Arseny
8
Карта сайта