ふらっと C#,C♯,C＃（初心者用） Part149

2020/12/16(水) 11:29:35.35

!extend:checked:vvvvv:1000:512
次スレを立てる時は↑を2行冒頭に書くこと（1行分は消えて表示されない為）

「どんなにくだらないC#プログラミングやVisual C#の使い方に関する質問でも誰かが優しくレスをしてくれるスレッド」です。
他のスレッドでは書き込めないような低レベルな質問、質問者自身なんだか意味がよく分からない質問、
ググろうにもキーワードが分からないなど、勇気をもって書き込んでください。
内容に応じて他スレ・他板へ行くことを勧められることがあります。ご了承下さい。
なお、テンプレが読めない回答者、議論をしたいだけの人は邪魔なので後述のC#相談室に移動して下さい。
C#に関係の無い話題や荒らしの相手や罵倒レスや酔っぱらいレスはやめてください
>>980を踏んだ人は新スレを建てて下さい。>>980が無理な場合、話し合って新スレを建てる人を決めて下さい。

■前スレ
ふらっと C#,C♯,C＃（初心者用） Part148
https://mevius.5ch.net/test/read.cgi/tech/1590578921/
■関連スレ
C#, C♯, C＃相談室 Part94
https://mevius.5ch.net/test/read.cgi/tech/1553075856/
■コードを貼る場合は↓を使いましょう。
http://ideone.com/
https://dotnetfiddle.net/

■情報源
https://docs.microsoft.com/ja-jp/dotnet/standard/class-libraries
https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/
https://docs.microsoft.com/en-us/dotnet/standard/class-libraries
http://referencesource.microsoft.com/
・Insider.NET > .NET TIPS - ＠IT
https://www.atmarkit.co.jp/ait/subtop/features/dotnet/dotnettips_index.html
・DOBON.NET　.NET Tips
https://dobon.net/vb/dotnet/index.html

VIPQ2_EXTDAT: checked:vvvvv:1000:512:: EXT was configured

2021/02/12(金) 03:41:53.86

>>610
煽る意図は全然ないつもりだけど、
そもそも非同期が何を意味してるのか理解しているように聞えないので
まずそれを理解するのが先だと思うよ。

設計とかその後の話でしょ。

2021/02/12(金) 04:13:40.25

>>610
非同期の勉強を始めると一度はぶつかるよね
同期メソッドから非同期メソッドを呼び出せばいいだけでググれはすぐでてくると思う

2021/02/12(金) 04:24:05.42

>>610
補足だけど設計という話になると、非同期メソッドを呼び出しているメソッドが非同期メソッドじゃない様な設計をいいとは思わない

2021/02/12(金) 04:35:12.34

でもこう間違ったのはよく理解できる
結構な数の入門書やブログに、こういった間違った記述があるんだよな

**610** (ﾜｯﾁｮｲ 9354-cU8Z) · 2021/02/12(金) 04:43:36.86

ありがとうございます。ググってみました。

非同期処理の結果を受け取るために、Result ばっかり使用していたのですが、デッドロックするという情報を見て脳死してました。

・戻り値が欲しい場合は呼び出し元の同期メソッドで、GetAwaiter(false).GetResult();
・非同期メソッド内で処理完結させたい場合は、GetAwaiter(false).OnCompleted();
・同期メソッド内で非同期処理を完結させたい場合は、Task.Run();
をすればとりあえず大丈夫そう？という感じみたいなので、これをベースに考えてみます。

今勉強で作ってるのが WinForms アプリですが、多くの処理の起点が戻り値 void のイベントハンドラで、
GetAwaiter(false) にすれば、空いてるスレッド使うことでデッドロックのリスクも下がる(なくなる？)みたいなので、試行錯誤してみます。

2021/02/12(金) 05:48:58.46

>>615
それはResult云々というより変な場所でawaitしてるとじゃないの？
Awaiterは直接扱う物じゃないしConfigureAwaitの間違いだと思うけど
基本的にはUIスレッドで待つ・結果を得るならTaskをawaitするだけ
ライブラリ作るとか、妙な組み方しなければ意識する必要はないはず

元スレッドを解放したい場所（多くの場合UIスレッド）でawaitを使う
非同期処理ならば結果を取り出して返すのではなくTaskを丸ごと返す
勿論それが絶対ではないが、大抵の場合はそれで済むんじゃないかな
あとはProgress<T>とかContinueWithとかTaskExtensions.Unwrapとか

