文字列を関数に直接渡す

Question

文字列を関数に直接渡すのは悪い習慣ですか？

メモリにエラーが発生しますか？

func(char *x) { 
    //do stuff 
} 

func("String"); 

これを行う方が安全でしょうか？

char *s = "string"; 
func(s); 

Answer 1

それはメモリエラーを引き起こすのでしょうか？

この文字列を変更しようとする可能性があります。

あなたの関数

func(const char *x) { 

を宣言しかし、何も悪いが、関数に文字列リテラルを渡すには存在しない安全であると。そうでなければ、99.99のプログラムがひどく書かれています。

これを行う方が安全でしょうか？

char *s = "string"; 
func(s); 

あなたは同じ文字列リテラル

へのポインタを渡すように、それは何も変わりません。しかし、あなたがRWを割り当てるよう以下のコードは安全です（読み取り/書き込み）文字列に対応するためのメモリ、およびこの割り当てられたスペースにリテラルシステムコピー

Answer 2

char s[] = "string"; 
func(s); 

func(char *x) { 
    //do stuff 
} 

この関数は、入力として文字列にポインタをとります。インターフェイスは、の書き込みをxから書き込むことができます。

しかしリテラル（あなたの例のどちらの場合も）文字列を使用しているので、元gccは仕方のコード/データを整理し、この文字列リテラル（およびほとんどのコンパイラのメモリを変更することは違法です

char s[] = "string"; 
func(s); 

場合や： - - もちろんまたは読み取り）SEGV）

「安全」として長く書くときは、文字列の範囲外に行かないような方法（：それはそれをやってからあなたを防ぎますの内容を変更するつもりはありません安全性に差はありません

func(const char *x)