Rubyの超初心者です。配列を用意して、配列の一番最後の値より大きければ、入力した値を追加していく、という処理を練習で作ってみようと思い、以下の様にしていますが、うまく言っていません。
array.pushなどで配列に追加できるということは学習済みです。この辺はJavascriptと似ていますね。
ただし、length-1などと言った表現があるのか、ないのかを検索してもあまりわからず、結果どこがおかしいのか少し困惑しています。JSとごっちゃになっているところが多かったら申し訳ありませんが、アドバイスいただけると助かります。
puts "番号を入力してください。配列の1番大きな値より大きければ配列に入ります" input = gets.chomp.to_i def sample(input) array = ["1, 2, 3, 4, 5, 6, 7, 8, 9, 10"] if input > array.length - 1 then array.push(input) end end puts sample(input)
気になる質問をクリップする
クリップした質問は、後からいつでもMYページで確認できます。
またクリップした質問に回答があった際、通知やメールを受け取ることができます。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
回答4件
0
Rubyには負のインデックスがあるので
ruby
1puts "番号を入力してください。配列の1番大きな値より大きければ配列に入ります" 2input = gets.chomp.to_i 3 4def sample(input) 5 array = [1, 2, 3, 4, 5, 6, 7, 8, 9, 10] 6 # 配列の末尾よりも大きければ 7 if input > array[-1] then 8 array.push(input) 9 end 10 # arrayを評価しないと返り値にならない 11 array 12end 13 14puts sample(input)
なおArray#maxを用いて
ruby
1array.push(input) if input > array.max
の方がわかりやすいでしょう。
投稿2019/05/16 10:19
総合スコア15147
0
「わたしだったらこう書くかもしれない」、というのを書いてみました
ruby
1# グローバル変数として配列を定義 2# TODO: class を定義してインスタンス変数に換えるなどスコープを短くする 3$array = [*1..10] # Range を「*」 (Splat Operator) で展開して配列の値として利用 4 5def push_to_array(input) 6 # 三項演算子で処理を分岐 7 input > $array.max ? $array.push(input) : $array 8end 9 10# 渡された文字列がすべて数字であるか判定する 11def numeric?(string) 12 return true if string =~ /\A\d+\z/ 13 14 false 15end 16 17def ask 18 puts "\n番号を入力してください。" 19 puts '既存の配列の1番大きな値より大きければ、指定した値が配列の末尾に追加されます。' 20 print '値の指定 ([Enter] で確定): ' 21 input = gets.chomp # 一旦 String として受け取る 22 23 # 数値として扱えるか判定して、ダメなら例外を発生させる 24 raise RuntimeError unless numeric?(input) 25 26 input.to_i 27 28rescue RuntimeError 29 # 数字以外が入力されたので 望む動作をユーザに伝える 30 31 puts '' 32 puts '*' * 10 # 「*」を10個出力する 33 puts 'エラー: 数値を入力してください' 34 puts '*' * 10 35 retry 36end 37 38 39# 無限ループで何度でも実行できるようにする 40loop do 41 input = ask 42 p push_to_array(input) 43 # => [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11] 44end
誰かのコードを読んだり、自分で書いたりしているうちに、ほかにも、「ここは こうできるんじゃないか」「ここは こうしたほうがいいんじゃないか」といった部分が思い浮かんでくるかもしれません
なにか参考になれば幸いです
Links
投稿2019/05/16 18:15
編集2019/05/16 19:07総合スコア2321
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2019/05/16 19:04 編集
0
https://www.ruby.or.jp/ja/tech/development/ruby/tutorial/
こういうチュートリアルなどを確認しておくといいかもしれません。
JSを既に理解しているとして、JSでも ["1, 2, 3, 4, 5, 6, 7, 8, 9, 10"]
こんな書き方をしたら文字列一つだけが入った配列になるのは推測できますよね?
数値は 1
のように書けば良くてダブルクォーテーションは必要ないので、数値の配列は [1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
となります。
配列の一番最後の値より大きければ
とありますが、 array.length - 1
は「配列の一番最後の値」でしょうか?
試しに array = [3,2,1,0]
で確認してみてください。
投稿2019/05/16 09:40
総合スコア6753
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2019/05/16 09:43
2019/05/16 09:51
2019/05/16 09:51
2019/05/16 09:53
2019/05/16 09:57
2019/05/16 10:20
0
ベストアンサー
コードは画像ではなくコピペし、マークダウンのコードブロックをお使いください。
以下の様にしていますが、うまく言っていません。
どういう動作を期待してどういう結果になっているのかを書くと回答を得やすいです。
length-1などと言った表現があるのか、ないのかを検索してもあまりわからず
「length-1」がセットではなく、「array_name.length」がセットです。
-1はただの計算なので、そこはしっかり理解しましょう。
また、検索して出ない場合はどんなワードで検索したのか、どんなページを見たのかわかるとアドバイスを貰えるかもしれません。
前置きを終えてコードの問題点に入ります。
・gets.chomp.to_i
はちょっと無駄でgets.to_i
で良いです。
・sampleという関数が引数を必要としているのに、呼び出し(puts sample
)で渡す値を指定していません。
指摘漏れがあったので追記
・配列の初期化方法がおかしいです。
・配列の最後の値以下の値を入力したとき、if文の中が評価されないため、nilが返ってきてしまいます。
投稿2019/05/16 09:08
編集2019/05/16 09:32総合スコア5158
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。
2019/05/16 09:14
2019/05/16 09:34
2019/05/16 09:42
2019/05/16 09:46
2019/05/16 09:49
あなたの回答
tips
太字
斜体
打ち消し線
見出し
引用テキストの挿入
コードの挿入
リンクの挿入
リストの挿入
番号リストの挿入
表の挿入
水平線の挿入
プレビュー
質問の解決につながる回答をしましょう。 サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。 また、読む側のことを考えた、分かりやすい文章を心がけましょう。
バッドをするには、ログインかつ
こちらの条件を満たす必要があります。