2021/02/12(金) 09:14:15.17

>>600
WinFormsは要素を大量生成するとバカ重くなるという現代GUIには向かない根本的欠陥がある
WinFormsもといWin32はコントロール一つ一つがCreateWindowで生成されるウインドウだからな

その点WPFはWindowクラス以外は基本「ただの絵」であるため大量生成してもそこまで重くならない
が、WPFは抽象化に抽象化を重ねまくったせいで初期の時点でかなり重いという元も子もない産物だが

2021/02/12(金) 09:30:08.77

WPFは無かった扱いにしてパフォーマンスも記述も洗練された奴を作り直してくんねえかな
悪くなかったとしても15年前のアーキテクチャだ
バージョン2が出てもいいだろう

2021/02/12(金) 10:18:15.52

>>618
それが、こんどのMS流 MVU（Model-View-Update）ジャマイカ？
https://devblogs.microsoft.com/dotnet/introducing-net-multi-platform-app-ui/
https://forest.watch.impress.co.jp/docs/news/1254643.html

2021/02/12(金) 13:13:53.93

yield

2021/02/12(金) 19:15:40.43

ｲｴｰｲ

2021/02/12(金) 19:53:50.62

遺詠

2021/02/13(土) 23:26:09.07

WPFはもうサポート終了にして終わりでいいのではｗ
XAMLがクソ過ぎるｗ
もっとマシなものを作ってほしいわ

2021/02/13(土) 23:33:59.55

新しいものが出たってどうせついてこれないんだからそのままForms使ってればいいんでない？

2021/02/13(土) 23:36:28.82

>>623
XAMLじゃなくてMVVMじゃねえの？
WPFはwinFormsの上位互換だが。

2021/02/13(土) 23:36:39.68

あからさまにメリットなかったやん
書籍も全くない有様

2021/02/14(日) 00:04:22.88

メリットがないは言い過ぎで、レイアウトやデザイン性の自由度は明らかにFormより圧倒的に高い。
反面GDI+の機能を完全に包含してない、みたいな問題点もあった。

2021/02/14(日) 00:31:12.63

こんど WPF + VMU で
少し状況かわるんじゃねーーの。

2021/02/14(日) 00:31:45.69

WPF + MVU

2021/02/14(日) 00:44:18.09

>>624
ホントそう思うしMSも下手にWinFormsと同じようにイベントハンドラー書けるようにするもんだから>>625みたいに上位互換とか勘違いする奴が出てくる
別物としてMVVM守んねーと書けないからちゃんと勉強しろ、できねー奴は切り捨てる！
ぐらいすればいいのに

2021/02/14(日) 00:49:41.65

で、MSが開発者から切り捨てられるわけですね
笛吹いても誰も踊らない

2021/02/14(日) 00:50:37.43

じゃあ、それができてるサンプルないの？ってゆーたら1個も出てこなかったじゃない？

2021/02/14(日) 00:51:52.68

基本線で上位互換です。
① WPF + イベントハンドラー
② WPF + MVVM
③ WPF + MVU
メジャーなもんでもこんなに...

2021/02/14(日) 00:52:35.39

この世のどこにもないものを何故か偉そうに語ってたってのが真相

2021/02/14(日) 00:55:34.53

XPSをどうしたいのか？とか
疑問は尽きないよな

2021/02/14(日) 01:01:03.63

WinFormsと同じコントロールを用意しているなら上位互換だけど
WPFはコントロールの数は減っているし、同じ名前ですらないし
ちょっとしたツール作るだけならWPFで作る理由が全然無いな

世に売るようなものでデザインとかスケーリングとか考慮するなら
WPFでもアリだろうけど、今はelectronとかで作ったほうがmacにも対応出来るし
WPFはもう過去のものという感じがするわ

2021/02/14(日) 01:16:47.00

業務アプリだけど、WPF出始めから採用して、全然問題なくみんな移行できたよ。

2021/02/14(日) 01:32:57.20

前も書いたかもしれんけどデータ数が多い折れ線グラフなんか実装しようとすると
行き詰るんだよねWPFは。これでWPFが嫌いになった。

あと、これは勉強不足で勘違いしてるかもしれないけどGDI+みたいに印刷と画面を
透過的に扱えない。

2021/02/14(日) 01:45:32.23

