Java入門・初心者質問スレ Part.5©2ch.net

**デフォルトの名無しさん転載ダメ©2ch.net** · 2017/09/29(金) 17:53:25.24

Java初心者のためのスレッドです。

※前スレ
Java入門・初心者質問スレ Part.4
http://mevius.2ch.net/test/read.cgi/tech/1498828456/

**デフォルトの名無しさん** · 2017/10/19(木) 21:44:03.04

BufferedReader br = Files.newBufferedReader(path,charset)

**デフォルトの名無しさん** · 2017/10/20(金) 09:35:26.11

>>180
バッファがあると一回の読み書きに時間が掛かる機器の場合に効率が良くなって速度が上がる可能性がある。

例えばHDD。一回の読み書きが遅いのでまとめて読んだり書いたりした方が速度は上がる。
ということでファイル入出力にはバッファがあった方が良い。

**デフォルトの名無しさん** · 2017/10/20(金) 09:42:05.09

どのくらい早くなりますか？

**デフォルトの名無しさん** · 2017/10/20(金) 10:04:52.52

その手の質問には「1万回繰り返すと2倍くらい違う」と答えることにしてる

**デフォルトの名無しさん** · 2017/10/20(金) 10:20:18.18

100回繰り返した場合はどうですか？

**デフォルトの名無しさん** · 2017/10/20(金) 10:30:19.12

割合算でいいんじゃないの
つまり速い方に書き換えると「その部分だけが」2％くらい速い
そこの処理が0.03秒だった場合は処理速度が0.0294秒になって0.0006秒の改善になる

どっちかってと速度なんかじゃなくHDDとメモリ酷使するのを良しとするかどうかで動作決めていいよバッファ系は

**デフォルトの名無しさん** · 2017/10/20(金) 10:40:23.48

OSはバッファしないのですか？

**デフォルトの名無しさん** · 2017/10/20(金) 12:01:07.16

ググレカス

**デフォルトの名無しさん** · 2017/10/20(金) 12:44:07.68

>>188
するよ
そこまで行ったならオブジェクト生成コストやGC負荷とアルゴリズム、加えて「コンパイラが完全最適なオブジェクト構成に置き換えてくれる」まで考えて、
結局は実際に計測して遅くなきゃいいや、人力脳内最適化は無意味でクソである、といういま一番トレンドな結論に至る
おめでとう

**デフォルトの名無しさん** · 2017/10/20(金) 12:44:37.57

ヨハン・セバスチャン

**デフォルトの名無しさん** · 2017/10/20(金) 13:02:09.19

>>184
接続されてるHDDやプログラムがどのぐらい細切れにデータを読み書きするかで変わる。
ま、なんだったら自分でバッファありとなし作って試してみな。だいたいはバッファありの方が速くなる筈。

速度差が出ないとか、むしろ遅くなる場合はバッファの大きさがそのプログラムでの一度の読み書き量に対して適切ではないとか接続されているHDDに対して適切ではないのかも知れない。
あるいは最初から適切だったためにバッファありにしても殆ど変わらないかかな。固定長の大きな電文を自分で作ってその単位で読み書きする場合はその電文の塊がバッファみたいなものなので変化がないかも知れない。

**デフォルトの名無しさん** · 2017/10/20(金) 14:59:15.51

特定環境だけ意識するのか

**デフォルトの名無しさん** · 2017/10/20(金) 16:29:37.22

HDDはあまり関係ないような気がする。

**デフォルトの名無しさん** · 2017/10/20(金) 16:37:33.80

OSはほぼ確実にバファリンしてるのでHDDの読み書きではなく、システムを呼び出すオーバーヘッドで
変わるだけじゃないだろか。

**デフォルトの名無しさん** · 2017/10/20(金) 16:42:56.58

BufferedReaderを使うとシステム呼び出し回数が減らせるのでスレッドの切り替え確率が
減るという説明のほうが納得がいくような気がスルスル。

**デフォルトの名無しさん** · 2017/10/20(金) 22:57:05.24

OS・HDD は、バッファリングしてる

だから突然の電源断により、書き込まれない事があるため、
必ず正常にシステムを終了させること

普通OSは、5秒ごとに、HDDに書き込む

フラッシュメモリーには、書き込み回数の上限があるため、
SSD の書き込み回数が気になるなら、15秒ごとに書き込むように設定できる

**デフォルトの名無しさん** · 2017/10/21(土) 02:10:59.27

Javaの最大の欠点は100kbのデータを扱うプログラムにGBクラスのメモリーが必要になるような
効率の悪さ。

**デフォルトの名無しさん** · 2017/10/21(土) 02:34:16.89

