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

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

ただいまの
回答率

90.34%

  • C++

    3768questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

AtCoder:ABC80 D問題について

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 174

cinnamoroll

score 6

 前提・実現したいこと

AtcoderのABC80のD問題について,作成したプログラムがシステムテストを通らない(誤りを含む)理由がわかりません.
使用言語はC++です.

問題ページ

 方針

それぞれのチャンネルに対してmapを用いて番組の放送開始時間と終了時間をキーと値にとる.
すべての時刻で同時に録画器を動かす必要のある数を求め,その最大値が答えである.
ただし,同じチャンネルで連続した時間(ex.時刻1-3と時刻3-5)に番組がある場合,別々に計算すると放送前の待機時間0.5が重複してしまうため,1つの番組として扱う.

 発生している問題・エラーメッセージ

基本的な挙動は正しいが,テストセットの一部でWA(誤った解が出力された状態)になる.

 該当のソースコード

#include<iostream>
#include <stdio.h>
#include<string>
#include <cmath>
#include <algorithm>
#include <vector>
#include <cstdint>
#include <queue> 
#include <map>
#include <set>
#define MOD 1000000007
#define FOR(i,a,b) for(int i=(a);i<(b);i++)
#define REP(i,n)  FOR(i,0,n)
#define ITR(itr,mp) for(auto itr = (mp).begin(); itr != (mp).end(); ++itr)
#define dump(x)  cout << #x << " = " << (x) << endl;
#define debug(x) cout << #x << " = " << (x) << " (L" << __LINE__ << ")" << " " << __FILE__ << endl;
typedef long long ll;
using namespace std;

int main(){

  int n,c;
  cin >> n >> c;

  map <int,int>channel[c+1];
  REP(i,n){
    int s,t,C;
    cin >> s >> t >> C;
    channel[C].insert({s,t});
  }


  REP(i,c){
    int key = 0,t = 0;
    ITR(itr,channel[i+1]){
      if(itr->first == t){   // 連続した時間の番組があれば,1つにまとめる.
        channel[i+1][key] = itr->second;
        channel[i+1].erase(itr);
      }
      key = itr->first;
      t = itr->second;
    }
  }

  const int NMAX = 100010;
  int s[NMAX] = {};

  REP(i,c){
    ITR(itr,channel[i+1]){
      s[itr->first-1]++; //待機時間があるため,録画器は番組開始の1手前の時刻から使用
      s[itr->second]--;
    }
  }

/*    REP(i,c){
      cout << i+1 << endl;
    ITR(itr,channel[i+1]){
 cout << itr->first << " " << itr->second << endl;
    }
    }
  */

  int ans = 0;

  REP(i,NMAX-1){
    s[i+1] += s[i];
    ans = max(ans,s[i+1]);
  }

  cout << ans << endl;

 return 0;
}
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

元のコードでは連続している番組をまとめた際にkeyの設定を間違えていると思います。3つ連続した番組があると3つ目の番組の終了時間が期待とは異なるエントリーに記録されてしまうと思います。そのため折角削除したエントリーがまた復活してしまいますよね・・・


また、自分の環境はAtcoderの環境とは違うのですが、(cygwin, g++ (GCC) 6.4.0, g++ -std=gnu++1y -O2でコンパイル)元のコードだと連続番組を統合する論理で、itrがおかしな要素を指し示したり無限ループに陥いる現象が起きました。

C++の標準ライブラリーに暗いのでここを見た上で感じた(ほとんど推測)で修正してみてますが、以下のコードに直したところ少なくとも無限ループはしなくなり番組の統合も期待通りに行われているように見えました。

問題ではないかと感じたのは「eraseしたiteratorに対してfirstやsecondにアクセスしたり++演算子で次の要素に正しく位置付けられることを前提としている」点です。それをしないようにしたのが下記のコードです。

for(auto i = 1; i <= c; i++) {
  int key = 0, t = 0;
  map<int, int> &ch = channel[i];
  for (auto itr = ch.begin(); itr != ch.end(); ) {
    if (itr->first == t) {
      t = ch[key] = itr->second;
      itr = ch.erase(itr);
    } else {
      key = itr->first;
      t = itr->second;
      ++itr;
    }
  }
}


(すみませんが、関係ない部分も好みに合わせて変更しました。改悪になってなければよいのですが・・・)

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/30 22:46

    イテレータの扱いに不慣れで文法に怪しさを感じてはいましたが,3つの連続した番組の連結をテストしたとき表面上では問題なさそうだったので見逃していました...
    回答ありがとうございました!

    キャンセル

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

  • C++

    3768questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。