C# Discussion Boards - CodeProject

sammy1502412-Jul-09 3:58

12-Jul-09 3:58

Hi Henry,

Thanks for your proposal. But I dont think it would be easier to understand for other people if I shorten it out. But I have another suggestion. If you can compile the code and then I can give you the example of security.txt and position.txt and then you may see the result and the problem itself.

Any thoughts?

OriginalGriff12-Jul-09 4:03

OriginalGriff

12-Jul-09 4:03

I can't speak for Henry, but I would not compile and run source code from someone I didn't know well and hadn't examined thoughly - any more than I would run an EXE file I "just found" on a website.

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones

sammy1502412-Jul-09 4:22

12-Jul-09 4:22

But if you just go through a bit you would never find that it is anything harmful. Actually I am new in this language and facing a tough problem wanted to share with others so that I can get the help. But I think my thinking or point of view was wrong.

Regards,
Sam

OriginalGriff12-Jul-09 4:33

OriginalGriff

12-Jul-09 4:33

sammy15024 wrote:
But if you just go through a bit you would never find that it is anything harmful

I think that is Henry's point - we don't want to go though all that stuff. You have to remember that we don't get paid for this - so we aren't going to put ourselves out too much unless we are really interested. Large amounts of unformatted, dull looking code with a "it doesn't allways work properly" comment are unlikely to be looked at.

A short, snappy formatted code fragment with "why does it ignore all lines with 'Banana' in them?" is a lot more likely to get a good response.

No trees were harmed in the sending of this message; however, a significant number of electrons were slightly inconvenienced.

This message is made of fully recyclable Zeros and Ones

Henry Minute12-Jul-09 4:09

Henry Minute

12-Jul-09 4:09

You carry on being right and doing it your way.

I'll carry on not helping you, as, I suspect, will most others.

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.”

sammy1502412-Jul-09 4:19

12-Jul-09 4:19

It's ok Henry.

Thanks for your help and suggestion.

With regards,
Sam

Christian Graus12-Jul-09 4:32

12-Jul-09 4:32

sammy15024 wrote:
. But I dont think it would be easier to understand for other people if I shorten it out.

Why do you think that ? you're dead wrong. If you actually try to debug it yourself, you'll either find the answer, or come up with a code snippet that you can intelligently explain what you expect, and what it is doing. What you're saying right now is, I wrote all this code, and I'm too dumb to understand it, so you do the work for me.

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

sammy1502412-Jul-09 4:38

12-Jul-09 4:38

Hi Christian,
Thanks for your reply. I tried with all of my little knowledge. But was undone fully. Somewhere I am doing something silly so my exe is not working exactly. Since my code is dependant with other two files, I am not able to understand where there something is misiing in source files or in source code. That was the reason I posted all the source code.

Regards,
Sam

Christian Graus12-Jul-09 4:44

12-Jul-09 4:44

So, you stepped through the debugger, and all you can tell us is that something doesn't work in the source code as a whole ?

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

sammy1502412-Jul-09 4:51

12-Jul-09 4:51

[Message Deleted]

Christian Graus12-Jul-09 5:06

12-Jul-09 5:06

I'm sorry, but this is a disaster. This code is close to unreadable. It is full of idioms that make no sense. You don't define 'not working' and it's clear you've not bothered to use the debugger to work out why it's doing what it is. No-one can fix this code without running it, with the files you are using, and no-one is going to do that for you. We'd be happy to explain to you how to use the debugger, although you really should have a book to tell you that. This code is way too bad, and your skill level way too low, for you to be doing this for a paid job, so I assume you're doing this for a class. Your best bet, is to talk to your professor, so he/she knows how lost you are and is able to better help you going forward.

Of course, the industry in the state it's in, I am aware that someone may well be paying for this mess. If that's true, it's an absolute disgrace. Of course, if it's not, then everyone has to learn, the issue here is that instead of learning, you're asking us to fix it for you. The debugger is there for a reason, you will never cut it as a programer unless you take the time to learn how to use it.

But, if you're willing to at least try to diagnose your own problem, and if you have a specific question when you know exactly what is going wrong, I'm sure we'd be happy to help you. I would recommend reading the guidelines on how to ask questions before posting another one, you broke a fair number of them in your post this time.

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

sammy1502412-Jul-09 5:12

Re: Can anybody please help me [modified]

12-Jul-09 5:12

[Message Deleted]

Christian Graus12-Jul-09 5:18

12-Jul-09 5:18

sammy15024 wrote:
But I guess your earlier post is too offensive.

