|
Try posting your question in the forum at the end of the article, so the author can see and respond to it.
Just say 'NO' to evaluated arguments for diadic functions! Ash
|
|
|
|
|
you should Move there[^] to ask question.
modified on Wednesday, December 22, 2010 11:51 PM
|
|
|
|
|
I've emailed her and sent her a link, hopefully she'll reply ... but I was also hoping someone else had encountered this and had some input.
|
|
|
|
|
I've tried a ton of tutorials but none work. Or rather, sooner or later ALL digress and leave the crucial part of transfering a file that isn't pure text.
I'm remodeling an app I've previously used to transfer text and project specific objects over a LAN. I'm just having trouble to receive the file and save it again after the transfer (I assume the sending of the file is successful, but of course have no way to actually verify this).
Here is the code for my client (sending) app:
private void Send(string arg)
{
TcpClient client = new TcpClient(TB_Host.Text, (int)UD_Port.Value);
FileStream fstream = File.Open(OFD_File.FileName, FileMode.Open);
NetworkStream nstream = client.GetStream();
int data = 0;
while (data > 0)
{
data = fstream.ReadByte();
nstream.WriteByte((byte)data);
}
fstream.Close();
nstream.Close();
client.Close();
}
And this is the code for the server (receiving) app:
public void Listen()
{
int port = 21112;
byte[] result = new byte[1024];
IPAddress address = IPAddress.Parse("127.0.0.1");
TcpListener listener = new TcpListener(address, port);
ipEnd = new IPEndPoint(address, port);
listener.Start();
socket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);
int f = 0;
object state = new object();
socket.Connect(ipEnd);
while (true)
{
TcpClient client = listener.AcceptTcpClient();
NetworkStream ns = client.GetStream();
f = socket.Receive(result);
Invoke(new UpdateDisplayDelegate(UpdateDisplay), result);
}
} The current problem is that the code never reaches the invoke call (the Listen method is on a 2nd thread), which confuses me as there never had been any problem when I transferred text or objects.
I tried a SoapFormatter , but while it works great for text and objects, I can't get it to work for files.
I also tried using no sockets but a NetworkStream to write into the buffer:
result = ns.Write(result, 0, int.MaxValue); but I always get an ArgumentOutOfRange exception for the size (I used int.MaxValue) parameter.
Can you tell me how to get it to work? Or is this approach wrong in the first place?
Thanks!
|
|
|
|
|
Hi,
that does not look good.
1.
Megidolaon wrote: while (data > 0)
the test fails right away, so nothing gets sent.
2.
Megidolaon wrote: data = fstream.ReadByte();
Now why would you handle one byte at a time? it will take forever on medium-sized files, and even longer on large ones.
You are trying to receive 1024 bytes, why not do something similar on the sender side?
Use a byte array, Stream.Read(), and make sure to watch the actual number of bytes transferred.
3.
Megidolaon wrote: the code never reaches the invoke call
see (1)
4.
Megidolaon wrote: I tried a SoapFormatter
I have no idea why you would use a formatter. Sockets don't care about the data they carry.
5.
Megidolaon wrote: I used int.MaxValue
If the method needs to allocate a buffer, it will fail to get int.MaxValue bytes of memory, no matter how much RAM your system has.
I suggest you try something like this as a sender (not tested!):
int DIM=1024;
byte[] bytes=new byte[DIM];
for(;;) {
int count=fstream.Read(bytes, 0, DIM);
if (count==0) break;
nstream.Write(bytes, 0, count);
}
|
|
|
|
|
...what?
All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
|
|
|
|
|
... What?
All he did was provide a good critique that you should learn from and some code that has more than a snowball's chance of working.
|
|
|
|
|
Critique? I did not ask for critique, I asked for help. Which he pretended he gave, but actually did not give at all.
|
|
|
|
|
Look, if you aren't up to working out what he said, that's your problem. The answer he have was helpful:
Your code couldn't possibly run, was inefficient and poorly designed. He covered each of those points. He even gave you a sample of code that fixes those problems.
I suggest you read the other comments and figure out why you are wrong, rather than attempting flip answers.
|
|
|
|
|
Megidolaon wrote: All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
Except that yours does nothing, that you can verify anyway. You were given a reason for this, have you investigated it?
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.”
|
|
|
|
|
Megidolaon wrote: int data = 0;
while (data > 0) ...
and how many iterations will this loop execute in your opinion?
|
|
|
|
|
I'll give him a clue: it's the same value as data . Lets see if he cracks the code
|
|
|
|
|
My vote of 1... Try doing a bit of the thinking yourself. He provided you exactly what you need to get it going. All you have to do is follow instructions.
I wasn't, now I am, then I won't be anymore.
|
|
|
|
|
All you did was provide code that couldn't possibly work. Luc provided reasons why, and a practical suggestion on how to take your code forwards. Luc's comments were valid, pertinent and usable - if you think his code does the same as yours, I suggest that you need to step through the code line by line. BTW - had you actually bothered to debug your application, you'd have spotted that you could never step into the loop.
|
|
|
|
|
Megidolaon wrote: and post code that does exactly the same thing as mine.
With one major difference, your code will never work, Luc's always will.
Just say 'NO' to evaluated arguments for diadic functions! Ash
|
|
|
|
|
Signature material. Hijacked.
|
|
|
|
|
Fame at last!
Just say 'NO' to evaluated arguments for diadic functions! Ash
|
|
|
|
|
I have a question here: Do you want somebody to do your job or do you want to learn?
If you want somebody to do your job, then look for a professional, ask him to do the job and pay him for it.
If you want to learn then read again Luc's answer and try to apply his advices. Additionally, you might say "Thank you for your time" or nothing at all, but this kind of reply is a great invitation to blacklist you and do never answer any other of your questions.
|
|
|
|
|
Megidolaon wrote: Or is this approach wrong in the first place?
As you said it is a local network so why not just use the OS to do the transfer?
And if there is some security reason for that then just set up a ftp server and client.
|
|
|
|
|
How we can write a program with c# for Approximation Taylor and Furie series with neuro network .
|
|
|
|
|
Hi,
You have two choices :
- use a basic text editor
- use an IDE
More seriously, as you're new to these forums, there are several rules to follow when you post some questions :
- don't post your question at several places ; rather find the right forum and post it in that place
- post precise questions, show what you have searched/done so far, with pieces of code and error messages relevant to your problem. No one is gonna make google searches for you.
I suggest you first read the forum guidelines before posting.
Regards.
|
|
|
|
|
I find that breaking the problem down into ever smaller problems, and solving those problems all the way down the chain helps. I tend not to start with the code; I'd rather thrash out the requirements first and then break those requirements down into smaller and smaller requirements until I have a good grasp what problem I'm solving. That's how I'd write the program, but then again I'm just a picky professional developer. How would you go about writing it?
|
|
|
|
|
Just slam the code out and stick it up for Beta. (The Microsoft Technique)
Real men don't use instructions. They are only the manufacturers opinion on how to put the thing together.
|
|
|
|
|
Leave it beta for ever. (The Google Technique).
|
|
|
|
|
Ooooo, that's below the belt
return 5;
|
|
|
|