質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.59%

javaで定義したメソッド(戻り値、引数含む)をdoGet内のメソッドにてメソッドを呼び出す処理をしたいがうまくいかないです。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 2
  • VIEW 1,349
退会済みユーザー

退会済みユーザー

java初学者なので粗い質問をご了承下さい。
### 前提・実現したいこと

userProfileというユーザー定義メソッドを作り、引数は数値型

戻り値を文字列の配列で返すやり方と引数のやり方を、メソッド内にて複数の配列を作る方法を知りたいです

またdoGet以下のメソッドにて受け取った配列の表示を for 文で行いたいのですが手詰まりといった感じです。何か良い書き方はありますでしょうか

valueofとintegerでの値と文字列の変換変換を習っております

戻り値:文字列型を要素に持つ配列型

メソッド内で値を要素とする配列(計3つ)を作成し、

また、どの値にも一致しない場合は null を返却したい。
順番に各配列の0番目の値と引数から受け取った数値型の値で比較処理をして
比較して一致した場合、呼び出し元へ一致した配列を返却したい。
※数値型と文字列型を比較する時に型変換をする。

メソッドを作成したら,処理が正しく実装されているか確認するため、
このメソッドを呼び出す処理を記述。
呼び出した後、返却された配列の表示を行って下さい。
・3つある配列の内、1つが表示される処理にしたい。
・返却された配列は反復処理(for)を使用して表示。その際に0番目の要素は表示しない。

3つ作成した配列の値を引数と比較し型変換することに歯が立ちませんご指摘お願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • 退会済みユーザー

    退会済みユーザー

    2019/03/24 11:13

    jimbe様へ回答ありがとうございます。
    valueofとintegerでの値と文字列の変換変換を習っております

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/03/24 11:32

    shinamiさんご指摘ありがとうございます。実現したいのは3の3.それとも戻り値を文字列の配列で返すやり方と引数のやり方が主です。おっしゃる通り配列を見直してコードを書き直して見ます。

    キャンセル

  • 退会済みユーザー

    退会済みユーザー

    2019/03/24 11:43

    swordone様ご指摘ありがとうございます。
    配列の初期化がおかしいとは、String[] datas = new String[]以下がおかしいということでしょうか? 複数の配列を用いたやり方が全くできていないのでご指摘お願いしたいです。宜しくお願いします。

    キャンセル

回答 3

checkベストアンサー

+3

userProfileというユーザー定義メソッドを作り、引数は数値型
戻り値:文字列型を要素に持つ配列型

String[] userProfile(int datas){
 :
}

で合っていると思います.
出来れば, 引数の変数名は "datas" では無く(配列の番号の指定のようですので) "number" 等のほうが良いかなと思います.


メソッド内で以下の値を要素とする配列(計3つ)を作成し、
値1 ... ["1","技育太郎","東京","男","プログラマー"]
値2 ... ["2","技育花子","北海道","女","システムエンジニア"]
値3 ... ["3","技育三郎","大阪","男","Webエンジニア"]

String[] datas = new String[] {"1","技育太郎","東京","男","プログラマー"},  
{"2","技育花子","北海道","女","システムエンジニア"},  
{"3","技育三郎","大阪","男","Webエンジニア"} 

これは, swordone さんがご指摘の通り, まずコンパイルを通りません. 2次元配列を習っていらっしゃるかどうかというところです.
また, 問題文には「配列(計3つ)を作成」とあります. すなわち,

String[] data1 = {"1","技育太郎","東京","男","プログラマー"};  
String[] data2 = {"2","技育花子","北海道","女","システムエンジニア"};  
String[] data3 = {"3","技育三郎","大阪","男","Webエンジニア"}; 


ではないでしょうか.


順番に各配列の0番目の値と引数から受け取った数値型の値で比較処理をして
比較して一致した場合、呼び出し元へ一致した配列を返却したい。
※数値型と文字列型を比較する時に型変換をする。

3つの配列のそれぞれ 0 番目すなわち"1","2","3"の部分と引数 datas を比較して, 一致したらその配列を返す, 件の中心部分でしょうか.

data1 のみで考えます.
0 番目の値は data1[0] で String 型です. 引数の datas は int 型です.
型変換には2通りあります. String を int にするか, int を String にするか, です.
valueOf を習ったそうですので, String の datda1[0] を int (正確には Integer) に変換します.

Integer.valueOf(data1[0])


です. そして引数と比較します.

Integer.valueOf(data1[0]) == datas


比較して一致したら処理を分岐するので if 文の条件とします

if(Integer.valueOf(data1[0]) == datas){
}


一致した配列, すなわち data1 を返却します.

if(Integer.valueOf(data1[0]) == datas){
  return data1;
}


これを各配列で順番に行います.

