連想配列を丸ごと引数にするべきか？
- プログラミングの作法的な質問ですが、連想配列を丸ごと引数にするべきか悩んでいます。
- 引数を個別に渡す場合には分かりやすいですが、引数が多くなる場合もあります。
- 一方、連想配列を引数にすると、引数の数が変わっても呼び出し元を変更する必要はありませんが、中身を確認する必要が出てきます。
- ベストアンサー
連想配列を丸ごと引数にとるのはアリ？
プログラミングの作法的な質問ですが、 たとえばdoSomething($param)メソッドを自分で作るとして、 そのときに渡す引数がたとえば顧客情報だとしまして、このとき、 doSomething($aryCustomerInfo["name"], $aryCustomerInfo["address"], ...) と要素をひとつずつ渡すべきか、もしくは doSomething($aryCustomerInfo) と丸ごと渡した方がいいでしょうか? 1つずつ渡せば何を渡しているのか分かりやすいですが、引数がやたら多くなるときがあります。 doSomething($aryCustomerInfo)は、引数の配列の要素数が減ったり増えたりしても呼び出し元を変更しないでいいですが、実際何を渡しているのかはメソッド内を見てみないと分からない問題がありますよね? どちらがよろしい、あるいは美しいのでしょうか？
- dalianse
- お礼率25% (24/94)
- PHP
- 回答数2
- ありがとう数0
- みんなの回答 （2）
- 専門家の回答
質問者が選んだベストアンサー
実際にそういう形をとるものを最近よく見かける気がします。 私見ですが、形式自体は別に問題ないと思います。肝心なことは引数となる連想配列の内容がしっかり解説されていることだと思います。
その他の回答 (1)
- mizutaki
- ベストアンサー率33% (111/333)
関数に渡す値は出来るだけ最小の形が望ましいです 無駄な値や配列は出来るだけ渡さない方がデバッグや拡張がしやすくなります 分かりにくい場合は、出来るだけ関数名などで分かるようにしましょう ただ、例外として 関数内でデータベースに問い合わせしたり、 配列の前後や配列に入ってるフラグや値が関係してくる（将来そうなる可能性が高い）場合は 配列のまま渡した方がよかったりします
関連するQ&A
- 配列の値をまとめる？
いつもお世話になっております。 Javaプログラミングについて質問させてください。 プログラム中で配列を用意しています。 (String[] param = new String[7]) この配列には値が入っています。 (param[0]="000",param[1]="000",param[2]="000",param[3]="111",param[4]="111",param[5]="333",param[6]="333") 配列に入っている値は全部で７つですが、種類としては３つです。 ("000","111","333") 配列に入っている値を元に、上記の種類(３つ)を格納するようなメソッドはありますか？ ご存知の方がいらっしゃいましたら、ご教授ください。 宜しくお願い致します。
- ベストアンサー
- Java
- 配列や連想配列について
言語一般の配列や連想配列について 最近どういう解釈をしていいのか わからなくなってしまいました。 仮にPerlで話をさせてもらいますと、 連想配列%FORMがあるとき、 個々の要素にアクセスする場合 例えば$FORM{'name'}などと記述しますが、 この$FORM{'name'}を みなさんはどのように解釈しているでしょうか？ $FORM{' '}にキー nameを 代入しているようなイメージで、 「連想配列%FORMからキー'name'の値を取得する」 という考え方でしょうか？ わたしは最近、 $FORM{'name'}はそれ自体が一つの変数という 考え方が出来るのではないかという気がしているのですが、 連想配列の実装から考えると この考え方は正しいのでしょうか？ 以前、配列について同様の質問を したことがあります。 例えばC言語でint a[100]と宣言した場合は、 連続的なアドレスに確保されるため、 a[2]はaという名前のアドレスから a + 2 番目の場所という意味になるが、 言語によっては不連続なアドレスに 確保されるため、a[2]は一つの変数名として 考える必要がある・・ との回答でした。 ならば連想配列の場合も もし不連続なアドレスに確保されるならば、 $FORM{'name'} は一つの変数名と 考えたほうが正しいのではないかと 思ったからです。 連想配列の内部的な処理がどうなっているのか 教えて頂きたいです。 よろしくお願いします。
- ベストアンサー
- Perl
- IntArrayクラスのプログラムを組んでいるのですが・・・
javaプログラミングで以下の内容を満たす、IntArrayクラスを作成しています。 ・インスタンス変数 int型の配列data ・コンストラクタ (1)int型の配列を受け取り、そのコピーを内部的に保持する (2)第1引数で指定された要素数を持つ配列を確保し、全ての要素に初期値として第2引数で指定された値をセットする (3)第1引数で指定された要素数を持つ配列を確保し、全ての要素に初期値としてゼロをセットする ・メソッド (1)sort 内部的に保持している配列を、引数の値がtrueであれば昇順、falseであれば降順にソートする (2)length IntArrayが保持している配列の要素数を取得する (3)getElement 引数に指定された要素番号の値を取得する (4)setElement 第1引数に指定された要素番号に第2引数で指定された値を格納する (5)getArray IntArrayが保持している配列のコピーを取得する 実際にプログラムを組んでみたのですが、※の部分が冗長だと指摘を受けました。 しかしどのように修正したらよいかわかりません。 どうかアドバイスなどをよろしくお願いいたします。 package java_Lesson; import java.util.Arrays; /** * int型の配列dataの要素を取得、あるいは操作するクラス */ class IntArray { private int[] data; /** * int型の配列を受け取り、そのコピーを内部的に保持する * * @param args コピー元となる配列 */ IntArray(int[] args) { data = (int[])args.clone(); } /** * 第1引数で指定された要素数を持つ配列を確保し、 * 全ての要素に初期値として第2引数で指定された値をセットする * * @param arg1 配列の要素数 * @param arg2 配列の初期値 */ IntArray(int arg1, int arg2) { this(arg1); /* ※ */ for (int index = 0; arg1 > index; index ++) { data[index] = arg2; } } /** * 第1引数で指定された要素数を持つ配列を確保し、 * 全ての要素に初期値としてゼロをセットする * * @param arg 配列の要素数 */ IntArray(int arg) { data = new int[arg]; } /** * 内部的に保持している配列を昇順、あるいは降順にソートする * * @param arg trueであれば昇順、falseであれば降順にソート */ void sort(boolean arg) { Arrays.sort(data); if (arg) { return; } int array = data.length - 1; for (int index = 0; array > index; index ++, array --) { int temp = data[index]; data[index] = data[array]; data[array] = temp; } } /** * IntArrayが保持している配列の要素数を取得する * * @return 配列の要素数 */ int length() { return data.length; } /** * 引数に指定された要素番号の値を取得する * * @param arg 返す要素の要素番号 * * @return 配列dataの要素番号argの値 */ int getElement(int arg) { return data[arg]; } /** * 第1引数に指定された要素番号に第2引数で指定された値を格納する * * @param arg1 値を格納する要素番号 * @param arg2 配列に格納する値 */ void setElement(int arg1, int arg2) { data[arg1] = arg2; } /** * IntArrayが保持している配列のコピーを取得する * * @return 配列dataのコピー */ int[] getArray() { return data.clone(); } }
- ベストアンサー
- Java
- [perl]メソッドの引数で受け取った配列データを直接編集したい
現在、業務でperlプログラムを書いてます。 perlを初めて２か月、プログラムは ほぼ初です。 よろしくおねがいします。 例えば以下のようなコードで、「sub_func」というメソッド内で「main」メソッド内の配列データを直接編集したいのですが、うまくいきません。 これは、引数で渡しているのは配列の「ポインタ」ではなく「リファレンス」であるため、「sub_func」というメソッド内で、新たに「array_ref」という配列の実態を作成してしまっているのだと認識しています。 そのため、「main」メソッドに戻ったタイミングではデータは元に戻ってしまうのだと思っています。 sub sub_func { my ($self, $array_ref) = @_ ; $array_ref = [4,5,6]; # returnはせずに、引数で指定したデータに格納したい } sub main { my ($self) = @_ ; # メソッドに渡すデータ my $data = { array => [1,2,3], }; my $data2 = [1,2,3]; my @data3 = (1,2,3); # メソッドの中で、$data->{array}を [4,5,6] に変えたいが・・・ $self->sub_func( $data->{array} ); $self->sub_func( $data2 ); $self->sub_func( \@data3 ); # この時点で、どれも [1,2,3] のまま } 私の認識は間違っているのでしょうか？？ また、どのようにすれば配列の要素を別のメソッド内で編集できるのでしょうか？？ 御助言をお願いします。
- ベストアンサー
- Perl
- String配列とHashMap
メソッドの引数の数がとても多いので、String配列かHashMapにまとめて、値を渡すことになりました。 どちらを使うかなのですが、私の記憶だとString配列とHashMapの違いを次のように認識してるのですが、調べても答えが見つかりません。合ってるか教えてください。 String配列は、インデックスを検索しつつ値を取得してるので、インデックスの大きい番号ほど、値取得時の処理に時間を使う。HashMapは、特別な検索方法（？）で検索してるので、値取得時の処理に、時間をかけずにすむ。 なので、メソッドの引数として使うのはHashMapの方がよい。 メソッドの引数という性格上、要素数を固定しなくてもいいHashMapの方が使いやすいと思ってるのですが、現場のリーダーさんの認識とずれちゃってます。説得したいので、この辺りのことが解説されているサイトがありましたら、合わせて教えていただけると嬉しいです。
- ベストアンサー
- Java
- 引数で指定された配列の要素数の取得
どうもこんにちは。 C言語でプログラムを作成しています。 ある関数に配列を渡すことを考えていますが、渡した配列の要素数を取得する方法は何かありますか？ 標準の関数を見ても、配列の先頭アドレスのポインタとともに、配列の要素数を渡しているものばかりで、配列のポインタを渡しているものは見かけません。 要素数があらかじめわかっていれば、それを引数の型に指定できますが、呼び出されるまで不明な場合はうまくいきません。 配列の要素数も引数として一緒に渡す必要がありますか？ [作ってみたサンプル] #include <stdio.h> #include <stdlib.h> #include <string.h> // func1 と func2 をまとめられないだろうか。。。 void func1(int (*p)[10]) { ________int n = sizeof(*p) / sizeof((*p)[0]); ________int i; ________for (i = 0; i < n; i++) { ________________printf("%d\n", (*p)[i]); ________} ________printf("\n"); } void func2(int (*p)[5]) { ________int n = sizeof(*p) / sizeof((*p)[0]); ________int i; ________for (i = 0; i < n; i++) { ________________printf("%d\n", (*p)[i]); ________} ________printf("\n"); } int main(int argc, char *argv[]) { ________int ary1[10] = { 2, 4, 6, 8, 0, 1, 3, 5, 7, 9 }; ________int ary2[5] = { 3, 6, 9, 12, 15 }; ________func1(&ary1); ________func2(&ary2); ________return 0; }
- ベストアンサー
- C・C++・C#
- 配列とその要素数をメンバにもつ構造体
C言語では，配列を引数とするときに，合わせて要素数を渡さなければいけないといわれます． そこで，配列とその要素数をメンバにもつ構造体を定義して，これを要素数つき配列のように扱えば，引数として渡す場合には，この構造体を渡すだけでよいと思うのですが，このような用法は一般的に用いられているでしょうか． 単純なアイデアだと思うんですが，あまり紹介されているのを見たことがありません． もしあまり用られない用法であれば，その理由を教えてください． 現在，プログラムを書いているのですが，引数の多さに閉口しており，上の考えで，引数を減らしたいと考えています． よろしくお願いいたします．
- ベストアンサー
- C・C++・C#
- 関数へのポインタ渡しでの配列の初期化について
はじめまして、C言語の基本的な質問をさせてください。 C言語で、外部関数へポインタで引数を渡す場合に、 関数に渡されるのはアドレスですよね？ で、渡された関数側でそのポインタの配列の初期化を するときにはアドレスだけの情報だと、要素数がいくつ あるか分からず、領域の破壊をしてしまいそうな気が するのですが？いかがでしょうか？ また、関数かなんかで、配列の要素数が分かる関数が あったような気がするのですが、それもアドレスだけ でわかるのでしょうか？
- ベストアンサー
- C・C++・C#
- エクセルマクロのサブルーチン引数で配列を使用したい
VBA初心者です。 エクセルのマクロを組んでいるですが、サブルーチンの引数を配列にしたいのですが、どうすればよろしいでしょうか？ 下記に例を示します。 呼び出し（Call)とサブルーチンの配列変数の記載方法がわかりません。 特に気になるのは、サブルーチンの配列に配列数（下記であれば10）を記入する必要があるのか？ 必要な場合、元の配列数を変更した場合、どうすればミスなく、漏れなく修正できるか、良い方法はありまんせんか？ （例） （呼び出し側） Dim AA(10) As Integer Call Test(AA) ← 引数はAAでよい？それともAA() Sub Test(ByRef AA As Integer) ← AA or AA() or AA(10) ? AA(3) = AA(2) + AA(1) 以上、お手数をおかけしますが、よろしくお願いします。
- ベストアンサー
- オフィス系ソフト
- VB.net 引数で配列変数を渡す際の要素数
VB.net 2010 の質問です。 配列変数を他のプロシージャに引数で渡す際の、配列変数の要素数をあらかじめ宣言していないとエラーになります。 Dim Hairetsu() As String ' ←エラー Dim Hairetsu(100) As String ' ←正常 Public Sub aaa(ByRef Hairetsu() As String) Hairetsu(0) = "Test" '←正常の場合と、エラーの場合 End Sub Hairetsu(100) のように、要素数を宣言していると、エラーにならないのですが、 プログラム中、要素数がどこまで増加するか、分かりません。 Hairetsu()のように、要素数未確定の配列変数を渡すことは出来ないのでしょうか。 ご存知の方がいらっしゃいましたら、なにとぞアドバイスの程、よろしくお願いいたします。
- ベストアンサー
- Visual Basic
補足
なるほどー。分かりにくい部分をドキュメントやコメントで補う発想でね。参考になりました。 結局は設計者次第でしょうか？