1) Yes, strings are (generally) interned (by default, thanks @Eric), so any instance of the same "something"
would point to the same object, therefore you'd be ok. This is very bad practice though, because someone else, in another library for example, could lock on your string, thus giving the potential for deadlocks. See here: Using string as a lock to do thread synchronization
You should do this:
private static readonly object mutex = new object();
lock(mutex)
{
//....
}
2) Yes, same with all statements. Anything* where you have:
{
// One line
}
could just be
// One line
*Almost anything, see @LukeH's example of the catch
block, which requires the braces.
与恶龙缠斗过久,自身亦成为恶龙;凝视深渊过久,深渊将回以凝视…