Deepanshu Sood Ответов: 3

Создайте программу для печати простых чисел до N

e.g. if N=9 prime numbers upto N are 2,3,5,7

Может ли кто-нибудь помочь мне определить, что не так с моим кодом

Что я уже пробовал:

#include<iostream>
    #include<cmath>
    using namespace std;
    int main()
    {
      short N,pc=0,num=2;
      bool prime;
      while(pc!=N)
      {
       prime=true;
        for(short i=2;i<=sqrt(N);i++)
        {
           if(N%i==0)
           {
               prime=false;
               break;
           }
        }
     }
      if(prime)
       {
          pc++;
          cout<<num;
       }
      return 0;}

3 Ответов

Рейтинг:
2

Patrice T

Первая проблема:

Цитата:
Создайте программу для печати простых чисел до N

В чем заключается ценность N в этом коде ?

Вторая проблема:
Научитесь правильно делать отступы в вашем коде, это покажет его структуру, и это поможет чтению и пониманию. Это также помогает выявлять структурные ошибки.
#include<iostream>
#include<cmath>
using namespace std;
int main()
{
  short N,pc=0,num=2;
  bool prime;
  while(pc!=N)
  {
    prime=true;
    for(short i=2;i<=sqrt(N);i++)
    {
      if(N%i==0)
      {
        prime=false;
        break;
      }
    }
  }  // this line is misplaced
  if(prime)
  {
    pc++;
    cout<<num;
  }
  // and should be here
  return 0;
}

Профессиональные редакторы программистов имеют эту функцию и другие, такие как сопоставление скобок и подсветка синтаксиса.
Блокнот++ Главная Страница[^]
личные[^]

Третья задача:
Ваш код всегда проверяет, является ли N простым или нет. Это неправильно.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш cpde, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Обратная сторона этого решения:
- Это DIY, вы один отслеживаете проблему и находите ее корни, которые ведут к решению.
Положительная сторона этого решения:
- Это также отличный инструмент обучения, потому что он показывает вам реальность, и вы можете увидеть, какие ожидания соответствуют реальности.

Вторичное воздействие
- Вы будете гордиться тем, что сами находите Жуков.
- Ваши навыки обучения улучшатся.

Вы должны довольно быстро найти, что не так.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]
1.11 — отладка программы (пошаговое выполнение и останова) | выучить C++[^]
Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.

Рейтинг:
1

Richard MacCutchan

Вы не задали переменной N никакого значения.

short N,pc=0,num=2;
bool prime;
while(pc!=N) // what is the value of N?
{

Рейтинг:
1

KarstenK

Вы должны сделать то же самое основное ввод-вывод в вашем коде, чтобы получить N.

А проверка неравенства-это очень сложная задача. плохая идея потому что чем N должно быть равно pc, иначе оно работает вечно. Я бы назвал это а недостаток конструкции чтобы не называть это а серьезная ошибка.

Вам лучше проверить с оператором less, но ваш код пока неясен.

Patrice T

Смотрите комментарии, которые я вставил в код в S2.