wb_sunny Solved wb_sunny
gavel
どなたかお知恵をかしてくださいまし。もう少しスマートに書きたいです。
PHP
CakePHP
thumb_up_alt
Good: 2
person
tanoki
2020/06/14

関数名と変数名は適当につけてます。 この部分でもっと良い書き方ってあったりしますでしょうか? 一応3項演算子を使ってみて、ちょっと省略したんですが。。。 どなたかお知恵をかしてくださいまし ``` public function test($a2, $b2, $c2, $d2, $e2) { $a1 = ''; $b1 = ''; $c1 = ''; $d1 = ''; $e1 = ''; $a1 = isset($a2) ? $a2 : ''; $b1 = isset($b2) ? $b2 : ''; $c1 = isset($c2) ? $c2 : ''; $d1 = isset($d2) ? $d2 : ''; $e1 = isset($e2) ? $e2 : ''; //下省略 } ```
php7.0以降で、「null 合体演算子」使えるようになったから、今回の場合isset使うならそれ使った方がよい かも。 あと変数の再代入を防げるのもでかいよ (この辺の記事が参考になるかも) https://qiita.com/jwhaco/items/6851a0a88a35c5a54fc0 ``` public function test($a2, $b2, $c2, $d2, $e2) { $a1 = $a2 ?? ''; $b1 = $b2 ?? ''; $c1 = $c2 ?? ''; $d1 = $d2 ?? ''; $e1 = $e2 ?? ''; } ```
person
いえーい!
2020-06-14 13:27:40
引数がnullである可能性があるのであればデフォルト引数値を設定しては如何でしょうか? ``` public function test($a2 = '', $b2 = '', $c2 = '', $d2 = '', $e2 = '') { $a1 = $a2; $b1 = $b2; $c1 = $c2; $d1 = $d2; $e1 = $e2; } ```
person
ほげるる
2020-06-16 07:53:31
ほげるるさん、ありがとうございます。 参考になります
person
tanoki
2020-06-16 22:52:16
いえーい!さん、ありがとうございます。 ぜひ使います。
person
tanoki
2020-06-16 22:52:46
×
😄😃😀😊😉😍😘😚😗😙😜😝😛😳😁😔😌😒😞😣😢😂😭😪😥😰😅😓😩😫😨😱😠😡😤😖😆😋😷😎😴😵😲😟😦😧😈👿😮😬😐😕😯😶😇😏😑👲👳👮👷💂👶👦👧👨👩👴👵👱👼👸😺😸😻😽😼🙀😿😹😾👹👺🙈🙉🙊💀👽💩🔥🌟💫💥💢💦💧💤💨👂👀👃👅👄👍👎👌👊👋👐👆👇👉👈🙌🙏👏💪🚶🏃💃👫👪👬👭💏💑👯🙆🙅💁🙋💆💇💅👰🙎🙍🙇🎩👑👒👟👞👡👠👢👕👔👚👗🎽👖👘👙💼👜👝👛👓🎀🌂💄💛💙💜💚💔💗💓💕💖💞💌💘💋💍💎👤👥💬👣💭🐶🐺🐱🐭🐹🐰🐸🐯🐨🐻🐷🐽🐮🐗🐵🐒🐴🐑🐘🐼🐧🐦🐤🐥🐣🐔🐍🐢🐛🐝🐜🐞🐌🐙🐚🐠🐟🐬🐳🐋🐄🐏🐀🐃🐅🐇🐉🐎🐐🐓🐕🐖🐁🐂🐲🐡🐊🐫🐪🐆🐈🐩🐾💐🌸🌷🍀🌹🌻🌺🍁🍃🍂🌿🌾🍄🌵🌴🌲🌳🌰🌱🌼🌐🌞🌝🌚🌑🌒🌓🌔🌕🌖🌗🌘🌜🌛🌙🌍🌎🌏🌋🌌🌠🌀🌁🌈🌊🎍💝🎎🎒🎓🎏🎆🎇🎐🎑🎃👻🎅🎄🎁🎋🎉🎊🎈🎌🔮🎥📷📹📼💿📀💽💾💻📱📞📟📠📡📺📻🔊🔉🔈🔇🔔🔕📢📣🔓🔒🔏🔐🔑🔎💡🔦🔆🔅🔌🔋🔍🛁🛀🚿🚽🔧🔩🔨🚪🚬💣🔫🔪💊💉💰💴💵💷💶💳💸📲📧📥📤📩📨📯📫📪📬📭📮📦📝📄📃📑📊📈📉📜📋📅📆📇📁📂📌📎📏📐📕📗📘📙📓📔📒📚📖🔖📛🔬🔭📰🎨🎬🎤🎧🎼🎵🎶🎹🎻🎺🎷🎸👾🎮🃏🎴🀄🎲🎯🏈🏀🎾🎱🏉🎳🚵🚴🏁🏇🏆🎿🏂🏊🏄🎣🍵🍶🍼🍺🍻🍸🍹🍷🍴🍕🍔🍟🍗🍖🍝🍛🍤🍱🍣🍥🍙🍘🍚🍜🍲🍢🍡🍳🍞🍩🍮🍦🍨🍧🎂🍰🍪🍫🍬🍭🍯🍎🍏🍊🍋🍒🍇🍉🍓🍑🍈🍌🍐🍍🍠🍆🍅🌽🏠🏡🏫🏢🏣🏥🏦🏪🏩🏨💒🏬🏤🌇🌆🏯🏰🏭🗼🗾🗻🌄🌅🌃🗽🌉🎠🎡🎢🚢🚤🚣🚀💺🚁🚂🚊🚉🚞🚆🚄🚅🚈🚇🚝🚋🚃🚎🚌🚍🚙🚘🚗🚕🚖🚛🚚🚨🚓🚔🚒🚑🚐🚲🚡🚟🚠🚜💈🚏🎫🚦🚥🚧🔰🏮🎰🗿🎪🎭📍🚩🇨🇳🇩🇪🇪🇸🇫🇷🇬🇧🇮🇹🇯🇵🇰🇷🇷🇺🇺🇸1⃣2⃣3⃣4⃣5⃣6⃣7⃣8⃣9⃣0⃣🔟🔢#⃣🔣🔠🔡🔤🇦🇧🇨🇩🇪🇫🇬🇭🇮🇯🇰🇱🇲🇳🇴🇵🇶🇷🇸🇹🇺🇻🇼🇽🇾🇿🔄🔼🔽🆗🔀🔁🔂🆕🆙🆒🆓🆖📶🎦🈁🈯🈳🈵🈴🈲🉐🈹🈺🈶🈚🚻🚹🚺🚼🚾🚰🚮🅿🚭🈷🈸🈂🛂🛄🛅🛃🉑🆑🆘🆔🚫🔞📵🚯🚱🚳🚷🚸💟🆚📳📴🅰🅱🆎🅾💠🔯🏧💹💲💱©®🔝🔚🔙🔛🔜🔃🕛🕧🕐🕜🕑🕝🕒🕞🕓🕟🕔🕠🕕🕖🕗🕘🕙🕚🕡🕢🕣🕤🕥🕦💮💯🔘🔗🔱🔺🔲🔳🔴🔵🔻🔶🔷🔸🔹