blocks|key|3595171|text|我觉得你应该|type|unstyled|depth|inlineStyleRanges|entityRanges|data|3595172|3595173|字符串到实际方法参数的转换和|unordered-list-item|3595174|实际的"work+to+done"|3595175|3595176|使用两种不同的方法。它使您的方法简短，从而使您的代码具有可读性。|3595177|entityMap^0|0|0|0|0|0|0^^$0|@$1|2|3|4|5|6|7|M|8|@]|9|@]|A|$]]|$1|B|3|-4|5|6|7|N|8|@]|9|@]|A|$]]|$1|C|3|D|5|E|7|O|8|@]|9|@]|A|$]]|$1|F|3|G|5|E|7|P|8|@]|9|@]|A|$]]|$1|H|3|-4|5|6|7|Q|8|@]|9|@]|A|$]]|$1|I|3|J|5|6|7|R|8|@]|9|@]|A|$]]|$1|K|3|-4|5|6|7|S|8|@]|9|@]|A|$]]]|L|$]]

I think you should keep

<ul>
<li>the conversion from strings to actual method parameters and</li>
<li>the actual "work to be done"</li>
</ul>

in two separate methods. It keeps your methods short and, thus, your code readable.

blocks|key|4077370|text|我会使用bool.Parse(string)|type|unstyled|depth|inlineStyleRanges|offset|length|style|CODE|entityRanges|data|4077371|entityMap^0|4|I|0^^$0|@$1|2|3|4|5|6|7|H|8|@$9|I|A|J|B|C]]|D|@]|E|$]]|$1|F|3|-4|5|6|7|K|8|@]|D|@]|E|$]]]|G|$]]

I would use <code>bool.Parse(string)</code>

blocks|key|3595197|text|除非你能100%25控制输入，否则永远不要使用bool.Parse。|type|unstyled|depth|inlineStyleRanges|offset|length|style|CODE|entityRanges|data|3595198|无论如何，在处理用户输入时，始终使用TryParse方法。|3595199|bool+result;
if(!bool.TryParse(value,+out+result))
+++throw+new+Exception("There+was+a+problem+with+the+config..+etc..");

Method(result);|code-block|syntax|javascript|3595200|entityMap^0|L|A|0|I|8|0|0^^$0|@$1|2|3|4|5|6|7|O|8|@$9|P|A|Q|B|C]]|D|@]|E|$]]|$1|F|3|G|5|6|7|R|8|@$9|S|A|T|B|C]]|D|@]|E|$]]|$1|H|3|I|5|J|7|U|8|@]|D|@]|E|$K|L]]|$1|M|3|-4|5|6|7|V|8|@]|D|@]|E|$]]]|N|$]]

Never use <code>bool.Parse</code> unless you have 100% control over the input. 

Always use the <code>TryParse</code> methods when you are dealing with user input regardless.

<pre><code>bool result;
if(!bool.TryParse(value, out result))
 throw new Exception("There was a problem with the config.. etc..");

Method(result);
</code></pre>

blocks|key|247834|text|您的对象是否将由配置直接填充？如果是这样，就让它接受一个字符串。否则，单独解析字符串，并保留“bool”参数化方法。|type|unstyled|depth|inlineStyleRanges|entityRanges|data|247835|我可能会检查您是否获得了有效的字符串，例如"True"/"False“，否则抛出一个异常。否则，有人会提供"true”，而你会将其解释为FALSE。|offset|length|style|CODE|247836|你可以选择(作为一个相关点)相对容忍你所接受的内容(例如，修剪空格字符串等)|247837|entityMap^0|0|1W|5|0|0^^$0|@$1|2|3|4|5|6|7|L|8|@]|9|@]|A|$]]|$1|B|3|C|5|6|7|M|8|@$D|N|E|O|F|G]]|9|@]|A|$]]|$1|H|3|I|5|6|7|P|8|@]|9|@]|A|$]]|$1|J|3|-4|5|6|7|Q|8|@]|9|@]|A|$]]]|K|$]]

