Printfの変な文字

Question

なぜ機能しないのですか？プログラムの最後には、 "e primo"や "nao e primo"の代わりに2つの奇妙な文字が表示されます。あなたが私を助けることができれば、私は感謝しています。

#include <stdio.h> 
#include <stdlib.h> 

int main() { 
    // var 
    int n, c = 2; 
    char p[11]; 
    // code 
    printf("Informe um numero para checar se e primo: "); 
    scanf("%d", &n); 
    do { 
     if (n % c == 0) { 
      p[11] = 'e primo'; 
      break; 
     } else { 
      p[11] = 'nao e primo'; 
     } 
     c = c + 1; 
    } while (c != n/2); 
    printf("\nO numero %s", p); 
    return 0; 
} 

Answer 1

あなたのプログラムはいくつかの問題があります：あなたは単純な代入p[11] = 'e primo';で文字列をコピーすることはできません

• 。バッファを大きくするか、単に文字列ポインタ​​を使用すると、strcpy()を文字列に使用できます。

• nが4未満の場合、ループは永久に実行されます。より正確には、c = c + 1;が算術オーバーフローを起こすと、未定義の動作を引き起こします。

• 結果は、他の値と正反対です。

• 大きな素数のループは非常に遅いので、c * c > nのときは停止する必要があります。

ここ

が修正バージョンです：これはコンパイルされません

#include <stdio.h> 
#include <stdlib.h> 

int main(void) { 
    // var 
    int n, c; 
    const char *p = "e primo"; 
    // code 
    printf("Informe um numero para checar se e primo: "); 
    if (scanf("%d", &n) == 1) { 
     for (c = 2; c * c <= n; c = c + 1) { 
      if (n % c == 0) { 
       p = "nao e primo"; 
       break; 
      } 
     } 
     printf("O numero %s\n", p); 
     return 0; 
    } 

Answer 2

配列には12番目の要素はありません。それらはp .Your割り当て（p[11] = 'e primo';）でのみ11の要素は、このようundefined behaviourにおける結果です。

'e primo'は実装定義の動作とリテラルマルチバイト文字です。コピーするにはstrcpy()を使用します。

strcpy(p, "e primo"); 

（コメント内の他の文字列コピーに長い文字列を格納するために、配列サイズを大きくします）。

あるいは、ヨーヨーは本当に配列を必要としないリテラルとしてあなたは、単に文字列を指すポインタを使用することができます。

char *p = "nao e primo"; 
printf("Informe um numero para checar se e primo: "); 
scanf("%d", &n); 
do { 
    if (n % c == 0) { 
     p = "e primo"; 
     break; 
    } else { 
     p = "nao e primo"; 
    ... 

printf("\nO numero %s", p); 

関連：Single quotes vs. double quotes in C or C++

Answer 3

まず、あなたは

p[11] = 'e primo'; 

pは文字の配列であり、p [11]を使用すると、11位に文字を設定または取得することができると言うことはできません。第2に、インデックス11の文字はありません。Cでは、インデックスは0から始まるので、p [0]、p [1]、...、p [10]の文字を取り出すことができます。 11要素。

あなたは警告を読みましたか？

1.c: In function ‘main’: 
1.c:16:21: warning: character constant too long for its type 
      p[11] = 'e primo'; 
        ^
1.c:16:21: warning: overflow in implicit constant conversion [-Woverflow] 
1.c:21:21: warning: character constant too long for its type 
      p[11] = 'nao e primo'; 
        ^
1.c:21:21: warning: overflow in implicit constant conversion [-Woverflow] 

それは実際にcharacter constant too long for its typeことを言います。

あなたが言うことができるもの：

p[10] = 'e' 

その後、％sのあなたは "文字列" を印刷：確定0で決定文字の配列。 ので、表示されている必要があり、最後の文字の後に、あなたが例えばを言わなければならない。： P [10] =「\ 0」。

私は、コードの作業になるだろうが、私は実際にポイントが何であるかわかりません。それはいつも最後の文字を何かに割り当てているようです。