**デフォルトの名無しさん** · 2017/10/21(土) 02:58:50.47

https://i.imgur.com/ex4qwLj.jpg
大学の課題なんだが、for文のi<5のあとにlengthをつけなくてもこの課題は解決できるのかを教えてくれ。

**デフォルトの名無しさん** · 2017/10/21(土) 03:49:35.47

>>198
どういう場合にそうなんの？
普通なら大丈夫なんじゃね

**デフォルトの名無しさん** · 2017/10/21(土) 03:59:19.02

>>200
この問題
「am[0]」の使い方に違和感があるな

AMCounterクラスとか作って
カウントの責務を分離したい感じがする

**デフォルトの名無しさん** · 2017/10/21(土) 04:08:37.48

>>202
3つ目のクラスを作るってことか？それはまだ習ってないからそうしないのかも。

**デフォルトの名無しさん** · 2017/10/21(土) 04:20:38.55

>>202
ああ。違和感あるな。これダメな設計だよなあ?

am[0].countAutomobile() や am[0].countHeavy() でカウントした結果が出るということは
new Automobile() した時にコンストラクタで Automobile クラス内にある static のクラス変数の
カウントをしていて countAutomobile() や countHeavy() メソッドでそれを読み出すという
ことになるが、そんなクラスは一つのVMで同時に一つしか動かせない。複数のスレッドで
同時に使おうとすると意図した通りに動かない。Java のクラスとしてはなんだかとても
嫌なクラスだ。

**デフォルトの名無しさん** · 2017/10/21(土) 10:41:58.06

ハイエークラスに集計メソッドを持たせてはどうか。

**デフォルトの名無しさん** · 2017/10/21(土) 10:42:56.10

せっかくだからシークエルを渡せるようにしてはどうか。

**デフォルトの名無しさん** · 2017/10/21(土) 14:19:09.90

>>200
可能

5台と固定だから付ける意味ないんじゃないかな
クラス変数の挙動を知る課題かな
その使い方や設計としては悪いが

**デフォルトの名無しさん** · 2017/10/21(土) 15:37:16.16

>>200
amもweightも同じ要素数だから5回繰り返せばぴったりじゃん

**デフォルトの名無しさん** · 2017/10/21(土) 15:47:20.68

ぴったりじゃん？

**デフォルトの名無しさん** · 2017/10/21(土) 16:49:24.05

何この糞問題ｗｗｗひっでぇな・・マジで
これ作ったの教員？生徒？
なんで配列の一要素でしかないクラスが配列全体の情報を持ってこれるんだよｗ
こんなレベルだと生徒もダメになるゴミ問題だな

**デフォルトの名無しさん** · 2017/10/21(土) 16:53:51.90

しかもこれで大学だからなｗｗｗ
>200も相当な馬鹿っぽいし１００％Fランだなｗ

**デフォルトの名無しさん** · 2017/10/21(土) 16:58:21.30

あーこれあれかｗ
最初から車の総数と大型車の台数わかってるから
countAutomobileとcountHeavyは５台と２台を直打ちで返すようにしてるだけなのかｗｗｗ
中学校のマイコン倶楽部の問題かよｗｗｗｗｗｗｗ

**デフォルトの名無しさん** · 2017/10/21(土) 17:35:53.13

>200です。みんな色々とありがとう。まあFランなのは否定しないですw lengthなくてもいけるのはわかったのでもうちょい頑張ってみます

**デフォルトの名無しさん** · 2017/10/21(土) 17:58:44.98

countAutomobile()やcountHeavy()は
コレクションクラスに持たせるのが普通でしょ
AMCounterクラス作って切り出すほどでもない
この規模ならHighwayクラスのメソッドでも十分

for文の中括弧省略はアンチパターン
あと大型車を数えるのにcountHeavy()という名前は不適切

**デフォルトの名無しさん** · 2017/10/21(土) 18:16:41.12

>>214
まあ一応課題に沿って作成しないといけないんで与えられたクラス内でやるしかないみたいです。ご指摘ありがとうございます。

**デフォルトの名無しさん** · 2017/10/21(土) 18:22:53.02

>>204
その通りだね
課題の範囲だとstatic使うくらいしかないよな

>>205
それもアリだと思う
countメソッドはAutomobileよりかは
Highwayが持つ方が自然

ただ何を作るかにもよるけど
Highwayが将来的に肥大化しそうだから
Counterクラスに切り出す場合もありそう

**デフォルトの名無しさん** · 2017/10/21(土) 18:27:25.26

>>214
コレクションクラスでもいいというか
課題だとそっちの方がいいね