MVVMで使ってるコントロールも
内部実装はMVVMしてませんしねwww

2021/02/14(日) 01:46:58.58

酷い裏切り行為だな

2021/02/14(日) 01:51:03.43

コントロールをMVVMにする意味あるのか

2021/02/14(日) 02:02:16.26

>>641
無いです。
外向きにMVVM対応させて内部実装は普通の実装です。
まあ最短高速な実装です。

昔クラスが１０００以上あるでかいグラフコントロールを改修したときも、
内部はMVVMじゃなく普通の実装でしたし、
それ以外の関連コントール（とある会社のWFPコントールスイート製品）
もそんな感じでした。

2021/02/14(日) 08:37:53.56

chartコントロールが標準で用意されてないのが気に入らん
上位互換ならコントロールもwinform以上用意しとけよ

2021/02/14(日) 09:03:10.41

そもそも上位互換って言ってるのはMSじゃなくて>>625だし

2021/02/14(日) 09:16:57.56

禿道

2021/02/14(日) 09:23:18.05

>>627
そのGDI+ももう明確にレガシーとされたしね。Forms使ってる人もDirectXにしろってことなんだろう。

2021/02/14(日) 09:25:17.46

ザマリンはskia

2021/02/14(日) 09:33:25.22

formと互換性あるとかかなり痛い人だな

2021/02/14(日) 11:00:50.44

>>643
作ろうと思えば作れるだけマシ
WPFはちょっと頑張れば最新ゲームのUIかのようなトリッキーなものも普通に作れるからな
WinFormsで同様の事をしようとするとOnPaintオーバーライドするなりしてGDIやDirect2Dで自力描画しないといけない（描画のみならずマウスの感知動作なども自力計算）
実際俺は自力描画でWinForms版マテリアルデザインとか昔作ってたけど

2021/02/14(日) 11:35:44.81

2021/02/14(日) 13:02:54.63

>>649
俺もWPFでそれなりのことをやってるが、
そもそも業務アプリにはそんな「トリッキー」な機能は要らん
他の人も言ってるが、WinFormsで充分
…という話がまだ理解できないのか？

2021/02/14(日) 13:08:44.18

テンプレに書かれている日本語理解できないのか？

2021/02/14(日) 13:20:53.69

>>652
日本語初心者なんでしょう

2021/02/14(日) 13:22:05.52

作ろうと思えば作れるって話なのに、そんなもの要らん→だからFormsで充分、と
飛躍するところがいかにも頭が悪い。

2021/02/14(日) 13:25:35.27

>>651
別に俺はWinForms要らないとも言ってないしWPFマンセーする気はない
小規模アプリであれば「WinFormsで十分」は同意だけど、中規模以上だと何だかんだでMVVMのが生産性高いしカスタム要素が必要になる率も高いから「WPFで十分」も同様に言えるんだけどな
まぁ俺はそもそもqt派だからもう全然まともにデスクトップ開発でC#使ってないが

2021/02/14(日) 13:35:12.11

トリッキーな機能は要らないからプレゼンテーションはフレームワークにおまかせして
人間はモデルの実装に集中しようぜ
それができるのがMVVM、MVUだよって話だろ
WinFormsだと一番省力したいプレゼンテーションで疲弊するじゃん

2021/02/14(日) 13:39:25.24

ご移動下さい。
WPF(.NET4.x, .NET Core) GUIプログラミング Part25
https://mevius.5ch.net/test/read.cgi/tech/1612522463

2021/02/14(日) 14:04:10.76

MVVMなんかいらないよ
いらないから誰も使ってないんじゃん

2021/02/14(日) 14:14:37.91

MVVMは単なるXamlの付属品じゃなくて、Prism.Unityを中心としたページ遷移型のアプリケーション構築
EventAggregatorによる疎結合のイベント通知、DIコンテナによるモジュールの管理なども有る
勿論web系でも使われている技術だけど、Formsではあまり使わない技術だね
ここらへんを一度使っちまうとFormsには戻りたくなくなる

2021/02/14(日) 14:34:25.50

>>659
嘘じゃん
画面直したら処理も書き換えだよ
そんな嘘ばっかり言ってるから誰も相手にしなくなった

2021/02/14(日) 14:43:14.04

冗談のつもりだったのにホントに日本語理解できてないのか？