Is your object going to be directly populated by the config ? If so, get it to accept a string. Otherwise, parse the string separately and maintain your 'bool'-parameterised method.

I would perhaps check that you get a valid string e.g. "True"/"False" and throw an exception if you don't. Otherwise someone will provide "true" and you'll interpret it as <code>FALSE</code>.

You may choose (as a related point) to be relatively tolerant in what you accept (e.g. trim the string of whitespace etc.)

blocks|key|3595251|text|type|unstyled|depth|inlineStyleRanges|entityRanges|data|3595252|，或者我是否应该重构方法，以获取一个字符串作为参数，并在其中进行检查？|blockquote|3595253|3595254|3595255|不是的。单一的责任。该方法不应该是进行参数转换的方法。您可以创建一个接受字符串参数的重载来执行此操作，然后调用接受布尔值的impmlementation，但之后您必须为每个需要非字符串配置参数的方法执行此操作。|3595256|最好在一个地方进行转换，并像这样调用它：|3595257|bool+param+=+Configuration.GetBoolParameter("ParameterName");|code-block|syntax|javascript|3595258|entityMap^0|0|0|0|0|0|0|0^^$0|@$1|2|3|-4|4|5|6|Q|7|@]|8|@]|9|$]]|$1|A|3|B|4|C|6|R|7|@]|8|@]|9|$]]|$1|D|3|-4|4|5|6|S|7|@]|8|@]|9|$]]|$1|E|3|-4|4|5|6|T|7|@]|8|@]|9|$]]|$1|F|3|G|4|5|6|U|7|@]|8|@]|9|$]]|$1|H|3|I|4|5|6|V|7|@]|8|@]|9|$]]|$1|J|3|K|4|L|6|W|7|@]|8|@]|9|$M|N]]|$1|O|3|-4|4|5|6|X|7|@]|8|@]|9|$]]]|P|$]]

<blockquote>
 or should I refactor the Method to get as parameter a string and do the check in there?
</blockquote>

No. Single responsibility. The method should not be the one to do parameter conversion. You could create an overload accepting a string parameter to do that, which then calls the impmlementation accepting a boolean, but then you'll have to do that for every method that requires a non-string configuration parameter. 

It's better to do the conversion in one place, and call it like:

<pre><code>bool param = Configuration.GetBoolParameter("ParameterName");
</code></pre>

blocks|key|247876|text|我更喜欢尽快摆脱这根线。这是XML的一个限制，不需要将这个限制转移到您的方法的“契约”上。|type|unstyled|depth|inlineStyleRanges|entityRanges|data|247877|至于比较，我会说最健壮的是这样的|247878|var+parsedBool+=+parameter.ToUpper()+==+"TRUE";|code-block|syntax|javascript|247879|entityMap^0|0|0|0^^$0|@$1|2|3|4|5|6|7|K|8|@]|9|@]|A|$]]|$1|B|3|C|5|6|7|L|8|@]|9|@]|A|$]]|$1|D|3|E|5|F|7|M|8|@]|9|@]|A|$G|H]]|$1|I|3|-4|5|6|7|N|8|@]|9|@]|A|$]]]|J|$]]

I prefer getting rid of the string as soon as possible. It's a limitation of the XML, no need to transfer the limitation to the "contract" of your method(s).

As for the comparison, I'd say the most robust would be something like

<pre><code>var parsedBool = parameter.ToUpper() == "TRUE";
</code></pre>

blocks|key|4077373|text|Method(bool.Parse(parameter));|type|code-block|depth|inlineStyleRanges|entityRanges|data|syntax|javascript|4077374|unstyled|entityMap^0|0^^$0|@$1|2|3|4|5|6|7|G|8|@]|9|@]|A|$B|C]]|$1|D|3|-4|5|E|7|H|8|@]|9|@]|A|$]]]|F|$]]

<pre><code>Method(bool.Parse(parameter));
</code></pre>

