C#

sujeet321 wrote:

How to optimise code

private void myDataGridView_CellClick(object sender, DataGridViewCellEventArgs e)
    {
    DataGridView dgv = sender as DataGridView;
    if (dgv != null)
        {
        dgv.Rows[e.RowIndex].DefaultCellStyle.BackColor = Color.Red;
        }
    }

gridView.Rows[e.RowIndex].DefaultCellStyle.SelectionBackColor = Color.Red; // or other colour

gridView.Rows[e.RowIndex].Selected = true;

Dave Kreskowiak wrote:

The same as usually.

<br />
Monitor.Enter(o);<br />
S0;<br />
try {<br />
    S1;<br />
} finally {<br />
    Monitor.Exit(o);<br />
}<br />

<br />
Update 4/17/08: This was indeed fixed for the X64 JIT in Visual Studio 2008.  Note that when compiling C# code targeting both X86 and X64, if you do not use the /o+ switch, this problem can still occur due to extra explicit NOPs inserted before the try.<br />
<br />
The framework implements a method Monitor.ReliableEnter, by the way, that could be used to avoid orphaning locks in the face of thread aborts, but it's internal to mscorlib.dll.  It sets an out parameter within a region of code that cannot be interrupted by a thread abort, which the caller can then check inside the finally block.  The acquisition then gets moved inside so that, if the CALL is reached, the finally block is guaranteed to always run.  You'd then write this instead:<br />
<br />
    bool taken;<br />
    try {<br />
        Monitor.ReliableEnter(o, out taken);<br />
        S1;<br />
    } finally {<br />
        if (taken)<br />
            Monitor.Exit(o);<br />
    }<br />

devvvy wrote:

I don't like "Usually" ...

<br />
class Program<br />
    {<br />
        static object obj = new object();<br />
<br />
        static void Main(string[] args)<br />
        {            <br />
            try<br />
            {<br />
                for (int i = 0; i < 1000; i++)<br />
                {<br />
                    Thread thRun = new Thread(new ThreadStart(ThreadFunc));<br />
                    thRun.Start();<br />
<br />
                    Thread.Sleep(300);<br />
<br />
                    thRun.Abort();<br />
<br />
                    lock (obj) // IF ThreadFunc didn't release lock on obj properly, then you wouldnt see following line<br />
                    {<br />
                        Console.WriteLine(i + ": Done locking obj from main");<br />
                    }<br />
                }<br />
<br />
                Console.WriteLine("Hit any key to exit");<br />
                Console.ReadLine();<br />
            }<br />
            catch (Exception Ex)<br />
            {<br />
                Console.WriteLine("Error in main: " + Ex.Message);<br />
            }<br />
<br />
            return;<br />
        }<br />
<br />
        static void ThreadFunc()<br />
        {<br />
            lock (obj)<br />
            {<br />
                using (DataContainer SomeContainer = new DataContainer())<br />
                {<br />
                    Console.WriteLine("ThreadFunc lock begins");<br />
                    Thread.Sleep(1000 * 10);<br />
                }<br />
            }<br />
            Console.WriteLine("ThreadFunc lock released");<br />
<br />
            return;<br />
        }<br />
    }<br />
<br />
    class DataContainer : IDisposable<br />
    {<br />
        protected bool m_isDisposed = false;<br />
<br />
        public void Dispose()<br />
        {<br />
            Dispose(true);<br />
            GC.SuppressFinalize(this);<br />
        }<br />
<br />
        ~DataContainer()<br />
        { // finalizer<br />
            Dispose(false);<br />
        }<br />
<br />
        protected virtual void Dispose(bool disposing)<br />
        {<br />
            Console.WriteLine("DataContainer.Dispose: " + disposing);<br />
<br />
            if (m_isDisposed)<br />
                return;<br />
<br />
            if (disposing)<br />
            {<br />
                // Dispose managed data<br />
                //m_managedData.Dispose();<br />
            }<br />
            // Free unmanaged data<br />
            //DataProvider.DeleteUnmanagedData(m_unmanagedData);<br />
            m_isDisposed = true;<br />
        }<br />
    }<br />