if(Integer.valueOf(data1[0]) == datas){
  return data1;
}
if(Integer.valueOf(data2[0]) == datas){
  return data2;
}
if(Integer.valueOf(data3[0]) == datas){
  return data3;
}

さて, お気づきかもしれませんが, 引用を一行飛ばしました.

また、どの値にも一致しない場合は null を返却したい。

null の返却は

return null;


です.
どの値にも一致しなかった場合なので, 3つの if 文の後に記述する必要があります.


という感じで userProfile の中に記述する部分を 3つに分けてみました.
それぞれ左上に "java" と出ている枠内のコードを userProfile の中に順に記述すれば, userProfile は完成すると思います.

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/25 19:48

    上記のコードを改変です。何度も申し訳ございません。

    protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    // TODO Auto-generated method stub
    response.setContentType("text/html;charset=UTF-8");
    try(PrintWriter out= response.getWriter()){
    String[] data1=userProfile(1);


    for (int i=1;i<data1.length;i++) {
    out.println(data1[i]);

    }





    }

    これで3つある配列のうち一つの配列を表示することができましたが
    改善点がありましたらご指摘お願いします。



    }

    キャンセル

  • 2019/03/25 20:11 編集

    あらら、コメントボタンを何度か押されてしまったのでしょうか.

    > ・3つある配列の内、1つが表示される処理にしたい。
    → userProfile(1)

    > ・返却された配列は反復処理(for)を使用して表示。その際に0番目の要素は表示しない。
    → for 文のループカウンタ(i)の開始を 1 とする

    どちらも達成できているのではないでしょうか^^
    十分「歯が立っている」と思います.
    この場合の for 文は, 拡張 for 文では返って面倒なことになるので, 普通の for 文が正解だと思います.

    強いてご注意点を挙げますと, userProfile(1); の戻りを受け取る変数名を data1 では無く data の方が良いかなと思います.
    data1 としますと, 「では userProfile(2) を受け取るときは data2 と変えるのかな」的に見えます. まぁこれは深読み過ぎという感じも致しますが.

    キャンセル

  • 2019/03/25 20:41

    コメントボタン重複失礼しました。
    指摘ありがとうございます。
    変数名をdataに変えてみます。
    長く付き合って頂いたおかげで達成することができました。本当にありがとうございました。

    キャンセル

+1

配列については調べ中&書き直し中と思われますので引数と戻り値について

引数

void userProfile(○○○○型 hensu) {
  //内容
} 

public void mainsyori() {
  userProfile(○○○○型の変数や定数データそのもの)
}


よって、○○○○は一致する型にしなければなりません。

戻り値

○○○○型 userProfile(){
  ○○○○ datas;
  return ○○○○型の変数や定数データそのもの; //datasなど
}

public void mainsyori() {
  ○○○○型 hensu;
  hensu = userProfile();
}


よって、○○○○は一致する型にしなければなりません。

質問者さんのソースでは

String[] userProfile(int datas){ //String[]は○、同じ変数名datasがあるのでどちらか変える。
  String[] datas = new String[] {"1","技育太郎","東京","男","プログラマー"}
  return datas; //int datasの同じ変数名がなければ○
}

protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {    
  String[] datas = userProfile(); //userProfile(この中はint型の変数か定数)が無い。
} 

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/25 13:43

    戻り値と引数についての解説ありがとうございます。勉強になりました。

    キャンセル

+1

jimbeさん等が配列を丁寧に説明されているので、少し難易度の高い話をしましょう。
質問はuserProfileというクラス内にメソッドを作り、引数はint型1つのみ、戻り値はstring型配列
とのことでしたが、返したい値を見ている限りこのような感じが良いのではないかと思いました。

値1 ... ["1","技育太郎","東京","男","プログラマー"]
値2 ... ["2","技育花子","北海道","女","システムエンジニア"]
値3 ... ["3","技育三郎","大阪","男","Webエンジニア"]
型解説  [int型,string型,string型,boolen型,string型]

class userProfileDatum{
    public int Number;
    public string name;
    public string address;
    public boolen distinction;//オカマは存在しないものとする。
    public string jobName;
};


上記のようなクラスを新たに生成した方が、設計上の仕様を固めやすいメリットがあり、
外部データを取得してきた時の管理クラスにも応用できるので非常に便利です。
質問にもあったnullでの判定もこちらの方が行い易いです。
それでは引き続き、プログラムの勉強頑張って下さい。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/03/25 13:51

    ありがとうございます。ご指摘ありがとうございます。これからも精進します。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.59%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • トップ
  • Javaに関する質問
  • javaで定義したメソッド(戻り値、引数含む)をdoGet内のメソッドにてメソッドを呼び出す処理をしたいがうまくいかないです。