Re: Execute And wait [modified] - C# Discussion Boards

1-Dec-10 5:08

I'm using the following code to run REGASM.EXE to register and create TLB files for some .NET assemblies. The output message indicates that the registration of each assembly was successful however since they are under an installation folder in C:\Program Files, the output message from REGASM doesn't show the full path of the assembly that was registered or exported as a tlb. It just shows 'C:\Program'. Is there any way to fix the output message so that it shows the full path?

private static bool ExecuteAndWait(string _FileToExecute, string _CommandLine, ref string _outputMessage, ref string _errorMessage)
{

    Process p = null;

    try
    {
        p= new Process();
        p.StartInfo.FileName = _FileToExecute;
        p.StartInfo.Arguments = _CommandLine;
        p.StartInfo.RedirectStandardError = true;
        p.StartInfo.RedirectStandardOutput = true;
        p.StartInfo.UseShellExecute = false;
        p.StartInfo.CreateNoWindow = false;
        p.Start();

        _errorMessage = p.StandardError.ReadToEnd();
        p.WaitForExit();

        _outputMessage = p.StandardOutput.ReadToEnd();
        p.WaitForExit();

        if (p.ExitCode != 0)
        {
            return false;
        }

        return true;
    }
    catch (Win32Exception _Win32Exception)
    {
        // Error .
        Console.WriteLine(&quot;Win32 Exception caught in process: {0}&quot;, _Win32Exception.ToString());
        return false;
    }
    catch (Exception _Exception)
    {
       // Error .
        Console.WriteLine(&quot;Exception caught in process: {0}&quot;, _Exception.ToString());
        return false;
    }
    finally
    {
        // close process and do cleanup .
        p.Close();
        p.Dispose();
        p = null;
    }
}

Jacob D Dixon1-Dec-10 5:32

1-Dec-10 5:32

One thing I see wrong is your don't need to p.WaitForExit() twice. The process REGASM.EXE will only exit one time.
Do it more like this:

p= new Process();
p.StartInfo.FileName = _FileToExecute;
p.StartInfo.Arguments = _CommandLine;
p.StartInfo.RedirectStandardError = true;
p.StartInfo.RedirectStandardOutput = true;
p.StartInfo.UseShellExecute = false;
p.StartInfo.CreateNoWindow = false;

p.OutputDataReceived += new DataReceivedEventHandler(p_DataReceived);
p.ErrorDataReceived += new DataReceivedEventHandler(p_DataReceived);

p.Start();

p.BeginErrorReadLine();
p.BeginOutputReadLine();

p.WaitForExit();

Then:

void p_DataReceived(object sender, DataReceivedEventArgs e)
{
      outputmsg += e.Data;
}

Edit:
Of course I just made the error and standard messages go to the same event. You could seperate them if you wanted.

JDM67631-Dec-10 6:10

1-Dec-10 6:10

Wouldn't you need some sort of delegate to do that though? Sorry, I'm still a bit green on C#.

Jacob D Dixon1-Dec-10 7:26

1-Dec-10 7:26

new DataReceivedEventHandler

That is the delegate.

JDM67631-Dec-10 8:09

1-Dec-10 8:09

Thanks. It's still works and it's still doing what it was doing before. Smile | :)

The output passes back a string chopped of path where the tlb file was saved.

ex: "C:\program "

It should bein the following format: C:\Program Files\<app fold="">\<datafolder>\<dllname>

Jacob D Dixon1-Dec-10 8:33

Re: Execute And wait [modified]

1-Dec-10 8:33

Wait a second... Is the REGASM.EXE cutting it off or is it only being cut off when using the Process code?

If you ran the REGASM.EXE from a command prompt does it still cut off the c:\Program?

JDM67631-Dec-10 9:49

1-Dec-10 9:49

Jacob D Dixon wrote:
Wait a second... Is the REGASM.EXE cutting it off or is it only being cut off when using the Process code?

If you ran the REGASM.EXE from a command prompt does it still cut off the c:\Program?