ふらっと C#,C♯,C＃（議論用） [無断転載禁止]©2ch.net
https://mevius.5ch.net/test/read.cgi/tech/1469538912/
WPF(.NET4.x, .NET Core) GUIプログラミング Part25
https://mevius.5ch.net/test/read.cgi/tech/1612522463

移動できない奴は、バカなのでぼくの主張は間違ってますって言ってるのと一緒だと気付けよ

2021/02/14(日) 16:08:58.24

全部Microsoftが悪い

2021/02/14(日) 17:40:25.04

何度も言うけどこんな過疎スレでなんで一生懸命「交通整理」してるのか、
その自分の動機の卑しさを自覚できない奴が一番バカっぽいと思うよw

誰かのため？みんなのため？
絶対違うよねそれ。
いやこれマジで大丈夫か。

2021/02/14(日) 19:57:36.92

>>616
615じゃないけど質問です
>基本的にはUIスレッドで待つ・結果を得るならTaskをawaitするだけ
wpfで結果を得てプロパティを変更しイベントが発生する場合
Dispatcher.BeginInvokeは必要ですか ?

2021/02/14(日) 20:12:15.63

>>663
移動しない理由は何？
頭のいい回答を期待してる。

2021/02/14(日) 20:44:22.17

>>663
死ねよ

2021/02/14(日) 20:53:10.91

Ruby on Rails で、控えめなJS フレームワークの、Stimulu を使えば良い

規約で、HTML のdata-controller 属性で、JSファイル名・コントローラーが決まるので、
そこにイベント処理を書くだけ

DOM・コントローラーは、多対多

1つのDOMは、複数のコントローラーで処理できる。
HTML内で、同じコントローラーを複数定義できる

2021/02/14(日) 21:14:32.55

>>664
引用してる文と質問内容が全然別物に見えるので回答できん
とりあえずINotifyPropertyChangedでのBindingならPropertyChangedは別スレッドで発生させても問題ない

2021/02/14(日) 21:22:32.21

>>664
そのレスはTask使ってるのに一々同期的に取り出してそうに見えて
固まらせたくない箇所を非同期にすれば良い的な意味合いで質問と噛み合うかどうか

そのプロパティを変更するのがUIコントロールでスレッドが異なれば必要…なんだけど
TaskをConfigureAwait(false)してなければawaitは待機元スレッドに帰ってくるよ

BeginInvokeはTaskベースで使うかな？別スレッドから実行させるならProgress<T>とか
4.5未満としてもSynchronizationContext.PostやTaskSchedulerを指定してContinueWithかな

2021/02/15(月) 08:14:10.72

surfaceやipadなどで使えるクロスプラットフォームの、ペイントアプリを作ってみたいのですが、
C#が採用されているペイントソフトや3DCGソフトで有名なのってありますか？

2021/02/15(月) 08:52:18.66

>>670
Paint.NET

2021/02/15(月) 08:53:55.45

>>670
Paint.NET

2021/02/15(月) 09:34:24.49

>>670
https://www.pinta-project.com/

なお paint.netのソースはwin32Api 知ってないと
キツいぞ。
projectはformだがwindow周りの実装は
ほぼwin32apiだ

2021/02/15(月) 11:14:53.43

int port;
に対して
if( port < 1 && 65535 < port )
みたいなtrueになりえない条件文ってコンパイル時の警告出せないんでしたっけ？

**664** (ﾜｯﾁｮｲ d602-w1GE) · 2021/02/15(月) 15:06:53.89

>>668,669
質問が下手ですみません。質問を変えます

private ａsync void Button_Click(object sender, RoutedEventArgs e)
{
Button.IsEnabled = false;
await Task.Delay(2000);
Button.IsEnabled = true; // *1
}
タスクが完了したときにUIスレッドが作業中だったとします。
そうすると、UIスレッドは作業を中断して *1を呼びますよね。
この解釈で正しいですか ?

だとすると、Buttonに対して
Button.IsEnabled += Button_IsEnabled がある場合、
UIスレッドが作業中だとUIスレッドは混乱するのではないかと思っています。

混乱を解消するには、awaitの後処理にはUIスレッドが暇になるのを待つ必要があるのではと考え、Dispatchr.BeginInvokeを思いつきました。
ただし、BeginInvokeを追加した例が見当たらないので、もっとうまい手があるのかもしれないと思い、質問しました。

長文失礼

2021/02/15(月) 15:24:35.05