ただ課題のはサンプルコードで超小規模だけど
実用的なプログラムだと
カウントの仕方が多種類で複雑になるから

**デフォルトの名無しさん** · 2017/10/21(土) 18:28:53.58

俺様が素晴らしい解答を用意してやったぞ
但し実行結果は見てない

class Automobile {
private static AtomicInteger count = new AtomicInteger();

private static AtomicInteger weight = new AtomicInteger();

public Automobile(int weight) {
this.weight.addAndGet(weight);
}

public int countAutomobile() {
return count.incrementAndGet();
}

public int countHeavy() {
return this.weight.get();
}
}

**デフォルトの名無しさん** · 2017/10/21(土) 18:37:22.59

よく考えたらincrementAndGetはコンストラクタでやるしかねえな
オブジェクト作る意味無いコード書いたの初めてだから中途半端なコードになっちまった

**デフォルトの名無しさん** · 2017/10/21(土) 18:42:09.63

ダメだ・・このあまりに低レベルな流れ見てると吐き気がしてくる・・
問題の質があまりに低レベルなの見れば設計とかそういう以前の問題で真面目に考える余地なんかないのに
staticだのコレクションだの将来性だのアホか・・

**デフォルトの名無しさん** · 2017/10/21(土) 19:10:52.71

>>220
だったら参加しなきゃいいのに
この話題が何日も続くわけじゃあるまいし

**デフォルトの名無しさん** · 2017/10/21(土) 19:17:48.71

スレタイ読めない奴は国語の勉強から初めた方がいいと思うの

**デフォルトの名無しさん** · 2017/10/21(土) 20:07:37.18

国語よりJavaのほうが必修だろ。

2017/10/22(日) 09:20:00.50

>>220
高レベルなレス期待してるわ

2017/10/22(日) 10:10:46.11

>>220
初心者スレに何を求めてるの？

2017/10/22(日) 12:27:36.98

初心者どうこういう以前にお前等のその解釈って無駄というか”無意味”なんだよね
問題文がインスタンス作りましょうとかっていうレベルのものなのに
問題に書かれない範疇のことまで勝手に想像を膨らませてあーだこーだ考えて
逆に質問者を混乱させ、陥れる無能の典型というか馬鹿の代表というか
なんだろうねこのド素人特有の客観的に問題点とそれに対する回答を指摘できない低知能ぶりは

2017/10/22(日) 12:33:56.55

いや学校の課題の質問だったから
わざとやってんだよ……

2017/10/22(日) 12:57:00.14

>>198
普通のパソコンが512mbくらいしか積んでなかったときからjavaは動いていたと思うが・・・

お前はどの世界線から来たんだ？

それとも、ただのうすら馬鹿か？

2017/10/22(日) 13:31:52.16

>>228
昔のjava重かっただろ

2017/10/22(日) 13:51:31.80

ただの口の悪いだけの奴だから気にするな

2017/10/22(日) 13:57:03.56

嘘を放置するのはよくないと思わんの？

2017/10/22(日) 14:05:17.09

仮にまともなこと言っていても口が悪い奴の相手するのは時間も精神的にも無駄

2017/10/22(日) 14:32:44.32

嘘つきがチヤホヤされるべきだとでも言うんかね
このバカは

つか、お前がうすら馬鹿かよwww

2017/10/22(日) 14:33:57.23

あ、うすら馬鹿に話しかけられると鬱陶しいから、消えてくれよな

頼むわw

2017/10/22(日) 14:40:05.22

>>226
>>210-212のレスも無意味やで
自分のやることには意味があると信じたいんだろうけど、もう少し客観的になろう

2017/10/22(日) 14:50:54.26

>>235
そうそう
空気読めないだけ

2017/10/22(日) 16:33:04.80

なんかさぁ
俺はjavaを勉強してるつもりだったのに気づいたらSQLやHTML/CSSの本を読んでるんだが間違ってないよな・・・？

2017/10/22(日) 16:45:56.69

バカがたむろってる所でしか粋がれないやつなんだからほっといてやれよ

2017/10/22(日) 18:06:04.46

>>226
そういうのは回答を指摘してから言え

2017/10/22(日) 18:17:46.07

入門書未満のくだらない質問の回答はお前等馬鹿の担当だけど
それですらまともにできないから呆れるわ
いつ見てもアホしかいない

2017/10/22(日) 18:20:42.92

>>240
じゃあお前は何でここにいるの？

2017/10/22(日) 18:29:26.25

>>237
CLI、GUIアプリ系の本を選べばだいたいJavaで完結する
あとは隠蔽するようなフレームワークとか使えば、とりあえずは知らなくてもできる

