質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

TCP

TCP(Transmission Control Protocol)とは、トランスポート層のプロトコルで、コネクション型のデータサービスです。

Kotlin

Kotlinは、ジェットブレインズ社のアンドリー・ブレスラフ、ドミトリー・ジェメロフが開発した、 静的型付けのオブジェクト指向プログラミング言語です。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

Q&A

解決済

1回答

3247閲覧

WPF → AndroidのTCP通信で巨大なファイルを送るとデータが壊れる

chika3742

総合スコア113

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Android

Androidは、Google社が開発したスマートフォンやタブレットなど携帯端末向けのプラットフォームです。 カーネル・ミドルウェア・ユーザーインターフェイス・ウェブブラウザ・電話帳などのアプリケーションやソフトウェアをひとつにまとめて構成。 カーネル・ライブラリ・ランタイムはほとんどがC言語/C++、アプリケーションなどはJavaSEのサブセットとAndroid環境で書かれています。

TCP

TCP(Transmission Control Protocol)とは、トランスポート層のプロトコルで、コネクション型のデータサービスです。

Kotlin

Kotlinは、ジェットブレインズ社のアンドリー・ブレスラフ、ドミトリー・ジェメロフが開発した、 静的型付けのオブジェクト指向プログラミング言語です。

WPF

Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

0グッド

0クリップ

投稿2020/03/30 13:29

編集2020/04/01 07:37

発生している問題

WPFアプリ(Windows)からAndroidアプリに向けてTCP通信を実装しています。
具体的には、
0. テキストかファイルか
0. ファイル名
0. 0xFF
0. ファイルサイズ
0. 0xFF
0. ファイルの中身

という構造で転送をしています。その逆も実装していて、AndroidアプリからWPFアプリへの送信はうまくいくのですが、WPFからAndroidへの送信がうまくいきません。小さいファイルなら正常に転送できますが、大きなファイル(大体5MB以上)になると壊れてしまいます。

試したこと

受信側で読み取るたびにスレッドを1ミリ秒くらい止めると少し大きめのファイルでも転送できるようになります。しかし、それだと転送速度が遅くなるので、他の方法を探しています。
他に、バッファーサイズを変えたりしましたが、だめでした。

ソースコード

送信側(WPF)

C#

1 var client = new TcpClient(ipAddress, 60001); 2 client.SendTimeout = 10000; 3 4 var ous = client.GetStream(); 5 var ins = new FileStream(FilePaths[0], FileMode.Open); 6 7 var bytes = new byte[8192]; 8 var nameBytes = Encoding.UTF8.GetBytes(System.IO.Path.GetFileName(FilePaths[0])); 9 var sizeBytes = BitConverter.GetBytes(ins.Length); 10 var firstBytes = new byte[bytes.Length - 1 - nameBytes.Length - 1 - sizeBytes.Length - 1]; 11 12 ins.Read(firstBytes, 0, firstBytes.Length); 13 14 bytes[0] = SendWindow.TYPE_FILE; 15 Array.Copy(nameBytes, 0, bytes, 1, nameBytes.Length); 16 bytes[nameBytes.Length + 1] = 255; 17 Array.Copy(sizeBytes, 0, bytes, nameBytes.Length + 2, sizeBytes.Length); 18 bytes[nameBytes.Length + 1 + sizeBytes.Length + 1] = 255; 19 Array.Copy(firstBytes, 0, bytes, nameBytes.Length + 1 + sizeBytes.Length + 2, firstBytes.Length); 20 21 ous.Write(bytes, 0, bytes.Length); 22 23 var buffer = new byte[8192]; 24 do 25 { 26 var len = ins.Read(buffer, 0, buffer.Length); 27 if (len == 0 || IsSendingFile == false) break; 28 ous.Write(buffer, 0, buffer.Length); 29 } while (true); 30 31 ous.Close(); 32 ins.Close(); 33 client.Close();

受信側(Android)

Kotlin

1 val socket = svSocket.accept() 2 3 val ins = socket.getInputStream() 4 5 val isFile = ins.read().toByte() == DataType.FILE 6 7 if (isFile) { 8 val nameBytes = mutableListOf<Byte>() 9 do { 10 val byte = ins.read() 11 if (byte == 255) break 12 nameBytes.add(byte.toByte()) 13 } while (true) 14 val name = String(nameBytes.toByteArray()) 15 16 val progressN = NotificationCompat.Builder(context, CHANNEL_PROGRESS) 17 .setContentTitle("${name}を受信中") 18 .setColor(Color.parseColor("#87cefa")) 19 .setSmallIcon(R.drawable.tasktrayicon) 20 .setOnlyAlertOnce(true) 21 .setOngoing(true) 22 23 context.notificationManager.notify(PROGRESS_NOTIFICATION_ID, progressN.build()) 24 25 val sizeBytes = mutableListOf<Byte>() 26 do { 27 val byte = ins.read() 28 if (byte == 255) break 29 sizeBytes.add(byte.toByte()) 30 } while (true) 31 sizeBytes.reverse() 32 val size = ByteBuffer.wrap(sizeBytes.toByteArray()).long 33 34 val bufferSize = DEFAULT_BUFFER_SIZE 35 36 val dir = File(context.getExternalFilesDir(null), "Download") 37 dir.mkdirs() 38 val file = File(dir, name) 39 val ous = BufferedOutputStream(FileOutputStream(file)) 40 41 val packetCount = size.toDouble() / bufferSize 42 43 val buffer = ByteArray(bufferSize) 44 var packet = 0 45 do { 46 val len = ins.read(buffer) 47 if (len == -1) break 48 ous.write(buffer, 0, buffer.size) 49 Thread.sleep(1) 50 packet++ 51 if (packet % 1000 == 0 && packetCount != 0.toDouble()) { 52 val progress = packet / packetCount * 100 53 progressN.setProgress(100, progress.toInt(), false) 54 .setContentText("${String.format("%.2f", packet / packetCount * 100)} %") 55 context.notificationManager.notify(PROGRESS_NOTIFICATION_ID, progressN.build()) 56 } 57 } while (true) 58 59 ous.close() 60 ins.close()

環境

Visual Studio Community 2019 v16.5.1
Android Studio 3.6.1
Android 10(実機)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

kotlin

1 val len = ins.read(buffer) 2 if (len == -1) break 3 ous.write(buffer, 0, buffer.size)

read は buffer に取り込んだバイト数を返します.
なぜなら, 必ずしも buffer.size 分を読み込めるとは限らないからです.
ですので, write も len バイト数分しか書き込まないようにするべきです.

(そしてこれは恐らく送信側も同じです.)

スレッドを1ミリ秒くらい止めると少し大きめのファイルでも転送できる

のは, その間に buffer.size 分のデータが貯まるからでしょう.

投稿2020/03/30 15:07

編集2020/03/30 15:15
jimbe

総合スコア12545

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

chika3742

2020/03/30 23:43

そういうことでしたか、完全に見落としていました。。。 ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問