I checked the command line I was passing and I was inadvertently prepending an extra full path of regasm.exe. Not sure why but it was still registering and creating the Tlb. I took off the extra regasm.exe and started getting the following error:

RegAsm : error RA0000 : The name of the type library cannot be specified for assemblies containing embedded type libraries

What I need and haven't figured out how to do is to add double quotes around to full path of the dll and the full path of the destination tlb because one of the folders in the path is "Program Files".

The command line should look something line this:

C:\WINDOWS\Microsoft.NET\Framework\v2.0.50727\RegAsm.exe "C:\path\to\dll\mydll.dll" /tlb: "C:\path\to\dll\mydll.tlb"

Ok.. I fixed it. Smile | :)

This is what I needed.

string szCmdLine = "\"" + szAssemblyPath + "\"" + " /tlb: \"" + szTlbPath + "\"";

modified on Wednesday, December 1, 2010 3:58 PM

Jacob D Dixon1-Dec-10 11:05

1-Dec-10 11:05

Awesome! Glad it is working for you.

Dalek Dave3-Dec-10 5:35

Dalek Dave

3-Dec-10 5:35

Well Spotted.

------------------------------------

I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave

CCC League Table Link
CCC Link[^]

Luc Pattyn1-Dec-10 6:06

Luc Pattyn

1-Dec-10 6:06

JDM6763 wrote:
_errorMessage = p.StandardError.ReadToEnd();
p.WaitForExit();

this sequence doesn't make much sense: ReadToEnd returns what is currently available without waiting, so waiting for the process end after that is meaningless. One way to fix things is first wait for end of process, then ReadToEnd on both streams. Alternatives with events or extra threads would allow to capture (lots of) output while the process is still running.

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use <PRE> tags for code snippets, they preserve indentation, and improve readability.

modified on Wednesday, December 1, 2010 12:34 PM

PIEBALDconsult1-Dec-10 8:08

PIEBALDconsult

1-Dec-10 8:08

Perhaps this[^] might be of some use?

JDM67631-Dec-10 8:14

Windows Tab Control Flickering

1-Dec-10 8:14

Thanks for the suggestion! Smile | :)

What I need to do is fairly simple. I don't think it really needs to be this complex.

registration form

AISIDA AKINWALE MAYOMI1-Dec-10 3:34

AISIDA AKINWALE MAYOMI

1-Dec-10 3:34

am a beginner in C# programing and i would like to write a program that will be able to register student' data by using Microsoft access for the database. thanks

Re: registration form

Pete O'Hanlon1-Dec-10 3:43

Pete O'Hanlon

1-Dec-10 3:43

Are you a beginner programmer, or just a beginner in C#? This is an important distinction.

The step you need to take before you even open up your code editor is to actually define your requirements. Start off by finding out what the user data is that you need to capture. It's pointless us giving you detailed help if you don't know what it is you are trying to capture. Work out what the business process is for capturing the data. For instance, is there an authorisation process? Does data have to be approved by an authorised user before it can be used elsewhere? Does it need any external interfaces?

While you are doing this, if I were you, I'd get a book on C# development and work through it. Pay particular attention to the sections on database programming - this is where you're really going to need to knuckle down and learn the topic. Also, you need to decide whether the data is going to be managed via a desktop application or a web application, and learn the ins and outs of those particular areas.

I have CDO, it's OCD with the letters in the right order; just as they ruddy well should be

Forgive your enemies - it messes with their heads

My blog | My articles | MoXAML PowerToys | Onyx

Re: registration form

Henry Minute1-Dec-10 3:45

Henry Minute

1-Dec-10 3:45

If you use a search engine to search the web for student database, you will find lots of useful information.

FYI: Questions as general as this do not go down well on The Code Project. It makes it look as if you have done no work yourself and you will get lots of messages telling you that no-one is going to do your work for you.

Once you have started coding your application and encounter a problem then by all means ask here. Remember to be specific, paste in error messages in full, if you get any. Include a code snippet for the section of code giving you problems and indicate the problem line(s).