2017/10/22(日) 19:11:00.87

>>237
web系なら他言語行ってもつぶし効くから損はないと思う

2017/10/22(日) 19:31:25.01

>>237
いいじゃん。全部覚えろよ。

**デフォルトの名無しさん** · 2017/10/22(日) 20:21:51.73

>>221
初心者スレに必ず湧く奴だよ。

**デフォルトの名無しさん** · 2017/10/22(日) 22:19:13.28

gradleの使い方って覚えといて損はない?

**デフォルトの名無しさん** · 2017/10/23(月) 00:52:06.58

>200です。もう無理だw大学で配られた資料だけじゃだめだ、理解が追いつかない。本とか買わないとまずいかな。まあ勉強不足なのはわかってるんだがやっぱ無理なもんは無理だ。誰か教えて…

**デフォルトの名無しさん** · 2017/10/23(月) 01:05:36.93

>>247
実現するならstatic変数を作ってコンストラクタでインクリメントするだけ

staticを解説するためだけのアンチパターンな設計だから深く考えたらだめなやつ

**デフォルトの名無しさん** · 2017/10/23(月) 01:06:44.62

>>247
とりあえず何がわかんないのか箇条書きで書いて見なよ
親切なおじさんと不親切なおじさんがレスするから

**デフォルトの名無しさん** · 2017/10/23(月) 01:14:38.16

>>249
まず、合計台数だけを出力するプログラミングを作ろうと思ったんですけど、
整数型配列の初期化とかはあってるはずなんだけど、Automobileクラスでint countAutomobile()｛（←コンストラクタ？）内で何を打てばいいのかわからないです。ただ、return weightだと配列の１つ目の値だけが出力されますので。
https://i.imgur.com/APIeS93.jpg

**デフォルトの名無しさん** · 2017/10/23(月) 01:16:06.32

>>249
もちろん課題の内容としてはさらに大型車の台数も出力しないといけないんですけど、まず簡単なとこからと思って合計台数だけをやり始めたらそこすらできない次第です。

**デフォルトの名無しさん** · 2017/10/23(月) 01:17:56.31

>>250
コンストラクタじゃなくて、インスタンスフィールド？適当なこと言ってすみません…

**デフォルトの名無しさん** · 2017/10/23(月) 01:23:14.60

>>250
なるほど、それほど筋悪く無いと思うよ

他の人がいろいろ書いてるように1台の車を表現してるAutomobileクラスのインスタンスが
全部で何台あるのかを把握してること自体がおかしい、つまり課題のコードのほうがおかしい

だからそこは分からないほうが普通というか
100人いたら100人ともam[0].countAutomobile()とは書かないから
他のやり方したらいいんでないの？

**デフォルトの名無しさん** · 2017/10/23(月) 01:29:14.79

Automobile(int g)がコンストラクタだよ
int weight　→　static int weight　って変えてみたらわかると思う
253が言うように筋はいいと思う

>>253
残念ながら課題として出てるのでこれに回答せねばならないんだよ
とてつもなく不服だがな

**デフォルトの名無しさん** · 2017/10/23(月) 01:29:54.34

>>253
課題がおかしいんですかね…どこか一部を直せばいいんでしょうか？それか他の意見でもあったように新しいクラスを作るべきなのでしょうか？たびたびすみません。

**デフォルトの名無しさん** · 2017/10/23(月) 01:33:18.03

>>254
そちらがコンストラクタでしたか、ご指摘ありがとうごさいます。

**デフォルトの名無しさん** · 2017/10/23(月) 01:38:13.20

>>255
HighwayクラスにcountAutomobile()やcountHeavy()を作って上げるのが一番簡単

どうしても課題のコードに従ってやるならAutomobileクラスに
クラスレベルの変数として台数合計を管理するものを用意する

**デフォルトの名無しさん** · 2017/10/23(月) 01:55:19.63

>>257
前者の方法についてもう少し詳しく教えていただけませんか？自分の解釈ではまずHighwayクラス内にint countAutomobile()と int countheavy()をつくる。

**デフォルトの名無しさん** · 2017/10/23(月) 01:55:42.85

>>257
countAutomobileに関してはreturn weightで返して、heavyに関してはif文で大型車の時だけreturn で返せる。と考えたのですが、あってますでしょうか？無知ですみません

**デフォルトの名無しさん** · 2017/10/23(月) 01:57:12.01

int weight　→　static int weight　って変えてみたらわかると思う

**デフォルトの名無しさん** · 2017/10/23(月) 02:00:11.05

>>260
先ほどの画像にstaticを足してみたところ5つ目の配列の要素が表示されたのですが、どこがおかしいんですかね？

