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

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

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

if

if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

Q&A

解決済

1回答

407閲覧

素数洞穴で少しのケースだけ答えが合わない

grape_ll

総合スコア83

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

if

if文とは様々なプログラミング言語で使用される制御構文の一種であり、条件によって処理の流れを制御します。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

コードレビュー

コードレビューは、ソフトウェア開発の一工程で、 ソースコードの検査を行い、開発工程で見過ごされた誤りを検出する事で、 ソフトウェア品質を高めるためのものです。

0グッド

1クリップ

投稿2020/07/06 12:36

質問内容

先日動かないということで質問させていただいたコードを改良して素数洞穴の問題を解いていたのですが,多くのケースで答えが一致するものと,この少数のケースに対応させたが多くのケースで間違ってしまうコードになってしまい,両方を満たすコードが出来ないので,どのように工夫すればよいのか教えていただきたいです.

素数洞穴

多くのケースが正解となるコード

C

1#include<stdio.h> 2#include<stdlib.h> 3#include<math.h> 4#include<string.h> 5 6#define right 0 7#define down 1 8#define left 2 9#define up 3 10 11int box[1000001]={1,1}; 12int value[1001][1001]; 13int map[1001][1001]; 14 15void Prime(){ 16 int i,j; 17 18 for (i=2;i<1000001;i++){ 19 if (box[i]==0){ 20 for (j=i+i;j<1000001;j+=i){ 21 box[j]=1; 22 } 23 } 24 } 25} 26 27int Max(int a,int b){ 28 if(a>b) return a; 29 else return b; 30} 31 32typedef struct Cave{ 33 int x,y,scripture; 34}Cave; 35 36 37int main(void){ 38 int m,n; 39 int max=0; 40 int i=0,j=0,k,l; 41 int lim=1000,direction=0,count=0; 42 43 //printf("c\n"); 44 45 Prime(); 46 47 Cave *cave; 48 49 cave=(Cave *)malloc(sizeof(Cave)*1000001); 50 51 //printf("a\n"); 52 53 k=1000000; 54 while(k>0){ 55 if(box[k]==0) cave[k].scripture=1; 56 else cave[k].scripture=0; 57 58 if(direction==right){ 59 map[i][j]=k; 60 cave[k].y=i; 61 cave[k].x=j; 62 j++; 63 count++; 64 if(count==lim-1){ 65 direction=down; 66 count=0; 67 } 68 } 69 else if(direction==down){ 70 map[i][j]=k; 71 cave[k].y=i; 72 cave[k].x=j; 73 i++; 74 count++; 75 if(count==lim-1){ 76 direction=left; 77 count=0; 78 } 79 } 80 else if(direction==left){ 81 map[i][j]=k; 82 cave[k].y=i; 83 cave[k].x=j; 84 j--; 85 count++; 86 if(count==lim-1){ 87 direction=up; 88 count=0; 89 } 90 } 91 else if(direction==up){ 92 map[i][j]=k; 93 cave[k].y=i; 94 cave[k].x=j; 95 count++; 96 if(count==lim-1){ 97 direction=right; 98 count=0; 99 lim-=2; 100 j++; 101 k--; 102 continue; 103 } 104 i--; 105 } 106 k--; 107 } 108 109 110 //printf("b\n"); 111 112 113 while(1){ 114 scanf("%d %d",&m,&n); 115 if(m==0 && n==0) break; 116 117 118 memset(value,0,sizeof(value)); 119 int l,r,Max_value=0,Max_prime=0; 120 int k; 121 122 direction=right; 123 124 l=cave[n].x; 125 r=cave[n].x; 126 127 max=cave[n].y; 128 for(k=1;k<=m;k++){ 129 max=Max(max,cave[k].y); 130 } 131 for(i=cave[n].y;i<=max;++i){ 132 for(j=l;j<=r;++j){ 133 if(map[i][j]>m){ 134 direction=down; 135 break; 136 } 137 if(i-1>=0 && j-1>=0){ 138 value[i][j]=Max(value[i][j],value[i-1][j-1]); 139 } 140 if(i-1>=0 && j+1<=999){ 141 value[i][j]=Max(value[i][j],value[i-1][j+1]); 142 } 143 if(i-1>=0){ 144 value[i][j]=Max(value[i][j],value[i-1][j]); 145 } 146 if(cave[map[i][j]].scripture==1){ 147 value[i][j]++; 148 if(value[i][j]>Max_value){ 149 Max_value=value[i][j]; 150 Max_prime=map[i][j]; 151 } 152 /* 153 else if(value[i][j]=Max_value){ 154 if(map[i][j]>Max_prime){ 155 Max_prime=map[i][j]; 156 } 157 } 158 */ 159 160 //printf("val:%d pri:%d\n",Max_value,Max_prime); 161 162 } 163 } 164 if(direction) break; 165 if(l-1>0 && map[i+1][l-1]<=m) l--; 166 else if(l-1>0 && map[i+1][l]>m) l++; 167 168 if(r+1<999 && map[i+1][r+1]<=m) r++; 169 } 170 printf("%d %d\n",Max_value,Max_prime); 171 } 172 free(cave); 173 return 0; 174 175}

######入力と実行結果
・入力
49 22
46 37
42 23
945 561
1081 681
1056 452
1042 862
973 677
1000000 1000000
0 0

・結果
0 0
6 23   ここが間違い
1 23
20 829
18 947
10 947
13 947
23 947
534 993541

間違いに適応させたコード

上記のコードのelse ifのコメントアウトを外したもの

######入力と実行結果
・入力
49 22
46 37
42 23
945 561
1081 681
1056 452
1042 862
973 677
1000000 1000000
0 0

・結果
0 0
6 43  合ってる
1 23
21 941
18 1069
10 953
14 953
23 953
878 997991

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

単なるイージーミスです。

diff

1- else if(value[i][j]=Max_value){ 2+ else if(value[i][j]==Max_value){

投稿2020/07/09 12:57

hope_mucci

総合スコア4447

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

grape_ll

2020/07/09 14:41

答えと一致しました.ありがとうございました
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問