static question

[Old Gun]

I am not sure my code,Is it right? public class Macro_RemoveExcelDefaultGridLines { /// /// Excel Macro : Remove Microsoft Excel's Default Grid Lines /// private static string _Name = "RemoveDefaultGrid"; private static string _Body ="" + " Sub RemoveDefaultGrid()\n" + " Cells.Select\n" + " With ActiveWindow\n" + " .DisplayGridlines = False\n" + " .DisplayZeros = False\n" + " End With\n" + "End Sub\n"; public static string Name { get{return _Name;} } public static string Body { get{ return _Body;} } }

[alexey N]

Just try it directly in Excel, and see what you get. Best regards, Alexey.

[Old Gun]

My Excel macro is test in excel,my mean is i am not sure my c# class,I use static private variable and property,I think there is somthing wrong.

[alexey N]

Old Gun wrote:

I use static private variable and property,I think there is somthing wrong.

Looks like all Ok. Try this, and if you'll get trobles, change properties to functions, or make them non-static. Best regards, Alexey.

[Guffa]

I see nothing wrong with the code. Why do you think that there is something wrong with it? If the class is only supposed to be a container for those static properties, you should make it sealed so that it can't be inherited, and make an empty private constructor to override the default empty public constructor, so that it's obvious that one should not create instances of the class. --- b { font-weight: normal; }

[Old Gun]

Thanks alexey N and Guffa,Both of you helped me.

[Old Gun]

Hi,Guffa,I can't understand "make an empty private constructor to override the default empty public constructor, so that it's obvious that one should not create instances of the class. ",Can you give me more tips?