How do I transfer files over a local network?
-
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
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.”
-
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
Megidolaon wrote:
int data = 0; while (data > 0) ...
and how many iterations will this loop execute in your opinion? :doh:
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum
Please use <PRE> tags for code snippets, they preserve indentation, improve readability, and make me actually look at the code.
-
Megidolaon wrote:
int data = 0; while (data > 0) ...
and how many iterations will this loop execute in your opinion? :doh:
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum
Please use <PRE> tags for code snippets, they preserve indentation, improve readability, and make me actually look at the code.
I'll give him a clue: it's the same value as
data
. Lets see if he cracks the code :-)Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^] -
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
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.
-
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
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.
I'm not a stalker, I just know things. Oh by the way, you're out of milk.
Forgive your enemies - it messes with their heads
-
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
-
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. :)
"Your code will never work, Luc's always will.", Richard MacCutchan[^]
-
...what? All you did is provide some vague comments that are unusable and post code that does exactly the same thing as mine.
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.
-
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 aSoapFormatter
, 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 aNetworkStream
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 -
... 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.
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^]Critique? I did not ask for critique, I asked for help. Which he pretended he gave, but actually did not give at all.
-
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.
Sort of a cross between Lawrence of Arabia and Dilbert.[^]
-Or-
A Dead ringer for Kate Winslett[^]