**デフォルトの名無しさん** · 2017/10/23(月) 02:01:39.23

>>261
配列の5つ目と言ったほうが正しいでしょうか

**デフォルトの名無しさん** · 2017/10/23(月) 02:05:09.61

>>261
8って出たならあってるよ
出力してるところでは1番目を指定してるのに5番目が出たってところが大事

**デフォルトの名無しさん** · 2017/10/23(月) 02:06:00.66

またアホどもが初心者を混乱させてるｗｗｗ
問題が糞で最初から台数わかってんだから台数はハードコーディングングでそのまま返せばいいんだよ
なんでsataticにすんだよ阿呆

**デフォルトの名無しさん** · 2017/10/23(月) 02:07:20.46

>>263
まだ途中の段階ってことですか？結果は8と表示されました。

**デフォルトの名無しさん** · 2017/10/23(月) 02:07:26.45

>>264
ちげぇねぇｗｗｗ
その発想はなかったわ

**デフォルトの名無しさん** · 2017/10/23(月) 02:11:15.70

>>265
まだまだ途中
そもそも台数を出すのであれば重さを設定する必要はない
static int weight　→　static int weight=0 って変えて
weight=g　→　weight++　って変えてみな

**デフォルトの名無しさん** · 2017/10/23(月) 02:20:42.33

>>267
言われた通りやってみたのですが、コンパイルでエラーが出てしまいました。int countAutomobile()の()内のintを消したのがまずかったでしょうか？
https://i.imgur.com/ULVykxO.jpg

**デフォルトの名無しさん** · 2017/10/23(月) 02:21:38.59

>>259
Automobileの配列使わないと

**デフォルトの名無しさん** · 2017/10/23(月) 02:23:07.33

>>268
yes
int gは消さずにそのまま
後で必要になるからね

**デフォルトの名無しさん** · 2017/10/23(月) 02:24:18.98

>>270
いけました！まあこれもまだ途中だと思うと…
しっかり5台と表示されました！

**デフォルトの名無しさん** · 2017/10/23(月) 02:28:32.08

>>271
ok
じゃあ次は
weight++　→　if(11<=g) weight++　って変えてみな

**デフォルトの名無しさん** · 2017/10/23(月) 02:28:43.54

あとはint countHeavy()を作るだけでしょうか？

**デフォルトの名無しさん** · 2017/10/23(月) 02:29:42.89

>>267
せめて static int total = 0 とかにして

**デフォルトの名無しさん** · 2017/10/23(月) 02:31:28.73

>>272
変えたところ2と出てきました。

**デフォルトの名無しさん** · 2017/10/23(月) 02:34:41.53

>>274
ひとまず、な
できるだけ手を加えずまずは動き
命名も大事だがまずは動かすことを目的とした

>>275
後はstatic変数を増やして
さっき5台って出たものと今2台って出たモノを組み合わせれば完成

**デフォルトの名無しさん** · 2017/10/23(月) 02:37:41.34

キモすぎてもう見てられん・・早く終わってくれ

public class Automobile {
static int TotalMobile = 0;
static int TotalLargeMobile = 0;

Automobile(int weight){
TotalMobile++;
if(weight >= 11){
TotalLargeMobile++;
}
}

public int countAutomobile(){
return TotalMobile;
}
public int countHeavy(){
return TotalLargeMobile;
}
}

public class Highway{
public static void main(String[] args) {
Automobile[] am = new Automobile[5];
int[] weight = {1, 15, 12, 5, 8};
for(int i=0; i<5; i++){
am[i] = new Automobile(weight[i]);
}
System.out.println("自動車総数：" + am[0].countAutomobile() + "台");
System.out.println("うち大型車：" + am[0].countHeavy() + "台");
}
}

**デフォルトの名無しさん** · 2017/10/23(月) 02:39:17.36

>>277
おーいいタイミング
ありがとな～

**デフォルトの名無しさん** · 2017/10/23(月) 02:42:53.75

皆さんありがとうございます！おかげでどうにか出来ました！この恩は一生忘れないです！

**デフォルトの名無しさん** · 2017/10/23(月) 02:45:26.99

>>279
恩は忘れていいのでこの問題の設計はウンコだって事だけ覚えててくれ

**デフォルトの名無しさん** · 2017/10/23(月) 02:48:37.75

>>280
けれどこんなヽ(・∀・)ﾉウソコ設計でもみなさんちゃんと理解して完成まで持ってけるなんてほんとにすごいと思います。

**デフォルトの名無しさん** · 2017/10/23(月) 02:51:09.46

お前と課題出した奴が馬鹿すぎるだけ