美しいコードって？

質問

こちらで質問すると、丁寧で素早い解答がいただけるので今回も質問させて下さいm(__)m

大学の授業でjavaを使ったプログラム作成が課題として出ました。
Javaビギナーなので、多分クラスでも一番オーソドックス、悪く言えば教科書通りすぎて長い？解答をしたと思います。

しかし、昨日、返却されたときに、教授から、正しく作動するから合格だけど、もうちょっと工夫して美しいコードを書くように、と言われました。

美しいコードって余分なものがない簡潔なものということだろうと思いますが、正直まだその工夫の仕方がわかりません。
友人何名かの解答も見ましたが、初心者クラスだからか？あまり差はありませんでした。

お時間があるときにでもきれいな？コードをかくためのアドバイスを頂けたら嬉しいです。

一応、その課題ものせてみます。
1000円札5枚、500円玉5枚、100円玉4枚、50円玉2枚の計8000円が入っているマシンAがある。
ユーザーが入れた金額を引数として、マシンが出す金額とその内訳がわかるプログラムを書け。
ただし、マシンAが出す金額が8000円を越えた場合は処理を終了する。

よろしくお願いします。

回答No.2

root139 · ベストアンサー

原則的に、下記の様な点が満たされているものが美しいコードと言えるかと思います。

1. 理解し易い。
2. 重複が無い。
3. 量が少ない。

これを達成するためには下記の様なプラクティスなどが考えられます。

・改行・インデントを読みやすく一貫したものにする。

・関数・変数・クラスなどの名前は、意味が分かるものにする。意味が曖昧にならない範囲で短くする。

・無駄に変数を使用しない。使わないと出来ない場合や使った方が分かり易い場合だけ使う。

・変数やオブジェクトは出来る限り不変にする。

これらは、広く普及しているベストプラクティスのごく一部です。

このような良いコードを書く話題を扱った書籍はいろいろと有りますので、参考になりそうなものを挙げておきます。
お勧めの順番に並べましたが、読みやすくて安いという意味では、「リーダブルコード」が良いかも知れません。また、大学の先生に受けが良いのは、アルゴリズム系を扱った「珠玉のプログラミング」かも知れません。

■ Code Complete第2版
http://www.amazon.co.jp/dp/489100455X

■ Effective Java 第2版
http://www.amazon.co.jp/dp/489471499X

■ プログラミング作法
http://www.amazon.co.jp/dp/4756136494

■ リファクタリング
http://www.amazon.co.jp/dp/4894712288

■ 珠玉のプログラミング―本質を見抜いたアルゴリズムとデータ構造
http://www.amazon.co.jp/dp/4894712369

■ 実装パターン
http://www.amazon.co.jp/dp/4894712873

■ Clean Code
http://www.amazon.co.jp/dp/4048676881

■ リーダブルコード
http://www.amazon.co.jp/dp/4873115655

ちなみに、ズバリ「ビューティフルコード」という本も有りますが、この質問とは少しスケールの違う話題を扱っている感じですね。
http://www.amazon.co.jp/dp/4873113636

参考URL：

http://www.alles.or.jp/~torutk/oojava/codingStandard/writingrobustjavacode.html

お礼

やはり、量が少ないのはキレイなコードの一つなのですね。また、自分の解答で重複しているが多々あり、美しくない一番の原因はこれかー！と思ってちょっとへこみました。この質問の枠におさまらなかったくらいなので；；でも確かにこの重複をもっと簡潔にかけるようになれば、短くて分かりやすいコードが書けるようになるなと思います。
インデントも結構、曖昧にしていた部分が多かったので、これから気をつけたいと思います。

全てのリンクに目を通させて頂きました。現在持っているのはjavaの基本的な分厚い二冊なので、お薦めしていただいたリーダブルコード、個人的に気になったeffective java、プログラミングの作法、clean code、ビューティフルコードに書店などで目を通して、購入を検討したいと思います。珠玉のプログラミングも教授側からみたキレイなコードの例ということで参考にさせて頂きます。

丁寧なアドバイスありがとうございました。

回答No.1

jjon-com

自分ならどんなコードを書くだろう，と興味がわいたので試してみました。

class Q7960224 {
public static void main(String[] args) {
int[] denomi = {1000, 500, 100, 50};
int[] denums = { 5, 5, 4, 2};

int amount = Integer.parseInt(args[0]);

if (amount > 8000 || amount % denomi[denomi.length - 1] != 0) {
System.out.println("マシンA内の金種では支払いできない");
System.exit(0);
}

for (int i = 0; i < denomi.length && amount > 0; i++) {
int number = Math.min(denums[i], amount / denomi[i]);
if (number != 0) {
System.out.println(denomi[i] + "円が" + number +"枚");
amount -= denomi[i] * number;
denums[i] -= number;
}
}
}
}

お礼

当方が書いたコードは、多分類似の問題でよくある繰り返しを使った（***円の場合、、、）ので、すごく長くなりました。なので、短さにびっくりです。しかもそれでも数時間かかったので（！）、解答された時間の早さに驚きました。
解答ありがとうございました。