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

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

ただいまの
回答率

88.09%

競技用プログラミング 大文字

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 628

score 57

AIZU

上記の問題は、大文字に変数を代入し任意の出力をするというものです。
これらのソースはターミナル上ではうまく動作するのですが、
提出時にRuntimeエラーを返されてしまいます。

while true
h,w = gets.chomp.split.map(&:upcase).map(&:to_i)

h.times do |t_1|
  w.times do |t_2|
    print "#"
  end
  puts ""
end

puts ""

break if h == 0 || w == 0
end
while true
 H,W = gets.chomp.split.map(&:to_i).map(&:freeze).freeze

h.times do |t_1|
  w.times do |t_2|
    print "#"
  end
  puts ""
end

puts ""

break if h == 0 || w == 0
end
$ ruby AOJ.rb
3 3
###
###
###

4 5
#####
#####
#####
#####

ご教授お願い致します

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • dice142

    2018/09/21 20:21

    肝心の処理をしているところを提示していただけないと何もわかりません。また、AOJは他の方の提出したソースコードを見ることができますが、そちらで何かヒントを得たりしてみましたか?

    キャンセル

  • y-doi

    2018/09/21 23:04

    ソースコードが2種類ありますが、提出したのはどちらでしょうか

    キャンセル

  • t.s_k

    2018/09/22 07:40

    どちらも提出したのですが、通りませんでした。

    キャンセル

回答 2

checkベストアンサー

+1

1つ目のソースコードについて

問題設定の理解に関して勘違いがあったのかと思います。
データセット(wとhが0でない)場合はその後ろに空行を入れる必要がありますが、
0 0の場合はそこで終了となりますので、その後ろに空行は入れてはだめです。
なので空行を出力するputsよりも先に、ループを抜けるbreakを書けばいいです。

while true
h,w = gets.chomp.split.map(&:upcase).map(&:to_i)

break if h == 0 || w == 0 # ここを変更

h.times do |t_1|
  w.times do |t_2|
    print "#"
  end
  puts ""
end

puts ""

end
 2つ目のソースコードについて

まずは1つ目と同じで、空行を出力するputsよりも先にbreakの判定をしないといけません。
さらにH Wに値をもたせたのに、h wを使おうとしているのがだめです。
(追記)すべてHWにしても、asmさんがおっしゃるように定数に再代入しているので警告もでます。


大文字は全く関係なく、upcasefreezeはやる意味がわからないので、全体的に無駄なことをしていると思います。
もっとシンプルにすると以下のようになりました。

loop do
  h, w = gets.split.map(&:to_i)
  break if h == 0 && w == 0
  h.times { puts '#' * w }
  puts
end


chompはしなくてもsplitすれば末尾の改行は取り除いてくれます。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/09/22 11:32 編集

    ありがとうございます。入力受付直下にbreakを移動させ、条件を&&に直したら通りました。
    根本的に00であった場合に即breakしないといけなかったのですが、
    大文字指定が規約なのかと勘違いしわざわざ upcase, freeze使ってました。

    キャンセル

+1

前者については無駄なupcaseの意味がわかりませんけど、通りそうな気がしますよ
ただし、終了条件を間違えている事には注意が必要です。
判定する箇所・判定する条件を見直しましょう。
後者はH,Wについてループのなかで複数回代入しているため定数の再代入で警告が出ます。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/09/22 11:10 編集

    ```
    while true
    h,w = gets.chomp.split.map(&:to_i)

    h.times do |t_1|
    w.times do |t_2|
    print "#"
    end
    puts
    end

    puts

    break if h == 0 && w == 0
    end
    ```
    簡潔にしたのですがこちらのソースでも通らなかったです。

    キャンセル

  • 2018/09/22 11:14 編集

    &&に直したのですが、質問文の方のソースを終了条件&&にしても、通らずでした

    キャンセル

  • 2018/09/22 11:17

    判定位置に問題があります。
    空行の出力より前にループを脱出しないと無駄な改行を挟んでしまいます。

    キャンセル

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

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

関連した質問

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