|
nstk wrote: I wrote this message because I wanted to get a feedback from the forum, since I
plan to write to my boss that this piece of code is worthless and should be
rewritten with a new concept.
Err..no.
You have some code that is unlikely to have an Object Oriented design.
That however doesn't make it "worthless".
If the code is being used in the business then it is far from "worthless".
If it isn't being used then it shouldn't be changed at all.
nstk wrote: We always deal with a lot of bugs and problems, no one is completely sure of
what is happening, we have not a complete picture of the structure, we are
steadily debugging step by step hoping to find out what changed that variable at
the last moment(because any class can do that) and even the person who did
program it and recently made some changes, had to search for some unexpected new
errors that arose.
Now those are problems. And those are problems that OOP will NOT fix.
However there are other practices that if put into place MIGHT fix those problems. Those practices can be applied to a OO and structured app. In certain cases it might be easier to apply them to OO code. But not necessarily.
It might the case that the code is badly implemented and thus it is more cost effective to rewrite it. However based on the problems listed above a rewrite MUST insure that the same problems are not allowed to exist with the rewritten code.
|
|
|
|
|
im developing a application ... that has some softwares which i have to install silentlt back side while installing the other on user selection... im developing using visual studio 2010 .. c# . please any one help
|
|
|
|
|
Why?
That sounds a lot like the kind of installation I really don't want to run...
Ideological Purity is no substitute for being able to stick your thumb down a pipe to stop the water
|
|
|
|
|
Are you talking about prerequisites such as installing SQL Server Express?
|
|
|
|
|
Here[^] is the standard list of command line operations for an installer. Have a look at the /quiet command mode.
|
|
|
|
|
Here is my code:
//Line below shows an email msg obj with To segment being an array (could have null to x number of receipients/emails):
emailMessage.EmailParamHeaders.To
//Below is creation of mail message obj:
MailMessage m = new MailMessage();
//Line blow is adding each To email address to the "m" object
emailMessage.EmailParamHeaders.To.ToArray().ToList().ForEach(x => m.To.Add(x));
How can I only add the elements of the array "To" where value is not null?
Thank you in advance.
|
|
|
|
|
try this
emailMessage.EmailParamHeaders.To.ToArray().ToList().ForEach(x => {if(m != null) m.To.Add(x);});
|
|
|
|
|
Thank you so much for the answer, that worked with minor change:
emailMessage.EmailParamHeaders.To.ToArray().ToList().ForEach(x => { if (x != null) m.To.Add(x); });
|
|
|
|
|
Why are you converting to array, and then to a list? Also, if your email message supports AddRange , an alternative is to try the following?
var addresses = (from p in emailMessage.EmailParamHeaders.To
where p != null
select p).ToList();
if (addresses != null && addresses.Count>0)
m.To.AddRange(addresses); Depending what's going on inside the Add method, you can often find that an AddRange works more efficiently.
|
|
|
|
|
In a C#.net 2010 application, I would like a suggestion on how to complete the following task:
In a string field, I need to make certain the only values that are returned by a routine are:
1. alphabetic characters only,
2. The result must be in uppper case,
3. No other characters are allowed.
Can you tell me how to accomplish this task?
|
|
|
|
|
Check your results for validty before you return the value from the method??
You can look into using Code Contracts[^] to simplify this process a bit.
|
|
|
|
|
I'd go with a Regex:
string input = "1234asdfASDFG,../;[][";
string output = Regex.Replace(input, "[^A-Z]", "");
Ideological Purity is no substitute for being able to stick your thumb down a pipe to stop the water
|
|
|
|
|
I want to place design surface on button control, which is located on design surface. Is there a way to place design surface inside other design surface?
I create new design control(design surface with button control as root component) in CreateInstance() of other design surface and have next exception:
The CurrentContext property of the LicenseManageris currently lockedand cannot be changed.
I think, during executing of CreateInstance()(of main design surface) current design host(of main design surface) is locked, therefore I can't create other design surface.
modified 12-Jun-12 3:54am.
|
|
|
|
|
Hi All,
I am writing an application and need to save some data with an incremental number.
My first thought was to use an integer that be could incremented and shoved out disk, now I am trying to do thing properly so I have come up with the following
String Path = Directory.GetCurrentDirectory();
string[] file_text = Directory.GetFiles(@Path, "*.txt");
string Name;
int FileTotal = 0;
for (int i = 0; i <= file_text.Length - 1; i++)
{
Name = file_text[i].Substring(Path.Length+1);
FileTotal++;
}
MessageBox.Show(FileTotal.ToString());
FileTotal++;
MessageBox.Show("Next File to Written is " + FileTotal.ToString());
It looks for the file (in the bit of example code .txt ...) adds one to the total for the next one. Is this the 'correct' way of doing things or am I creating a kludge that will come back and bite me later.
Glenn
|
|
|
|
|
What you are trying to do is unclear; please explain more clearly.
glennPattonWork wrote: Is this the 'correct' way of doing things or am I creating a kludge that will
come back and bite me later.
It might be OK, but I can't say for sure until I know what you are trying to accomplish.
Is there a database involved?
|
|
|
|
|
Hi,
Sorry about that I had not really had chance to proof read it when it got posted (mix of Web connection and finger trouble) What I am aiming to do is to create a load data files
such as Data001.msg, Data002.msg etc at the moment I have it updating a variable that goes to an ini file (I know, I am under orders to make an .ini, you bring up XML but no! separate story)I want the program to look at the location say there are six file with an .msg extension I would like the variable to be seven and create Data007.msg and on. I was trying to find a way to do this that wasn't too bad. Sorry, there is no database involved at this stage!
Glenn
|
|
|
|
|
glennPattonWork wrote: say there are six file ... I would like the variable to be seven
Well, then what about int FileTotal = file_text.Length ; ?
|
|
|
|
|
Thanks for that, I still wonder if I would better suited to something other than Windows programming!
|
|
|
|
|
|
Hmmm, LINQ never messed with it, I have a look....
|
|
|
|
|
|
Thanks for that when I have the time I will take a look at using LINQ, PIEBALDconsult's method works for what I am trying to do, can you recommend any sources for LINQ ?
|
|
|
|
|
|
I had found the MSDN stuff, so it looks like I have been LINQ'ing for a time with out realizing it....
|
|
|
|
|