问题描述
我正在使用 C++ (Windows 10 Visual Studio) 中的线程处理队列类。
在我的构造函数中,我首先将布尔错误值设置为 false。紧接着,我尝试初始化临界区。如果出现问题,则布尔值设置为真,否则保持为假。我在主函数(未显示)中检查该值,如果 crit sec 未初始化(bool 为 false),则结束程序。
Queue() {
// ERROR_CRIT is bool: false = no error in initialising crit section,true = there is an error
ERROR_CRIT = false;
// check init
if (!InitializeCriticalSectionAndSpinCount(&CritSec,0x00000400)) {
ERROR_INIT_CRIT_SEC = true;
}
totalCustomers = 0;
currentReadindex = 0;
currentWriteIndex = 0;
customerNum = 0;
};
我的问题是:我应该将哪些类型的布尔值默认设置为 true 或 false ?我在编写其他程序时多次考虑过这个问题,除非很明显,否则我不确定何时为 bool 设置默认值。有时从任一值开始似乎是公平的。我觉得将错误 bool 设置为 true 会很奇怪,但将其设置为 false 也可能很奇怪。
在现实世界或公司中,我会删除第 3 行并使用 else {ERROR_CRIT = false;}
在 if 中添加 else 语句吗?这会提高可读性吗?这可能是偏好,但在现实世界的编程中经常看到什么?
提前致谢:D
解决方法
如果不能创建临界区,更好的方法是完全阻止队列存在。这为 Queue
类建立了更严格的类不变量,从而简化了其实现。
这是通过抛出异常来完成的:
Queue() {
// check init
if (!InitializeCriticalSectionAndSpinCount(&CritSec,0x00000400)) {
throw std::runtime_error("failed to create the critical section");
}
totalCustomers = 0;
currentReadIndex = 0;
currentWriteIndex = 0;
customerNum = 0;
};
通过这种方式,您永远不需要检查 ERROR_CRIT
,因为现有的队列足以保证临界区被正确初始化。