実際の実装は知らんがWindowsのメッセージループを処理するスレッドがUIスレッド（メインスレッド）だろうから、その手のスレッドプールで処理されるものは処理が終わった時点でメインスレッドにメッセージをポストしてメインスレッドがメッセージを受信したときに終了後の処理がされると言う流れかと

2021/02/15(月) 16:12:35.84

>>675
TaskとThreadを混同してるってオチ？
君が懸念してることは別Threadを利用したときに起こる事であって、DispatcherInvoke類も"別Thread"からUI操作を行うときに使う機能

2021/02/15(月) 17:54:07.29

>>675
作業を中断して*1を呼ぶというのは不適切に感じます。中断というならButton_Clickがawaitで後続処理が保留になる所でしょうか
そうなると*1が再開される手順ですが、UI処理はメッセージ単位なので>>676が仰る様なその合間に完了を感知してになると思います
このメッセージキューがパフォーマンスや互換性問題から専用スレッドによる処理が前提なのでBeginInvokeなどが必要になる訳です

翻ってawaitを使うとその辺りをやってくれます。FormsやWPFで異なる手順を抽象化したものがSynchronizationContextやTaskSchedulerですが
それらを推し進めて簡便に扱えるようにしたのがasync/awaitです。※同期的待機内で非同期待機とかしなければそんなに留意する必要はありません

技術的な解説については↓等が参考になると思いますが、直下のコードについてはTaskベース世代のものではないですね
https://docs.microsoft.com/ja-jp/dotnet/desktop/wpf/advanced/threading-model?view=netframeworkdesktop-4.8

**664** (ﾜｯﾁｮｲ d602-w1GE) · 2021/02/15(月) 19:20:24.59

>>677,678
私の例で *1のプロパティ変更と通知ではメッセージループを使っているのかもしれませんが、*1の呼び出しは awaitの後続処理ではないでしょうか

次の例はまずいのではないでしょうか。
private ａsync void Button_Click(object sender, RoutedEventArgs e)
{
await Task.Delay(2000);
MyCLREvent(this, EventArgs.Empty);
}

2021/02/16(火) 00:47:37.27

なにをしてまずいというのかはわからんが
各所でThread.CurrentThread.ManagedThreadId表示させて
どこでどのスレッドになってるか確認してみれば？

2021/02/16(火) 01:07:19.33

>>679
*1はawaitの後続処理だけど、UIスレッドが空くまで実行されない

2021/02/16(火) 09:52:46.68

ボタン2つ用意してこういうので確認してみては
async void btn1_Click(object sender, RoutedEventArgs e) {
await Task.Delay(2000);
Trace.WriteLine("btn1");
}
void btn2_Click(object sender, RoutedEventArgs e) {
for (int i = 0; i < 10; i++) {
Thread.Sleep(300);
Trace.WriteLine("btn2");
}
}

2021/02/16(火) 22:48:01.66

csvで数字を読み込んで、それをグラフ化して表示して、GUIの画面ごとスクリーンショットして保存するWPFプログラムを作ってます。
グラフはwindowsformhostを使ってchartコントロールで表示させています。

VisualTreeHelperでGUIの縦横サイズを取得して、RenderTargetBitmapにDrawingVisualを渡してスクリーンショットしているのですが、chart部分が保存されません
Graphics.CopyFromScreenというのも試してみましたがこちらではchartも表示されるものの、ウィンドウのドロップシャドウみたいなものが一緒に保存されてその分縦横がズレてGUIの右下が見切れます。
解決策を教えてください

2021/02/16(火) 22:58:23.73

allowtransparencyがtrueになってるとか？

2021/02/17(水) 03:55:15.13

>>683
違ったらごめん
これじゃダメなの？
http://mevius.5ch.net/test/read.cgi/tech/1527325729/533-534

2021/02/17(水) 14:41:28.07

ディスプレイ上の座標を指定して、
画面の指定部分をそのままフォームに表示させたいと思っています
まずこれが可能なのかが資料見つかりませんでした
可能なら、毎回スクショ切り取って保存して表示、とかせずに、リアルタイムで反映させたいです

また、そのようにして表示した画面に、拡大や色反転などの加工を行うことは可能でしょうか？

2021/02/17(水) 15:11:52.55

