在枚举值之间切换:c++
Switching between enum values : C++
这应该是一个简单的问题。我没有接触过枚举,所以我不完全理解它们是如何工作的。在我的程序中,我试图将玩家的工作存储在enum中。我需要的功能,工作可以改变。我试图使用一个switch语句,但它并没有改变玩家的工作。
代码:// if change is True, change the job. If false, simply return the current value
int GameModeState::changeJob(bool change)
{
int job = Landman; // Default job is landman
if (change == true)
{
switch(job)
{
case Landman:
return job;
break;
case Geologist:
return job;
break;
default:
job = Landman;
return job;
break;
}
}
else
{
return job;
}
}
// when the player opens the stat sheet, it should change their job
void GameModeState::_statsheet()
{
changeJob(true);
}
我做错了什么让工作改变?我想问题出在switch语句
你的逻辑错了。switch(job)
语句将带您进入案例语句。在
case Landman
您立即返回Landman(因为您在执行switch语句之前将job设置为Landman,它将始终以您编码的方式返回Landman),这将从该函数完全返回您。它从不试图将工作更改为任何其他工作。请注意,break语句也永远不会执行,因为return会立即从这个函数调用返回。你可能想要这样:
case Landman:
job = geologist;
return job;
等等。此外,您将默认作业用例硬编码给Landman。你可能想要传递一个变量,或者从一个对象中读取玩家表中Job的当前值,然后根据它的当前值进行调整。
switch语句是好的。你有一堆无用的break语句,但没有引起问题(除了使代码可读性降低)。
这就是问题所在:
// if change is True, change the job. If false, simply return the current value
job
是一个局部变量,设置它的唯一效果是后面的return job;
语句。你的评论应该是:
// if change is true, return the new job. If false, simply return the current value
返回job
的新值,但丢弃返回值。
当然,您总是设置job = Landman
,这意味着您总是通过switch
的相同路径。整个函数相当于return Landman;
相关文章:
- C++中std::resize(n)和std::shrink_to_fit之间的区别
- int(c) 和 c-'0' 之间的区别。C++
- 在cuda线程之间共享大量常量数据
- 在c代码之间共享数据的最佳方式
- Mix_Init和Mix_OpenAudio SDL之间的区别是什么
- C++ 使用 assign 函数的字符串与直接使用 '=' 更改值的字符串之间的区别
- VSOMEIP-2个设备之间的通信(TCP/UDP)不工作
- std::atomic和std::condition_variable wait,notify_*方法之间的区别
- 大小相等但成员数量不同的结构之间的性能差异
- 类与私有变量的其他类之间的线程安全性
- 如何在cpp文件之间切换窗口?在Qt中
- 线程之间的布尔停止信号
- 我是C++编程的新手,这些代码之间有什么区别,我应该使用哪一个
- 在 const 函数中通过引用和指针返回之间的区别
- 我想知道长双倍和双倍之间的区别
- 如何防止clang格式在流运算符调用之间添加换行符<<
- 在两台机器之间进行时间戳的最佳c++chrono函数是什么
- 使用.find函数在c++中查找字符和另一个字符之间的大小
- 构造函数和转换运算符之间的重载解析
- C++LinkedList问题.数据类型之间存在冲突?没有匹配的构造函数