式を演算子を使ってまとめたい。 - PHPプロ！Q&A掲示板

2609

0P

式を演算子を使ってまとめたい。

質問日時 / 2010年3月9日 11:58 回答数 / 7件

Questioner: php_baby Tweet

　　　　//入力された西暦のフォルダの有無を調べ、無ければつくる。
if ( !is_dir ( ROOTPATH . "/$year" ) ) {
mkdir ( ROOTPATH . "/$yearr" ) ;
}
//日時のフォルダ調べ、なければ作る
if ( !is_dir ( ROOTPATH . "/$year/$date" ) ) {
mkdir ( ROOTPATH . "/$year/$date" ) ;
}

上記のように、フォルダの有無を調べ、その有無によって、さらに、その中にフォルダを作るような
そんなコードを書きました。
ROOTPATHは、固定のパスです。

ボスに見せたところ、演算子を使ってもっとかっこいいコードを書け！！
さらに、うまくいかないときに、エラーを返すようにしろ！

と言われてしまいました。

いろいろ考えてみたのですが、
まず、演算子で上手にまとめる方法がよくわかりません。

わかるかた、おしえてください。
よろしくお願いします。

この質問への意見の募集は締め切られ、ポイントは既に配分されました。
意見を投稿することはできますが、ポイントを受け取ることはできません。

この質問に返信

ツリー一覧

式を演算子を使ってまとめたい。php_baby

A01magicflute2・関数化・三項演算子・mkdir recursiveパラメー

A01-1php_babyすみません。 mkdirのrecursiveパラメータをTRUEと

A01-1-1shimix>mkdir(ROOT/$year/$date , TRUE); recursiveは第3

A01-1-2magicflute2> また、ラッパー関数というのがわからないので教えて

A02kaitau> ボスに見せたところ、演算子を使ってもっとかっこい

A03signal>>CODE <?php try { if (!is_dir(ROOTPATH."/{$y

A04php_babyみなさま、いろいろとご教授いただきありがとうござい

回答一覧

A01
answerermagicflute2 [3月9日 12:53] (最終編集：3月9日 13:08)

・関数化
・三項演算子
・mkdir recursiveパラメータをTRUE
要はmkdirのラッパー関数を作るという事。
うまくまとめると、4行以内で書ける。

A01-1
replyerphp_baby [3月9日 13:44] (最終編集：3月9日 13:47)

すみません。

mkdirのrecursiveパラメータをTRUEというのは

mkdir(ROOT/$year/$date , TRUE);
と書けば、$yearフォルダが、特になくても階層ごと$dateフォルダが作られると判断したのですが、
うまくできませんでした、
どこが間違っているのでしょうか？

また、ラッパー関数というのがわからないので教えてもらえますか？

よろしくおねがいします。

ちなみに、phpのバージョンは5.2.6です。

A01-1-1
replyershimix [3月9日 13:53] (最終編集：3月9日 13:57)

>mkdir(ROOT/$year/$date , TRUE);

recursiveは第3引数なんだが・・。せめて関数の引数くらいはマニュアルで確認してから取りかかって欲しい。

　　http://www.php.net/manual/ja/function.mkdir.php

三項演算子はこのあたり・・

　　http://www.php.net/manual/ja/language.operators.comparison.php

A01-1-2
replyermagicflute2 [3月9日 16:00] (最終編集：3月9日 16:07)

> また、ラッパー関数というのがわからないので教えてもらえますか？
私の解釈は以下の通りです。
http://e-words.jp/w/E383A9E38383E38391E383BC.html

今回ケースだと、以下の様に、機能拡張・初期設定をした関数です。
標準関数を少しだけ、目的にあった形にします。
・mkdir前にディレクトリ存在チェック
・エラー処理
・パーミッションデフォルト値の設定
・recursiveパラメータデフォルト値の設定

<?php
/**
* PHP5なので、引数にデフォルト値を設定できます
* @param $path string
* @param $mode int
* @param $recursive boolean
* @return boolean
*/
function wrapper_mkdir( $path, $mode=0705, $recursive=TRUE )

A02
answererkaitau [3月9日 13:53]

> ボスに見せたところ、演算子を使ってもっとかっこいいコードを書け！！
> さらに、うまくいかないときに、エラーを返すようにしろ！

多分これは教育の意味を込めての事なんじゃないかな。

だとすると、自分で考えて解くか、わからない事はそのボスに
聞くべきだと思ったり。

趣味ならともかく、仕事なら、自分の理解度を上司に示して
伸ばしてもらうのも大事な事だと思いますが。
というか、私が上司だったら、ネットで調べただけの成果物で
「出来ました」といわれても、「コイツどこまでわかってるんだ？」ってなります。
まぁ、一般論では無いので、意見が分かれるところではありますが。

wikipediaに出てくるような単語の意味、とかならともかく、
ここをこうしろ、と言われて、わかってないのに「はぁ、わかりました」
って持ち帰られるのが一番危険なんですよね。

A03
answerersignal [3月10日 23:28]

<?php
try {
if (!is_dir(ROOTPATH."/{$year}") &&
!@mkdir(ROOTPATH."/{$year}")) {
throw new exception("/{$year} が作れない");
}
if (!is_dir(ROOTPATH."/{$year}/{$date}") &&
!@mkdir(ROOTPATH."/{$year}/{$date}")) {
throw new exception("/{$year}/{$date} が作れない");
}
} catch (exception $e) {
print $e->getmessage();
print "のは全てボスの責任です";
}
?>

A04
answererphp_baby [3月11日 10:24]

みなさま、いろいろとご教授いただきありがとうございました。
確かに、教育の意味も込めてのことだと思いました。

最終的に、まとめたので、
紹介します。

エラーについては、今回はfalseを返すのみになってます。

if ( ( ( is_dir( ROOTPATH . "/$year" ) || mkdir ( ROOTPATH . "/$year" ) ) && ( is_dir( ROOTPATH . "/$year/$date" ) || mkdir ( ROOTPATH . "/$year/$date" ) ) ) ) {
} else {
return false;
}