Good luck. Smile | :)

Henry Minute

Do not read medical books! You could die of a misprint. - Mark Twain
Girl: (staring) "Why do you need an icy cucumber?"
“I want to report a fraud. The government is lying to us all.”

Re: registration form

musefan1-Dec-10 3:54

musefan

1-Dec-10 3:54

C#[^]

WinForms[^]

Access database design[^]

ADO.Net[^]

Come back when you get stuck

Life goes very fast. Tomorrow, today is already yesterday.

vishnukamath1-Dec-10 1:37

vishnukamath

1-Dec-10 1:37

hi,

i used tab control in windows form , and i added five tab pages in tab control
,and i added 2 datagridview controls in each and every tab page and binding data for all Datagridviews
in From_Load Event .In Form Displayin with Filckering, so how to stop flickering for form loading.Could
you suggest me better way.

Regards,
Vishnu.

Re: Windows Tab Control Flickering

Abhinav S1-Dec-10 1:42

Abhinav S

1-Dec-10 1:42

This[^] thread may help you.

The funniest thing about this particular signature is that by the time you realise it doesn't say anything it's too late to stop reading it.

My latest tip/trick

Visit the Hindi forum here.

Switching on type

creatiive30-Nov-10 23:44

creatiive

30-Nov-10 23:44

Hi,

This is probably a design issue really. I have an object which is AbsMyObject (abstract). Now, what I am currently doing is testing what type it is, and passing it in to a method like this;

if (AbsMyObject is type1)
myMethod((type1)AbsMyObject);
if (AbsMyObject is type2)
myMethod((type2)AbsMyObject);

Im just overloading 'myMethod' so depending on what type my object actually is, a different method is called. Whilst this implementation works as expected, im ending up with tons of these "if object is" If statements and it just feels like a bad design. It also feels like this might have been encountered before and so there might be a better way of achieving this Smile | :)

Any pointers would be greatly appriciated!
Regards,

phil.o1-Dec-10 0:01

phil.o

1-Dec-10 0:01

Hi,

Why not doing your tests inside your method ? This way, you wouldn't have to care about it when you call your method.

For example :

private void myMethod(AbsMyObject myObject)
{
   if (myObject is type1)
   {
      // Do something
   }
   else if (myObject is type2)
   {
      // Do something else
   }
   else
   {
      // Do something here when myObject is none of expected types
   }
}

Pete O'Hanlon1-Dec-10 0:49

Pete O'Hanlon

1-Dec-10 0:49

You're right - it sounds like an absolute design nightmare. If you just want to do something different inside your concrete implementation classes, why not rework it so that you use the same method for each implementation? That way, you completely remove the need to know about the implementation? By doing this, you will create a much more loosely coupled application.

I have CDO, it's OCD with the letters in the right order; just as they ruddy well should be

Forgive your enemies - it messes with their heads

My blog | My articles | MoXAML PowerToys | Onyx

musefan1-Dec-10 1:01

musefan

1-Dec-10 1:01

What do you mean by 'abstract'? that it is an abstract class? Which I believe cannot be instantiated so is AbsMyObject of a type that inherits from an abstract class?

If this is the case the your absract class can identify a common function such as MyMethod. Then each of your inheriting class types can implement this function in its own way.

Then you can call function like so...

((AbstractClass)AbsMyObject).MyMethod();

...maybe I am misunderstanding what you are trying to do, if this is way off then please further explain what you are trying to achieve and what you are working with (class/type wise)

Life goes very fast. Tomorrow, today is already yesterday.

creatiive1-Dec-10 2:34

creatiive

1-Dec-10 2:34

Thank you all for your replies!

This is what I need in this case;

"If this is the case the your absract class can identify a common function such as MyMethod. Then each of your inheriting class types can implement this function in its own way."

I knew something was wrong with it but didn't quite know how to alter it. This works much better and each object can just have its own implementation of the method. My design skills are a bit poor at the moment Smile | :)

Thanks!