你的两条指令在 if 两边都是一样的！我怀疑你粘贴错了东西。让我们假设它们是不同的（"Y" 和 "N"）:) 您可以简单地使用 三元运算符：
while (vIter.hasNext()) {
Long actId =(Long)vIter.next();
vRetActIds.put(actId, actId == -1 ? "N" : "Y");
}
编辑：从您的评论来看，您似乎确实想将值设置为 'N ' 每次？在这种情况下，您根本不需要 if...
while (vIter.hasNext()) {
vRetActIds.put((Long)vIter.next(), "N");
}
Your two instructions are the same on both sides of the if! I suspect you pasted something wrong. Let's assume they're different ("Y" and "N") :) You can write it simply using the ternary operator:
while (vIter.hasNext()) {
Long actId =(Long)vIter.next();
vRetActIds.put(actId, actId == -1 ? "N" : "Y");
}
EDIT: From your comments, it seems like you actually want to set the value to 'N' every time? In that case, you don't need the if at all...
while (vIter.hasNext()) {
vRetActIds.put((Long)vIter.next(), "N");
}
Long MINUS_ONE = Long.valueOf(-1);
while (Long actId : vIter)
{
vRetActIds.put(actId, MINUS_ONE.equals(actiId) ? "N" : "Y");
}
If we can assume that you are using a Collection of some sort in the new java, something like this would be more concise and won't crash due to null pointers?
Long MINUS_ONE = Long.valueOf(-1);
while (Long actId : vIter)
{
vRetActIds.put(actId, MINUS_ONE.equals(actiId) ? "N" : "Y");
}
If that's the case (and it, or something like it, almost has to be the case), then you can forgo the iterator entirely and make your code much easier to read, thus:
for (Long actId: list)
vRetActIds.put(actId, "N");
发布评论
评论（4）
你的两条指令在 if 两边都是一样的！我怀疑你粘贴错了东西。让我们假设它们是不同的（
"Y"和"N"）:) 您可以简单地使用 三元运算符：编辑：从您的评论来看，您似乎确实想将值设置为 'N ' 每次？在这种情况下，您根本不需要
if...Your two instructions are the same on both sides of the if! I suspect you pasted something wrong. Let's assume they're different (
"Y"and"N") :) You can write it simply using the ternary operator:EDIT: From your comments, it seems like you actually want to set the value to 'N' every time? In that case, you don't need the
ifat all...如果我们可以假设您在新的java中使用某种集合，那么这样的东西会更简洁并且不会因空指针而崩溃？
If we can assume that you are using a Collection of some sort in the new java, something like this would be more concise and won't crash due to null pointers?
三元条件语句就可以了。
Ternary conditional statement would be fine.
假设您从列表之类的地方获取迭代器：
如果是这种情况（并且它或类似的东西几乎必须是这种情况），那么您可以完全放弃迭代器并使您的代码更易于阅读， 因此：
Let's assume you're getting your iterator from somewhere like a List:
If that's the case (and it, or something like it, almost has to be the case), then you can forgo the iterator entirely and make your code much easier to read, thus: