Moonshield

[C#] On Properties surrounding arrays of bytes

2010-03-04T01:11:36-08:00

I used static property so you dont have to create an instance everytime. I moved the array creation in a static constructor so we don't have to recreate the same array everytime someone access the property. Also, the property use a private set so the collection can be changed from outside but its values could be changed though. If you dont want the values to be changed, just moved your array initialization inside your property, so you will end up with only 2 properties instead of 2 properties and 2 methods. Hope it helps.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            byte[] rich = Attivazioni.CodiceRichiesta;
            byte[] gest = Attivazioni.CodiceModuloGestionale;
        }
    }

    public partial class Attivazioni
    {
        public static byte[] CodiceRichiesta { get; private set; }
        public static byte[] CodiceModuloGestionale { get; private set; }

        static Attivazioni()
        {
            // CodiceRichiesta
            byte[] aRich = new byte[16];
            aRich[0] = 99;
            aRich[1] = 101;
            aRich[2] = 115;
            aRich[3] = 96;
            aRich[4] = 105;
            aRich[5] = 111;
            aRich[6] = 110;
            aRich[7] = 97;
            aRich[8] = 108;
            aRich[9] = 101;
            aRich[10] = 99;
            aRich[11] = 112;
            aRich[12] = 113;
            aRich[13] = 100;
            aRich[14] = 100;
            aRich[15] = 100;
            CodiceRichiesta = aRich;

            // CodiceModuloGestionale
            byte[] aGest = new byte[16];
            aGest[0] = 103;
            aGest[1] = 101;
            aGest[2] = 115;
            aGest[3] = 116;
            aGest[4] = 105;
            aGest[5] = 111;
            aGest[6] = 110;
            aGest[7] = 97;
            aGest[8] = 108;
            aGest[9] = 101;
            aGest[10] = 111;
            aGest[11] = 112;
            aGest[12] = 113;
            aGest[13] = 114;
            aGest[14] = 115;
            aGest[15] = 116;
            CodiceModuloGestionale = aGest;
        }      
    }
}

