Возвращаемое значение в методе c возвращаемым значением bool

@ShmelVG · Регистрация: 27.11.2015

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

Как бы грамотнее сделать, добавляю гудок, метод SetColors() возвращает true/false, потом так же выключаю через 1 сек, но вот думаю над terurn, как бы нормально сделать возвращаемое значение(возвращал тру/фолс)?

C#
1
2
3
4
5
6
7
8
9
10
public override bool Beep()
        {
            currentColor |= Colors.Alarm;
            SetColors();
            Thread.Sleep(1000);
            currentColor ^= Colors.Alarm;
            SetColors();
 
            return true;
        }

@Почтальон · 04.03.2019, 09:24

Скорее всего Beep - это процедура, а не функция. Можно добавить функцию - Beeped(), которая будет возвращать признак, был ли "гудок"

@ShmelVG · 04.03.2019, 09:26 **[ТС]**

Почтальон, в остальные методах я возвращаю SetColors(); , но тут надо сделать иначе...А как можно реализовать?

@Почтальон · 04.03.2019, 09:28

SetColors() у вас же процедура. Возможно проще было создать отдельный класс.

@ShmelVG · 04.03.2019, 09:31 **[ТС]**

Почтальон,

Сообщение от Почтальон

создать отдельный класс

вот сейчас вообще не понял) Метод SetColors() возвращает тру, если ответ сервопривода пришел успешно.

@Почтальон · 04.03.2019, 09:41

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

Метод SetColors() возвращает тру, если ответ сервопривода пришел успешно.

Не совсем верное решение, т.к. исходя из названия метода - метод устанавливает цвет и результатом должно возвращаться - установлен, либо нет (по каким-то причинам). Ну это уже чисто архитектурное предпочтение

@ShmelVG · 04.03.2019, 09:58 **[ТС]**

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

Почтальон,Пока сделал так)

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
public override bool Beep()
        {
            try
            {
                bool result = true;
                _commandExecuting.Reset();
                //Task.Factory.StartNew(() => result = InnerBeep());
                bool shouldExecuted = _commandExecuting.WaitOne(_TimeoutBeep);
                if (!shouldExecuted)
                {
                    currentColor |= Colors.Alarm;
                    SetColors();
                    Thread.Sleep(1000);
                    currentColor ^= Colors.Alarm;
                    SetColors();
                    return result;
                }  
 
                return false;
            }
            catch (Exception)
            {
                throw new IOException("Не удалось воспроизвести звук.", ex);
            }           
        }

@belalugoci · 04.03.2019, 10:42

замечания по коду:

C#
1
2
3
4
5
6
7
8
9
10
11
            try
            {
                bool result = true;
                ...
                if (!shouldExecuted)
                {
                    ...
                    return result;
                }  
                return false;
            }

я бы написал или так

C#
1
2
3
4
5
6
7
8
9
10
11
            try
            {
                bool result = false;
                ...
                if (!shouldExecuted)
                {
                    ...
                    result = true;
                }  
                return result;
            }

или так

C#
1
2
3
4
5
6
7
8
9
10
            try
            {
                ...
                if (!shouldExecuted)
                {
                    ...
                    return true;
                }  
                return false;
            }

или наверное даже более информативно

C#
1
2
3
4
5
6
7
8
9
            try
            {
                ...
                if (!shouldExecuted)
                {
                    ...
                }  
                return !shouldExecuted;
            }

@ShmelVG · 04.03.2019, 10:54 **[ТС]**

belalugoci, Почтальон, мб так?

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
public override bool Beep()
        {
            try
            {
                _commandExecuting.Reset();
                bool shouldExecuted = _commandExecuting.WaitOne(_TimeoutBeep);
                if (!shouldExecuted)
                {
                    if (SetColors())
                    {
                        currentColor |= Colors.Alarm;
                    }
                    else return false;
 
                    Thread.Sleep(1000);
 
                    if (SetColors())
                    {
                        currentColor ^= Colors.Alarm;
                    }
                    else return false;
                }
 
                return SetColors();
            }
            catch (Exception ex)
            {
                throw new IOException("Не удалось воспроизвести звук.", ex);
            }
        }

Добавлено через 2 минуты
По идее должен включиться и выключиться

@belalugoci · 04.03.2019, 10:55

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

мб так?

программа ваша, может быть и так. я, например, не понимаю зачем вы это делаете.

@ShmelVG · 04.03.2019, 11:01 **[ТС]**

belalugoci, надо включить гудок, через секунду выключить

@Почтальон · 04.03.2019, 11:04

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

надо включить гудок, через секунду выключить

в отдельном потоке можно это делать

@belalugoci · 04.03.2019, 11:06

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

надо включить гудок, через секунду выключить

написать метод для включения гудка на Х миллисекунд

C#
1
2
public void Beep(int _ms)
{}

в теле включить гудок, сделать задержку на _ms миллисекунд и выключить.

@ShmelVG · 04.03.2019, 11:10 **[ТС]**

Почтальон, belalugoci, теоретически мой метод будет работать правильно?

@Почтальон · 04.03.2019, 11:22

Теоритически - да, но он далеко от идеала.....

@belalugoci · 04.03.2019, 11:36

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

теоретически мой метод будет работать правильно?

неясным остается назначение bool и особенно включение звука сменой цвета

Новые блоги и статьи Все статьи Все блоги /
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .

@Почтальон управление сложностью 1693 / 1306 / 259 Регистрация: 22.03.2015 Сообщений: 7,545 Записей в блоге: 5
	04.03.2019, 09:24
	Скорее всего Beep - это процедура, а не функция. Можно добавить функцию - Beeped(), которая будет возвращать признак, был ли "гудок" 0

@ShmelVG 171 / 106 / 51 Регистрация: 27.11.2015 Сообщений: 1,269
	04.03.2019, 09:26 [ТС]
	Почтальон, в остальные методах я возвращаю SetColors(); , но тут надо сделать иначе...А как можно реализовать? 0

@Почтальон управление сложностью 1693 / 1306 / 259 Регистрация: 22.03.2015 Сообщений: 7,545 Записей в блоге: 5
	04.03.2019, 09:28
	SetColors() у вас же процедура. Возможно проще было создать отдельный класс. 0

@ShmelVG 171 / 106 / 51 Регистрация: 27.11.2015 Сообщений: 1,269
	04.03.2019, 11:01 [ТС]
	belalugoci, надо включить гудок, через секунду выключить 0

@ShmelVG 171 / 106 / 51 Регистрация: 27.11.2015 Сообщений: 1,269
	04.03.2019, 11:10 [ТС]
	Почтальон, belalugoci, теоретически мой метод будет работать правильно? 0

@Почтальон управление сложностью 1693 / 1306 / 259 Регистрация: 22.03.2015 Сообщений: 7,545 Записей в блоге: 5
	04.03.2019, 11:22
	Теоритически - да, но он далеко от идеала..... 1