配列型のクラス変数をクラス内のメソッド内で書き換えようとするとエラーがでます。

Question

上のコードのように、クラス変数を宣言し、下のコードのように、クラス内のfavoriteというメソッドで書き換えを行おうとしたのですが、
usernameにエラーは出ませんが、spinnerlitems[]=には、Unexpectedtokenや、';'expected といったエラーがでます。
なぜ、配列だからエラーが出るのでしょうか？
アドバイスをいただけないでしょうか？

String username="ta";
  String spinnerItems[]= {"GPSの現在地","新規登録/ログイン","表示画面を登録"};
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
}

//解決したコード

//クラス変数
String  spinnerItems[];
//favoriteメソッド
public void favorite() {


    MyOpenHelper helper = new MyOpenHelper(LocationActivity.this);
    SQLiteDatabase db = helper.getReadableDatabase();

    String sql = "select placename from favorite where username = '" + username + "';";

    Cursor c = db.rawQuery(sql, null);
  int count = c.getCount();

    spinnerItems = new String[count + 4];
    spinnerItems[0] = "GPSの現在地";
    spinnerItems[1] = "新規登録/ログイン";
    spinnerItems[2] = "googlemapで登録";
    spinnerItems[3] = "表示画面を登録";

    boolean mov = c.moveToFirst();

    int mam=0;
    while (mov) {
       String in=c.getString(0);
          spinnerItems[mam + 4] =  in ;
         mov = c.moveToNext();
        mam=mam+1;
           }
    c.close();
        db.close();
    }

Comments

[swordone]

まずfavoriteメソッドはどこよ

[edoooooo]

申し訳ありませんでした。

[swordone]

あの、質問中にこんな大幅なコード変更やめてください。こっちが混乱します。

[edoooooo]

最初から記述すべきでした。すみませんでした。 今回は元あったコードと別のとこに記載するように変更します。

[swordone]

あの、何やってるんですか？細切れな修正はやめてください。

[edoooooo]

解決済みにしたほうがよかったでしょうか？

[swordone]

追加部分(8回目の修正「説明」)が何を説明したものなのか、そもそも何をしようとしているのか、全く分かりません。

[edoooooo]

配列は一度初期化すると、要素数を増やすことができないことへの対応としてやっていることを記入しました。

Answer 1

spinnerItems[]= {"GPSの現在地", "新規登録/ログイン","googlemapで検索","表示画面を登録" + str};

↓

// []不要。[]があるせいで変数の宣言か何かとみなされたのでは
// 追加。変数宣言時以外でこの書き方をするとき、新たに型名を伴っての配列宣言が必要
spinnerItems = new String[]{"GPSの現在地", "新規登録/ログイン","googlemapで検索","表示画面を登録" + str};

Comments

[edoooooo]

ありがとうございます。[]をとると、{""+str};にエラーが出てしまいました。
要素ごとに代入することが必要とアドバイスをいただいたので、修正してみます。
ありがとうございました。

[edoooooo]

最初に宣言している配列の要素が3つ、つまり配列の長さが3で生成されているんです。
そこに、入れなおそうとしている要素は4つあるので、4つ目を入れることができません。
結局配列の再生成をすることになります。
⇩
ありがとうございます。配列を使う場所で、初期化ができればいいということですね。

[edoooooo]

解決しました。コードは記載しました。ありがとうございました。

Answer 2

Unexpectedtokenの意味ってわかります？

Comments

[edoooooo]

要するに意外なしるしがあります。という意味と考えております。

[toutou]

だったらそうなんじゃないんですか？なにが疑問なの？

[edoooooo]

余計なしるしと言われているものをとっても、違うエラーがでることです。

[swordone]

コンパイラは万能ではありません。
とりあえず構文的に間違っているところを指摘しているだけなので、
全体をどう修正すればいいのかまでは完全にはカバーできません。

[toutou]

その際に配列の書き方は調べましたか？

[edoooooo]

調べました。そして実際に、favorite()のなかで、書き換えを行うまでは、エラーなく動作していました。

[toutou]

書き換えを行うまでというのでがその書き換え方法が間違ってるとは思いませんでしたか？