[C#] On Find the list's last satisfying element and skip the rest

2009-04-22T04:47:57-07:00

I haven't fully tested yet but you could do something similar to this. I changed the IEnumerator for IEnumerable as your comment was already suggesting it..

// this should be IEnumerable.
private IEnumerable<DevicePosition> ValidPositions()
{
    // stub
    return null;
}

public bool FindRecentRepresentatives(int milliseconds, out DevicePosition current, out DevicePosition previous)
{
    if (milliseconds <= 0)
        throw new ArgumentOutOfRangeException("milliseconds");

    var aValidPositions = ValidPositions();
    current = aValidPositions.First();
    previous = aValidPositions.Last(x => x.Time.Ticks > (x.Time.Ticks - TimeSpan.FromMilliseconds(milliseconds).Ticks));

    return (current != null) && (previous != null);
}

[C#] On Looking for a pattern to handle this situation

2009-03-27T01:39:34-07:00

I've already posted something similar, you can take a look at http://refactormycode.com/codes/493-twitter-esque-relative-dates
The code below is probably what I would do, it is very similar to the code above but without the overhead

using System;
using System.Collections.Generic;
using System.Linq;

namespace ConsoleApplication4
{
    class Program
    {
        private static Dictionary<string, Action<string>> _QueryPriority;

        static void Main(string[] args)
        {
            if (_QueryPriority == null)
            {
                _QueryPriority = new Dictionary<string, Action<string>>();
                _QueryPriority.Add("Foo", DoFooWork);
                _QueryPriority.Add("Bar", DoBarWork);
                _QueryPriority.Add("Beer", DoBeerWork);
                _QueryPriority.Add("Coffee", DoCoffeeWork);
            }

            var action = Program._QueryPriority.FirstOrDefault(x => !string.IsNullOrEmpty(Request.QueryString[x]));         
            if (action.Key != null)
                action.Value.Invoke(action.Key);
        }

        private static void DoFooWork(string pParam) { Console.WriteLine(pParam); }
        private static void DoBarWork(string pParam) { Console.WriteLine(pParam); }
        private static void DoBeerWork(string pParam) { Console.WriteLine(pParam); }
        private static void DoCoffeeWork(string pParam) { Console.WriteLine(pParam); }
    }
}

[C#] On Showing several datetimes (event/concert style)

2009-03-27T01:00:07-07:00

Probably not perfect but you have the main idea and you can modify it to your needs.
I suggest to use the StringBuilder.AppendFormat method instead StringBuilder.Append(string.format("",something)), it's too much overhead :)

using System;
using System.Collections.Generic;
using System.Globalization;
using System.Text;

namespace ConsoleApplication3
{
    class Program
    {
        static void Main(string[] args)
        {
            DateTime oDate = DateTime.Now;
            List<DateTime> aDate = new List<DateTime>()
            {
                oDate.AddHours(1),
                oDate,
                oDate.AddDays(10),
                oDate.AddYears(-1),
                oDate.AddDays(-10),
                oDate.AddYears(2)
            };

            string Result = Program.GetDateString(aDate);
            foreach (string Date in Result.Split(','))
                Console.WriteLine(Date.Trim());

            Console.ReadLine();
        }

        private static string GetDateString(List<DateTime> paDate)
        {
            StringBuilder oSb = new StringBuilder();

            if (paDate != null && paDate.Count > 0)
            {
                int CurrentYear = int.MinValue;

                paDate.Sort();
                paDate.ForEach(oDate =>
                {
                    if (oDate.Year != CurrentYear)
                    {
                        CurrentYear = oDate.Year;
                        oSb.AppendFormat(CultureInfo.InvariantCulture, "{0:MMM dd yyyy HH:mm}, ", oDate);
                    }
                    else
                        oSb.AppendFormat(CultureInfo.InvariantCulture, "{0:MMM dd HH:mm}, ", oDate);
                });

                if (oSb.Length > 2)
                    oSb.Remove(oSb.Length - 2, 2);
            }

            return oSb.ToString();
        }
    }
}

[C#] On C# HTML Encoding

2009-03-08T20:36:57-07:00

I know what it means and should be used for. For exemple : I think it should not be used to get the length of a string like in posted code var l = b.Length;, use and simple int. And for long type name, using keyword should be the right solution IMHO

[C#] On C# HTML Encoding

2009-03-06T00:13:18-08:00

Use 'else if', so it doesn't need to evaluate other conditions when it succeeds to the first one. As Rikkus said, you better use the method provided in the framework. Still don't know why people use the var keyword when type is known, I'm curious. It is very usefull with Linq queries.. aside that

public static string CleanHtml(string pHtml)
{
    StringBuilder oSb = new StringBuilder();

    if (!string.IsNullOrEmpty(pHtml))
    {
        foreach (char c in pHtml)
        {
            // Known Replacements
            if (c == '&')
                oSb.Append("&amp;");
            else if (c == '<')
                oSb.Append("&lt;");
            else if (c == '>')
                oSb.Append("&gt;");
            else if (c == '`')
                oSb.Append("'");
            else if ((int)(c) > 127)
            {
                // Always Encode Extended Ascii Codes
                oSb.AppendFormat("&#{0};", (int)c);
            }
            else
            {
                // If nothing was added, add the original char
                oSb.Append(c);
            }
        }
    }

    return oSb.ToString();
}

[C#] On Help me with ineritance or the best way to do this

2009-02-06T01:36:35-08:00

I would probably do something similar to this

public class MvNetDB
{
    protected mvAccount _account = null;
    private static Logger logger = LogManager.GetCurrentClassLogger();

    protected void Login()
    {
        _account = new mvAccount(ConfigHelper.ServerLogin, ConfigHelper.ServerAddress);
    }

    protected void Logout()
    {
        if (_account != null)
        {
            _account.Logout();
            _account.Dispose();
        }
    }

    protected mvItem ReadRecord(string filename, string recordID)
    {
        mvItem item = null;

        try
        {
            this.LoginToDB();
            item = _account.FileOpen(filename).Read(recordID);
        }
        catch { }
        finally
        {
            this.LogoutFromDB();
        }

        return item;
    }
}

public class MvNetServiceCallRepository : MvNetDB, IServiceCallRepository
{
    private Customer GetCustomerByID(string custID)
    {
        Customer oCustomer = null;

        mvItem oItem = ReadRecord("CUSTOMER", custID);
        if (oItem != null)
        {
            oCustomer = new Customer()
            {
                ID = custID,
                Name = oItem["1"].ToString(),
                Address = oItem["2"].ToString(),
                City = oItem["3"].ToString(),
                State = oItem["4"].ToString(),
                Zip = oItem["5"].ToString(),
                CustCode = GetCustCodeByID(oItem["42"].ToString())
            };
        }

        return oCustomer;
    }

    private CustCode GetCustCodeByID(string custCodeID)
    {
        CustCode oCustCode = null;

        mvItem oItem = ReadRecord("CUST.CODE", custCodeID);
        if (oItem != null)
        {
            oCustCode = new CustCode()
            {
                ID = custCodeID,
                Name = oItem["1"].ToString()
            };
        }

        return oCustCode;
    }
}

[C#] On Polymorphic XML serializer

2009-01-22T03:45:27-08:00

Unfortunately, your code wasn't working for me. I think that my default xml encoding differs from yours so that your hard-coded string manipulation isnâ€™t working properly. On the other side, I worked on a cleaner way to achieve what you want. It can serializes an object of type A, then deserializes it back to the same type or to an object of type B : A like you mentioned. The key of the implementation is to use a general root name (see XmlRoot in my code) for both objects. Enough talking, see the code below :)

using System;
using System.IO;
using System.Xml.Serialization;

namespace ConsoleApplication1
{
    public class Program
    {
        static void Main(string[] args)
        {
            // Serialize a Person object and deserialize it back to a Person or an Employee
            string SerializedObject = SerializedObject = CustomSerializer.Serialize(new Person() { Id = 1, FirstName = "Peter", LastName = "Pan" });
            Person oPerson = CustomSerializer.DeSerialize<Person>(SerializedObject);
            Employee oEmployee = CustomSerializer.DeSerialize<Employee>(SerializedObject);
            
            // At this point the Employee.Title is empty, everything's normal. Then we will
            // serialize the same Employee object and deserialize it back to a Person object
            oEmployee.Title = "Hero";            
            SerializedObject = CustomSerializer.Serialize(oEmployee);
            oPerson = CustomSerializer.DeSerialize<Person>(SerializedObject);

            Console.ReadLine();
        }
    }

    /// <summary>
    /// Contains information about a person.
    /// </summary>
    /// <remarks>
    /// The XmlRoot attribute allow to serialize a Person object 
    /// and to deserialize it back to a Person or an Employee 
    /// object without "manipulating" the xml string.
    /// </remarks>
    [Serializable, XmlRoot("Base")]
    public class Person
    {
        public int Id { get; set; }
        public string FirstName { get; set; }
        public string LastName { get; set; }
    }

    /// <summary>
    /// Contains information about an employee.
    /// </summary>
    /// <remarks>
    /// The XmlRoot attribute allow to serialize a Person object 
    /// and to deserialize it back to a Person or an Employee 
    /// object without "manipulating" the xml string.
    /// </remarks>
    [Serializable, XmlRoot("Base")]
    public class Employee : Person
    {
        public string Title { get; set; }
    }

    /// <summary>
    /// 
    /// </summary>
    public class CustomSerializer
    {
        /// <summary>
        /// Serialize an object to xml
        /// </summary>
        /// <param name="pSource">Object Type</param>
        /// <returns>String</returns>
        public static string Serialize(object pSource)
        {
            // Normally I'd use this 4 lines but since you need cleaner xml see below
            //StringWriter oWriter = new StringWriter();
            //XmlSerializer oSerializer = new XmlSerializer(pSource.GetType());
            //oSerializer.Serialize(oWriter, pSource);
            //return oWriter.ToString();

            XmlSerializerNamespaces oNamespaces = new XmlSerializerNamespaces();
            oNamespaces.Add("", "");

            StringWriter oWriter = new StringWriter();
            XmlSerializer oSerializer = new XmlSerializer(pSource.GetType());
            oSerializer.Serialize(oWriter, pSource, oNamespaces);

            string Serialized = oWriter.ToString();
            return Serialized.Remove(0, Serialized.IndexOf('<', 1));
        }

        /// <summary>
        /// Deserialize an xml string to an object
        /// </summary>
        /// <param name="pXmlSource">Xml Source</param>
        /// <returns>Deserialized object</returns>
        public static T DeSerialize<T>(string pXmlSource)
        {
            return (T)(new XmlSerializer(typeof(T)).Deserialize(new StringReader(pXmlSource)));
        }
    }
}

[C#] On Filterting a List<> with a Regular Expression

2008-11-27T02:15:57-08:00

Some advices : The way you are using the regular expression is equivalent to using the regular way to do a string comparaison. Also, I would suggest also to filter documents in your document layer by adding the name and author to your GetDocuments(username) method, I would be more efficient as your method can retrieve 500 documents from the database to finally display one or two results for exemple.

List<Document> aDocuments = documentLayer.GetDocuments(username);

string docName = (string)Session["DocumentName"];
string docAuthor = (string)Session["DocumentAuthor"];
bool docNameEmpty = string.IsNullOrEmpty(docName);
bool docAuthorEmpty = string.IsNullOrEmpty(docAuthor);

if (docNameEmpty && docAuthorEmpty)
    gridDocuments.DataSource = aDocuments;
else
{
    List<Document> aDocumentsFilter = new List<Document>();
    aDocuments.ForEach(oDoc =>
        {
            if ((!docNameEmpty && !docAuthorEmpty && docName == oDoc.Name && docAuthor == oDoc.Author)
             || (docNameEmpty && docAuthor == oDoc.Author)
             || (docAuthorEmpty && docName == oDoc.Name))
                aDocumentsFilter.Add(oDoc);
        });
    gridDocuments.DataSource = aDocumentsFilter;
}

gridDocuments.DataBind();

[C#] On Drop all Indexes in SQL Server

2008-11-11T23:42:53-08:00

You can also reuse the example I gave you days ago
http://refactormycode.com/codes/497-backup-all-ms-sql-server-databases

[C#] On DirectoryInfo.CopyTo

2008-11-11T23:34:34-08:00

Just add the parameters validations you need and you'll be good I think

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            FileUtils.CopyDirectory(@"C:\MyFiles\Test", @"C:\MyFiles\Test2", true);
            Console.WriteLine("Already done...");
            Console.ReadLine();
        }

        public static class FileUtils
        {
            public static void CopyDirectory(string pSource, string pDestination, bool pOverrideExistingFile)
            {
                (from p in Directory.GetFiles(pSource, "*.*", SearchOption.AllDirectories)
                 group p by Path.GetDirectoryName(p) into GroupByDirectory
                 select new
                 {
                     Directory = GroupByDirectory.Key.Replace(pSource, pDestination),
                     Files = (from p in GroupByDirectory
                              select new
                              {
                                  SourceFile = p,
                                  Destination = p.Replace(pSource, pDestination)
                              }).ToList()
                 })
                 .ToList()
                 .ForEach(item =>
                 {
                     if (!Directory.Exists(item.Directory)) 
                         Directory.CreateDirectory(item.Directory);
                     item.Files.ForEach(fileInfo => File.Copy(fileInfo.SourceFile, fileInfo.Destination, pOverrideExistingFile));
                 });
            }
        }
    }
}

[C#] On Strip Html Comments

2008-11-11T05:14:22-08:00

I ran the test unit on the regex : 7/15 passed. If you still want to do it with a loop I've cleaned your code a little. I can take a look tomorrow too. I'm too tired to think tonight :)

public static class HtmlHelper
{
    public static string StripHtmlComments(string pHtml)
    {
        if (pHtml == null)
            throw new ArgumentNullException("pHtml");

        // Contains cleaned content
        StringBuilder oSb = new StringBuilder();

        // Not comment --> skip
        if (pHtml.IndexOf("<!", StringComparison.Ordinal) < 0)
            oSb.Append(pHtml);

        // Not comment --> skip
        if (oSb.Length == 0)
        {
            bool InHtmlComment = false;
            bool InHtmlTag = false;
            char CurrentChar;
            char NextChar;

            for (int CharIndex = 0; CharIndex < pHtml.Length; CharIndex++)
            {
                CurrentChar = pHtml[CharIndex];

                if (!InHtmlComment && !InHtmlTag)
                {
                    if (CurrentChar == '<' && (CharIndex + 1 < pHtml.Length))
                    {
                        NextChar = pHtml[CharIndex + 1];
                        if (NextChar == '!')
                        {
                            InHtmlComment = true;
                            continue;
                        }
                        else if (NextChar.IsEnglishLetter())
                            InHtmlTag = true;
                    }
                }
                else if (InHtmlComment)
                {
                    if (CurrentChar == '>')
                    {
                        if (InHtmlComment)
                        {
                            InHtmlComment = false;
                            continue;
                        }
                    }
                    continue;
                }
                else if (InHtmlTag && CurrentChar == '>')
                    InHtmlTag = false;

                oSb.Append(CurrentChar);
            }

        }

        // One entry point, one return point
        return oSb.ToString();
    }

    private static bool IsEnglishLetter(this char nextChar)
    {
        return ('a' <= nextChar && nextChar <= 'z') || ('A' <= nextChar && nextChar <= 'Z');
    }
}

[C#] On Sorting by dependencies

2008-11-05T23:18:39-08:00

I'd like to help but I'm not sure to understand what you try to achieve. Can you provide more details or provide a new Dictionnary to test with that produce something else then 1,2,3,4,5 when you execute it, it would help me to understand.

[C#] On Generating a list of time values

2008-10-23T02:35:29-07:00

I have the same console result as your code with this.
Not sure about the appended 'd', it is never displayed in my test. I'd like more explanations to know when to display it or not

int interval = 60; //in minutes
DateTime start = DateTime.Today;

for (int IntervalIndex = 0; IntervalIndex < (1440 / interval); IntervalIndex++)
    Console.WriteLine("local time: {0:HHmm}, GMT value: {1}", (start = start.AddMinutes(IntervalIndex == 0 ? 0 : interval)), start.ToUniversalTime());
Console.WriteLine("local time: 2400, GMT value: {0}", start.AddMinutes(interval).ToUniversalTime());

[C#] Capitalize directory name with LINQ

2008-10-02T01:31:54-07:00

I wrote a method to capitalize the first letter of each directories for a given path. I used it at the same time to work on my LINQ skills so don't remove it when refactoring please, improve it if necessary. I need to perform two different sub query to know if the word to capitalize is the first one or not, but I'm not sure if all this is really necessary. Also, I had to add a \ or a blank space before to word because I don't want to replace all 'a' because there is the word a in the folder name. See third directory name.

C:\MyFiles\Music\candlemass- ancient Dreams
C:\MyFiles\test\die apokalyptischen reiter - licht
C:\MyFiles\Music\a Dozen Furies - a Concept From Fire

I commented the two lines that rename the directory because I don't want to be responsable for renaming all your folders :)

public static void Capitalize()
{
    TextInfo oTextInfo = CultureInfo.CurrentCulture.TextInfo;
    StringBuilder oSb = new StringBuilder();
    string Folder;

    var DirectoryToCapitalize =
       (from p in Directory.GetDirectories(@"C:\MyFiles", "*.*", SearchOption.AllDirectories)
        let directories = p.Substring(p.LastIndexOf("\\") + 1)
        let words = directories.Split(' ')                    
        from word in words                   
        where word.Length > 0 && char.IsLower(word[0])
        group word by p into groupedChanges                
        select new
        {
            Folder = groupedChanges.Key,
            Words = (   from p in groupedChanges
                        where groupedChanges.Key.IndexOf("\\" + p) > 0
                        select "\\" + p)
                        .Union( from p in groupedChanges
                                where groupedChanges.Key.IndexOf(" " + p) > 0
                                select " " + p)            
        });
    
    foreach (var GroupItem in DirectoryToCapitalize)
    {
        Folder = GroupItem.Folder;

        foreach (var Word in GroupItem.Words)
            Folder = Folder.Replace(Word, oTextInfo.ToTitleCase(Word));

        oSb.AppendLine(GroupItem.Folder); // Original path
        oSb.AppendLine(Folder); // New path

        // Can't rename C:\MyFiles\test to C:\MyFiles\Test
        // We need to rename the folder with a temp name and rename it back with capitalized words
        //Directory.Move(Folder, Folder + "1");
        //Directory.Move(Folder + "1", Folder);
    }

    Console.WriteLine(oSb.ToString());
}

[C#] On Backup All MS SQL Server Databases

2008-09-19T23:55:56-07:00

Hi GateKiller

If you don't have many databases, lets say less than 75, the code below would do the job without cursor. (In general, cursor are not a good way to do thing). If you have too many databases, your script is good, I'll just move the condition in your if statement to the query that the cursor is made of as in the exemple below.

Hope you'll like it !

Use Master

Declare @ToExecute VarChar(8000)

Select @ToExecute = Coalesce (@ToExecute + 'Backup Database ' + [Name] + ' To Disk = ''D:\Backups\Databases\' + [Name] 	+ '.bak'' With Format;' + char(13),'')
From Master..Sysdatabases
Where [Name] Not In ('tempdb') And databasepropertyex ([Name],'Status') = 'online'

--Print @ToExecute
Exec (@ToExecute)