blocks|key|5805959|text|我会保留“我需要一个布尔人”的合同。|type|unstyled|depth|inlineStyleRanges|entityRanges|data|5805960|解析XML，使用上面推荐的方法将该文本值转换为布尔值，然后传递该值。|5805961|entityMap^0|0|0^^$0|@$1|2|3|4|5|6|7|F|8|@]|9|@]|A|$]]|$1|B|3|C|5|6|7|G|8|@]|9|@]|A|$]]|$1|D|3|-4|5|6|7|H|8|@]|9|@]|A|$]]]|E|$]]

I'd retain the contract that says "I need a bool".

Parse the XML, turn that text value into a bool using the method recommended above, and pass that.

blocks|key|3595294|text|我永远不会相信这种类型的输入会足够安全，可以直接传递给bool.Parse，相反，我会做一些更安全的事情，比如这种带有负面回退的积极解析。|type|unstyled|depth|inlineStyleRanges|entityRanges|data|3595295|bool+getBoolean(string+input)+{
++var+value+=+false;
++bool.TryParse(input+??+string.Empty,out+value);
++return+value;
}|code-block|syntax|javascript|3595296|entityMap^0|0|0^^$0|@$1|2|3|4|5|6|7|I|8|@]|9|@]|A|$]]|$1|B|3|C|5|D|7|J|8|@]|9|@]|A|$E|F]]|$1|G|3|-4|5|6|7|K|8|@]|9|@]|A|$]]]|H|$]]

I would never trust that this kind of input would be safe enough to just pass to bool.Parse
Instead I would do something more fail safe like this positive parsing with a negative fallback.

<pre><code>bool getBoolean(string input) {
 var value = false;
 bool.TryParse(input ?? string.Empty,out value);
 return value;
}
</code></pre>

I have a method like this:

<pre><code>public void Method(bool value)
{
}
</code></pre>

Now I want to call this method with a value parsed from an xml file (*.loadtest) which is a string:

<pre><code>&lt;ContextParameter Name="paratemeter" Value="True" /&gt;
</code></pre>

So, should I do this from the caller:

<pre><code>if(parameter.Equals("True")
 Method(true);
else
 Method(false);
</code></pre>

or should I refactor the <code>Method</code> to get as parameter a string and do the check in there?

Best way to use a string as a bool

翻译质量差，导致语言生硬或混乱。

没有提供实际的解决方法或示例。

解答不清晰，无法理解或解决问题。

页面排版不美观，阅读体验差。

文章

问答

视频

学习中心

腾讯云实验室

直播

竞赛

腾讯云代码分析专区

腾讯iOA零信任安全管理系统专区

腾讯云架构师技术同盟交流圈

腾讯云数据库专区

腾讯云顾问专区

腾讯云原生专区

腾讯混元专区

腾讯云TCE专区

腾讯云Lighthouse专区

腾讯云HAI专区

腾讯云Edgeone专区

腾讯云存储专区

腾讯云智能专区

腾讯轻联专区 

腾讯云开发专区

TAPD专区

腾讯轻量云游戏服专区

腾讯云最具价值专家

腾讯云架构师技术同盟

腾讯云创作之星

腾讯云开发者先锋

腾讯云代码助手

云原生构建

TAPD 敏捷项目管理

Cloud Studio

SDK中心

API中心

命令行工具

涵盖代码开发、场景应用、自动测试全流程，助你从零构建专属AI助手

一站式MCP教程库，解锁AI应用新玩法

我有一个这样的方法：public void Method(bool value){}现在我想用一个从xml文件(*.loadtest)中解析出来的值调用这个方法，这个值是一个字符串：<ContextParameter Name="paratemeter" Value="True" />那么，我是否应该从调用者执行此操作：if(parameter.Equals("True") Method(t

问将字符串用作布尔值的最佳方法
EN

回答 9

Stack Overflow用户

Stack Overflow用户

Stack Overflow用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问将字符串用作布尔值的最佳方法EN

回答 9

Stack Overflow用户

Stack Overflow用户

Stack Overflow用户

社区

活动

圈层

关于

腾讯云开发者

热门产品

热门推荐

更多推荐

问将字符串用作布尔值的最佳方法
EN