C++ 素朴な疑問 using namespace std; は使わないほうが良い？

Question

C++で慣れないと冗長になるSTD::を省略するおまじないとして覚えたusing namespace std;ですが、
どこかで使わないほうが良いと見たことがありそれ以降意識してusing namespace std;を使わないようにしていました。
うろ覚えなので何故使わないほうが良いと思ったのかの理由も忘れてしまっているので再確認したいと思いました。

ネットで検索したコードなどではusing namespace std;を使っているケースが思ったよりも多いので迷っています。
個人的にはvectorの2次元配列などをよく使うので長ったらしいなとは思っていました。

使う環境や制限や個人で使ってる分には好みでどちらでも良いなどの回答になると思いますが後学の為に
「一般的にはどういう扱い」なのか、
「そんなの実務では使わないのが常識だよ」などの温度感が知りたいなと思いました。

※漠然とした質問で恐縮ですがご意見やアドバイスをお聞かせください。

using namespace std;あり

#include <bits/stdc++.h>
#define rep(i, m, n) for (int i = m; i < n; ++i)

using namespace std;

int main() {
  cin.tie(0);
  ios::sync_with_stdio(false);

  unsigned N, M;
  vector<vector<unsigned>> vec(N, vector<unsigned>(M, 0));
  cin >> N >> M;
  rep(i, 0, N) {
    rep(j, 0, N) { cin >> vec[i][j]; }
  }

  getchar();
  return 0;
}

using namespace std;なし

#include <bits/stdc++.h>
#define rep(i, m, n) for (int i = m; i < n; ++i)

int main() {
  std::cin.tie(0);
  std::ios::sync_with_stdio(false);

  unsigned N, M;
  std::vector<std::vector<unsigned>> vec(N, std::vector<unsigned>(M, 0));
  std::cin >> N >> M;
  rep(i, 0, N) {
    rep(j, 0, N) {
      std::cin >> vec[i][j];
    }
  }

  getchar();
  return 0;
}

Answer 1

「そんなの実務では使わないのが常識だよ」などの温度感が知りたいなと思いました。

趣味で開発しているならコーディングスタイルをどうするかは自由、複数人または組織に属して開発しているなら、そのコーディングスタイルに準拠するということになるとは思います。
とくに決められていないとしたら、実務では使わないほうがいいでしょうね。

---

どこかで使わないほうが良いと見たことがありそれ以降意識してusing namespace std;を使わないようにしていました。

個人的には goto と同じぐらい使わないほうがいい機能だと思います。
名前空間という機能で区別できるようになっている識別子を using ディレクティブを使ったことで同じ名前空間にインストールされ、名前が衝突したりする可能性が生じます。(例えば、begin という変数を定義していたとしたら、std::begin と衝突する)

using namespace std; の危険性と注意点・代替案【なぜ使わないほうがいいのか】 | MaryCore

例えば、Google のコーディングスタイルでも禁止しています。

Google C++ スタイルガイド(日本語全訳) Google C++ Style Guide (Japanese)

また、usingディレクティブ(using namespace foo)は使ってはいけません。 インライン名前空間も使ってはいけません。 無名名前空間については無名名前空間と静的変数を参照してください。

---

とはいえ、名前空間が入れ子になっていると長くなる場合があるので、使用するスコープ内で以下のようにすることはたまにやります。

名前のエイリアス作成

namespace fs = boost:filesystem;

特定の識別子だけ使えるようにする

using std::cout;

型のエイリアス作成

using SomeType = std::vector<std::pair<<std::string, int>>;

Comments

[cateye]

私も「特定の識別子だけ使えるようにする」ですかね？
ーーーーーーーーーーーーーー
using std::cerr;
using std::cout;
using std::endl;
こんな感じ・・・使うファイルの冒頭に書いています。

[tiitoi]

コメントありがとうございます。
確かに std::cout、std::endl はデバッグするときに記述頻度が多いので、面倒ですよね。

[Zuishin]

cout は count と間違いやすいので不可派も存在するようです。

https://qiita.com/elipmoc101/items/01003c82dbd2e464a071

C# は using 推奨で、むしろ using しないと使えない LINQ という便利なものもありますが、それで困ったことはありません。名前が衝突したクラスを使おうとした時には必ず警告が出て、あいまいさを無くすよう修正できるし、名前空間をルートから指定できる仕組みもありますから。
逆に言えば、そのような衝突を回避する仕組みを作ることができれば C++ も using を使っていいのではないかと思います。
もし、回避する仕組みがすでにあるのに使ってはいけないという主張が主流なのであれば、もしかしたらその考えは古いのかもしれません。

[mjk]

回答ありがとうございます。
>>実務では使わないほうがいい
>>個人的には goto と同じぐらい使わないほうがいい機能だと思います。
>>例えば、Google のコーディングスタイルでも禁止しています。

使わない方が良いことがよくわかりましたしリンク先もとても勉強になります。
他の方の回答にもあった使うものだけ局所的に使うことで冗長さの回避も出来そうです。

Answer 2

「使うな」というルールの下でコードを書いているのでないならば，使いたければ使えばよい．
必要なのは「使うと何が問題として起こり得るのか」を把握しておくこと．
起こり得ることを恐れる(?)ならば，質問文内の例のように広範囲に効かせるのではなく，範囲を絞って使うとよい．
（話を把握していれば，ヘッダファイル内に書くような暴挙は控えるようになるだろう．）

int main(void)
{
    {//この中では"std::"って書きたくない
        using namespace std;
        vector<int> V1;  //OK
    }

    vector<int> V2;  //NG
    return 0;
}

Comments

[fana]

個人的には， std:: は省略しないですね．
理由はよくわからんですが，"vector" とか "string" とかだけ書かれていると何かキモい．
単に "std::vector" とか "std::string" とかで慣れているからかな？

（自分で設けた名前空間を，自分で使う際に「長すぎてめんどくせぇよクソが」ってのはよくある）

[fana]

まぁよっぽど見難くない限り（※「書き難い」ではない）は略さないですね．

[mjk]

回答ありがとうございます。
>>"vector" とか "string" とかだけ書かれていると何かキモい．
確かに自分も慣れてきてたので他人様のコードを拝借させて頂く時にstd::付け直すことがあるのですが違和感がありますね。

>>範囲を絞って使うとよい．
これは知りませんでしたので参考にさせていただきます。

Answer 3

個人的にはvectorの2次元配列などをよく使うので長ったらしいなとは思っていました。

でしたら、using std::vector;だけ書いておく方法もあります。using namespace std;で何が含まれるかわからない以上、思わぬものを拾ってしまうリスクがあります。

---

そもそも論ですが、#include <bits/stdc++.h>は標準ヘッダではありませんので、このようなものを実用プログラムには書きません。

Comments

[mjk]

回答ありがとうございます。
>>そもそも論ですが、#include <bits/stdc++.h>は標準ヘッダではありませんので、このようなものを実用プログラムには書きません。
ですよね。AOJ提出するのに楽なのでこの1行で済ませてる悪い癖だと自負しております。

>>using std::vector;だけ書いておく方法もあります。
別の回答でもありましたが勉強になります。