I'm sorry if the truth offends you. The truth is, no-one can help you just by reading the code. They would have to run it. And, no-one is going to do that. So, if you want to write code, then learn to debug it. It's REALLY easy. Press F9 to set a breakpoint and your code will pause where you set it, the debug menu will let you step through and see what is happening.

What is obviously happening is that your file format is not exactly what you think it is, and so the code fails for some lines. Again, the debugger is how you work out what is going on.

sammy15024 wrote:
If you can't assist other you dont have any right to insult he/she.

I have given you more help than you deserve. Just because I didn't magically pull a solution out of thin air ( when finding one based on this is impossible ), doesn't mean I didn't try to help.

If you're not able to debug this code, then it's probably true that a wealth management company shouldn't be managing people's wealth based on code that you don't really understand. Why not pay someone to write something that works, if you're not willing to take the time to learn to do it yourself ?

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

modified on Sunday, July 12, 2009 11:31 AM

JimmyRopes12-Jul-09 5:39

12-Jul-09 5:39

Christian Graus wrote:
If you're not able to debug this code, then it's probably true that a wealth management company shouldn't be managing people's wealth based on code that you don't really understand

Sammy is from India. Would you send your wealth to India to be managed? Dead | X|

If the “wealth management” companies in South Asia are anything like the “wealth management “ companies in Southeast Asia (and why should one thing they are different) the only wealth management is to transfer money to a bank in Hong Kong (or some other such place) and never see it, or the proceeds of the management efforts, again. Roll eyes | :rolleyes:

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

sammy1502412-Jul-09 6:04

12-Jul-09 6:04

[Message Deleted]

JimmyRopes12-Jul-09 6:06

12-Jul-09 6:06

sammy15024 wrote:
We do manage BILLIONS OF USA WEALTH.

No doubt you do, but do the investors get anything in return? Unsure | :~

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

sammy1502412-Jul-09 6:10

12-Jul-09 6:10

[Message Deleted]

JimmyRopes12-Jul-09 6:13

12-Jul-09 6:13

sammy15024 wrote:
We think they get a healthy and wealthy returns. Thats why we are doing it for few many years.

No doubt you do, but do your investors believe they are getting healthy and wealthy returns? Unsure | :~

If you manage billions in US wealth you are surely a publically owned company. What is you stock symbol and what exchange do you trade on?

I may be interested in buying some stock in your company. Roll eyes | :rolleyes:

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

JimmyRopes12-Jul-09 6:23

12-Jul-09 6:23

sammy15024 wrote:
I am not authorised to do that Sir, But our return percentage is too high and you cant be the part of it. You will stuck to eligibility criteria

Sounds like a scam to me. Unsure | :~

If you are not authorized to disclose the company you work for or the stock exchange your company stock is traded on than you are a "boiler room" in India stealing money from fools who believe the fantastic stories of wealth and fame they will receive if they invest in you. Roll eyes | :rolleyes:

I have millions to invest. Where do I send my money? Confused | :confused:

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

sammy1502412-Jul-09 6:26

12-Jul-09 6:26

[Message Deleted]

JimmyRopes12-Jul-09 6:30

12-Jul-09 6:30

sammy15024 wrote:
We do mmanage for company not for an individual.. Are you a company or individual. If you have millions why are you wasting your valuable time here gor at WSJ/ Bloomberg.

I am a company with millions to invest. Roll eyes | :rolleyes:

sammy15024 wrote:
Dont do non productive time waste thanks.

Why do you not want to take my money? Confused | :confused:

Let me speak with your MD! Big Grin | :-D

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

Christian Graus12-Jul-09 7:22

12-Jul-09 7:22

ROTFL !!! I am enjoying this. Poor guy thought he'd get something written for free, and instead he gets exposed.

Christian Graus

Driven to the arms of OSX by Vista.

Read my blog to find out how I've worked around bugs in Microsoft tools and frameworks.

JimmyRopes12-Jul-09 7:25

12-Jul-09 7:25

Christian Graus wrote:
Poor guy thought he'd get something written for free, and instead he gets exposed.

I just call them as I see them.

Simply Elegant Designs JimmyRopes Designs
Think inside the box! ProActive Secure Systems

I'm on-line therefore I am.
JimmyRopes

dan!sh 12-Jul-09 3:38

dan!sh

12-Jul-09 3:38

Went through your code a bit. Looks like you are dealing with some kind of delimited files. You can create a datatable out of it using OleDB classes instead of all those loops. Also, at one place you are replacing space by space. Why is that?

Here:

str = str.Replace(" ", " ");

This is not going to solve your problem but will reduce the amount of code. Can you explain what exactly is the problem since your post isn't clear enough.