この辺使ってどうにかなりそうかな
試してないけど
https://docs.microsoft.com/ja-jp/dotnet/api/system.windows.forms.screen
https://docs.microsoft.com/ja-jp/dotnet/api/system.drawing.graphics.copyfromscreen

2021/02/17(水) 16:27:46.54

可能かどうかで言えばWindows標準の拡大鏡ツールで実現できてるから可能といえる
C#で実現するのにはまあまず687を試してみればいいんじゃないかな
あと著作権保護されてるコンテンツを表示してる部分は多分無理

2021/02/17(水) 21:03:31.28

PaintWindowで取れないかな

2021/02/17(水) 23:31:19.68

リアルアイムでやるならWindows.Graphics.CaptureやDesktop Duplication APIだな

2021/02/17(水) 23:35:50.45

VSのProjectの構成って何仕様書で定義するものでしょうか？
外部仕様では見たこと無いのですが世間様はどうしてるんだろうと

2021/02/18(木) 00:11:30.22

安心の俺様仕様

2021/02/18(木) 00:19:08.60

一般的には詳細設計で各プロジェクトの構成定義する

2021/02/18(木) 08:41:32.49

世間様じゃ詳細設計で決めてるのですね
参考になります
外仕作成でプロジェクトの構造考えずプロトタイプもりもり作っていたので気になってました

2021/02/18(木) 09:18:08.93

んなもん設計書にはかかないよ

2021/02/18(木) 09:27:17.33

Swiftスレにも似たようなこと書いてる奴がいるから、きっとこのことをまとめてアフィブログにでもするんだろう

2021/02/18(木) 10:41:06.50

Debug.Printの引数はリリース版でも評価されますか?

2021/02/18(木) 12:42:01.95

現実的にプロジェクトの構造なんて誰も気にしない

2021/02/18(木) 18:00:30.64

プロジェクトの構成なのか構造なのか
そもそも構成とか構造って具体的に何？

2021/02/18(木) 22:14:18.02

Ruby on Rails は、設定よりも規約

プロジェクト構成・名前付けルールなど、すべて決まっているから、
誰でも、同じ構成で作る

これが「レールに乗る」と言う事。
Rails のルールから外れて、独自なものを作ってはならない

オレオレ構成みたいなものを認めていない

2021/02/19(金) 00:45:43.92

なんでRoRの話が突然出てくるのか

2021/02/19(金) 01:48:24.06

>>701
他のスレ追い出されたやつだから

2021/02/19(金) 02:16:52.43

定期的に沸くたびに丁寧に相手するから居着いたパターン

2021/02/19(金) 05:03:59.49

こいつわざわざpythonスレにやってきて時代遅れの遺物を押し付けに来る荒らしだろ

2021/02/19(金) 05:37:12.65

ム板全域にいるよねこのガイジ
bashスレにもいた時は頭が痛くなったな
ふれあい作業所とかで遊んでればいいのにね

2021/02/19(金) 08:26:30.93

相手するから喜ぶ
粛々とNGしとけ

2021/02/19(金) 09:49:46.59

ルビキチが珍しく質問に合ったレスを返してるのにお前らヒドイなw

2021/02/19(金) 09:55:27.59

プロジェクトのディレクトリ構成はアーキテクチャ設計を反映したもの

アーキテクチャ設計が詳細設計に含まれてるならで詳細設計段階で決める

Railsのようにアプリの機能構成に関係なく一律同じディレクトリ構成にするのもそれなりのメリットがある

2021/02/19(金) 15:40:30.15

>>707
日頃の行いが相応に報われてるだけ
フェミフェミ噛みついて本気でフェミに迷惑掛けてるのがフェミ

2021/02/19(金) 18:51:07.37

初心者です。画像の一部をクリックすると画面遷移を行うようなプログラムを
作りたいと考え、自分なりに作ってみたのですが
using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.SceneManagement;

public class NG2to3 : MonoBehaviour
{

Vector2 Pos;
void Start()
{}
void Update()
{
if (Input.GetMouseButtonDown(0))
{
Pos = Input.mousePosition;
if (-15 <= this.Pos.x && this.Pos.x <= 15 &&
　 -35 <= this.Pos.y && this.Pos.y <= -15 )

{
SceneManager.LoadScene("scene3");
}
}

}
}

このプログラムで指定したPosの範囲の場合画面遷移を行いたいのですが、
画面の至るところをタッチすれど遷移してくれません。
どこが間違っているのか教えていただきたいです。