My submission:- https://pastebin.com/aL9vCYWu
Please anyone tell what is wrong in this code
# | User | Rating |
---|---|---|
1 | tourist | 4009 |
2 | jiangly | 3773 |
3 | Radewoosh | 3646 |
4 | ecnerwala | 3624 |
5 | jqdai0815 | 3620 |
5 | Benq | 3620 |
7 | orzdevinwang | 3612 |
8 | Geothermal | 3569 |
8 | cnnfls_csy | 3569 |
10 | gyh20 | 3447 |
# | User | Contrib. |
---|---|---|
1 | cry | 161 |
2 | maomao90 | 160 |
3 | awoo | 159 |
4 | atcoder_official | 157 |
5 | -is-this-fft- | 155 |
5 | nor | 155 |
5 | adamant | 155 |
8 | Um_nik | 152 |
8 | maroonrk | 152 |
10 | djm03178 | 146 |
My submission:- https://pastebin.com/aL9vCYWu
Please anyone tell what is wrong in this code
Name |
---|
you can solve it with a one dimension dp array like this
first set the whole array to -inf execpt dp[0] = 0
for(int i = 2; i < 10001;++i)
for(int x : v) // v is the vector that hold prime numbers and primatic numbers
{
if(i — x < 0)continue;
dp[i] = min(dp[i], dp[i — x] + 1);
}
answer = dp[input]
With
N = 9973
(the largest prime below 10 000), your code prints0
.You aren't using last prime number
v[m - 1]
in your dp loop, because you are going from1..m-1 (for i)
, but then you are usingv[i - 1]
in calculation, so in reality you are using0..m-2
prime numbers.Second problem is array overflow, because in your
for(;l<v.size();l++)
for number like 10000 you will end up withl == m
, which doesn't with yourdp
declaration.To fix it change
int dp[m][10001]
toint dp[m+1][10001]
and also your dp loop fromfor(int i=0;i<m;i++)
tofor(int i=0;i<=m;i++)
. That being said, you don't need 2 dimensional array, as pointed out by Mohamed_Saad62