[edoooooo]

おもいました。

[toutou]

その際何かしましたか？

[edoooooo]

エラーを調べたり、文法的な間違いを探したり、Logで出力して、変数の中に問題がないかなどを確かめました。

[toutou]

その結果問題はなかったですか？

[edoooooo]

自分で調べた中では、解決策が見つからなかったので、質問をさせていただきました。
そこで、配列の初期化してる値の要素数と、代入しようとしている値の要素数が違うため、
要素を余分に確保しておく必要があるということを教わりました。
そこで、それを今回の件に当てはめて考えたところ、spinnerに使うので、から文字のボタンができると困るため、今は、文字列でクラス変数に渡して、そこから初期化をする配列に直接文字列を埋め込めないかと考えています。
自分の調べる量は不足していたかもしれません。すみませんでした。失礼します。

[toutou]

毎回しらべかたが足りないとか言われる感じなのでそこら辺頑張って欲しいですね。注意力と根気が足らないです。

[edoooooo]

最近は結構調べてはいるのですが、きおつけます。

Answer 3

初期化時の配列の一斉代入方法は、初期化時以外では書き方が違います。

または、以下のように要素ごとに代入することが必要です。
spinnerItems[0] = なんとか
spinnerItems[1] = なんとか
spinnerItems[2] = なんとか

Comments

[edoooooo]

ありがとうございます。
すぐ取り掛かります。

[swordone]

要素数が1個多いので無理だと思います。

[edoooooo]

一個多いとは strのことでしょうか？
str の説明も記述いたしました。

[swordone]

最初に宣言している配列の要素が3つ、つまり配列の長さが3で生成されているんです。
そこに、入れなおそうとしている要素は4つあるので、4つ目を入れることができません。
結局配列の再生成をすることになります。

今更だけど、自分の回答のコメントに書くべきだったなこれ

[HogeAnimalLover]

え？ひょっとして、spinnerItems[2]の中の文字列を追加するのではなく、spinnerItems[3]なる新たなる文字列を作ろうとしています？だとするともう少し修正が必要です。予め、多めに配列要素をキープしておく（空文字列で埋めるなりする）とか、要素数の多い配列に乗せ換えることが必要です。

[edoooooo]

ありがとうございます。
多めに配列を作る必要があったのですね。
今回作った配列を使って、動的なspinnerにセットしようと考えておりました。
から文字を使ってしまうと、から文字の項目がspinnerにできてしまうと考えています。
そこで、一度文字列に変更してクラス変数に代入してから、それを配列の初期化の場所で、配列に戻して、初期化をするという方法ができないか考えてみます。
ありがとうございました。

[edoooooo]

解決しました。コードは記載しました。ありがとうございました。

Answer 4

セミコロンが全角でコンパイルエラーでも起こしているのではないですか？

Comments

[edoooooo]

回答をいただきましてありがとうございます。
申し訳ありません。
質問文を作成する際に、うち間違えをしてしまい、　”となってしまっていました。
実際のコードは、 String spinnerItems[]= {"GPSの現在地","新規登録/ログイン","表示画面を登録"};
となっていました。申し訳ありませんでした。

Answer 5

このコード、ダブルクォーテーションが足りていないのではないでしょうか？
関係はありますか？

  String spinnerItems[]= {"GPSの現在地","新規登録/ログイン","表示画面を登録”};

↓修正

  String spinnerItems[]= {"GPSの現在地","新規登録/ログイン","表示画面を登録"};

Comments

[edoooooo]

回答をいただきましてありがとうございます。
申し訳ありません。
質問文を作成する際に、うち間違えをしてしまい、　”となってしまっていました。
実際のコードは、 String spinnerItems[]= {"GPSの現在地","新規登録/ログイン","表示画面を登録"};
となっていました。申し訳ありませんでした。

[swordone]

コピペしてください。
こんな質問の本質と関係ないところでいつまで時間をとっているつもりですか。

[edoooooo]

一度、macのメモにコピペして、まとめてから、てらているにコピペしたのですが、
一部が勝手に自動変換が行われていました。きおつけます。