Помогите нет ошибок но прогрмма не работает вот пример

[ergocom]

#include <iostream>
using namespace std;
int main()
{
   
   int number ;
   int number2;
   
   
   
   
   for(number=2; number<17; number++){

for(number2=2; number2<number; number2++)
{

   if((number%number2)!=0) continue;

   else
      number=true;
      
      break;
}
   }
   if(!number)

   {
   
   
      
         
      
         
         
         
      cout<<number;
}
   else

number=false;         
      
   return 0;
}


вроде моя програмка должна выводить простые числа тока она не работает, при проверки ошибок не нанаходит их!
что делать

[Антон (LogRus)]

ergocom, нет ничего удевительного в том, что код не работает
1. не кместу используете break/continue
2. присваеваете переменной int значение bool, вы уверены в том чему будет равна переменная после приваивания?
3. форматирование кода отдельный разговор
4. прочем некоторая логика есть
5. рекомендуется также использовать префиксный оператор ++ если вы не собираетесь использовать старое значение объекта в точке вызова оператора, постфиксный оператор ++ создаёт лишний временный объект.

я код немного поправил и вот что получилось

Код:

#include "stdafx.h"
#include <string>
#include <iostream>
#include <sstream>

using namespace std;

int _tmain(int argc, _TCHAR* argv[])
{
	for(int number=2; number<17; ++number)
	{
		int number2;
		for(number2=2; number2<number; ++number2)
			if((number%number2)==0)
				break;
		if (number2 == number)
			cout<<number<<"\n";
	}

	return 0;
}

[ergocom]

Код:

#include <iostream>
// выводит простые числа с 1 -19 !
using namespace std;



int main()

{
	

	for(int i=1 ; i<21 ; i+=2)
cout<< i;
	

  return 0;
}

Сам наконец придумал как проще написать!

[Антон (LogRus)]

Код:

#include <iostream>
// выводит простые числа с 1 -19 !
using namespace std;



int main()

{
	

	for(int i=1 ; i<21 ; i+=2)
cout<< i;
	

  return 0;
}

Сам наконец придумал как проще написать!

ну-ну
если так проще то конечно,
вообще это не правильное решенье, попробуй использовать свою прогрммаму для поиска прочтых чисел <100 и посмотри как много там будет не простых чисел

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

[ergocom]

я тока учусь как лучше применить цыкл!
А так несомнено это не лучший вариант!

[Антон (LogRus)]

учись

вот тебе пример был очередной цикл в цикле

[Артем]

Код:

#include <iostream>
// выводит простые числа с 1 -19 !
using namespace std;



int main()

{
	

	for(int i=1 ; i<21 ; i+=2)
cout<< i;
	

  return 0;
}

Сам наконец придумал как проще написать!

хм, а разве 9 и 15 простые числа?

[Джон]

хм, а разве 9 и 15 простые числа?

ergocom хотел сказать НЕЧЁТНЫЕ! )

[ergocom]

ага цыкл не правильно работает!
на др форуме дали пример вместе с классом и обььектов!
ищу краткое решение этой задачи!

[Антон (LogRus)]

ergocom, какой задачи?
про простые числа или про нечетные числа?
с натуральными числами кратное решенье уже есть выше на несколько постов, переменные названны длинно но так оно вполне краткое

[Артем]

, а если про нечетные--твое решение правильное

[ergocom]

Вот как мона ище проще сделать

Код:

#include <iostream> 
using namespace std; 

int main() {  
_wsetlocale(LC_ALL, L"Russian");
  int i, j; 
  bool prime; 

  for(i=1; i < 100; i++) { 
    prime = true;  

    
    for(j=2; j <= i/2; j++) 
     
      if((i%j) == 0) prime = false; 

    if(prime) 
      cout << i << "  Простые числа с 1-100\n"; 
  } 

  return 0; 
}

[Dimka]

for(j=2; j <= i/2; j++)

А ещё лучше не до i/2, а до sqrt(i).

[Антон (LogRus)]

Вот как мона ище проще сделать

Код:

#include <iostream> 
using namespace std; 

int main() {  
_wsetlocale(LC_ALL, L"Russian");
  int i, j; 
  bool prime; 

  for(i=1; i < 100; i++) { 
    prime = true;  

    
    for(j=2; j <= i/2; j++) 
     
      if((i%j) == 0) prime = false; 

    if(prime) 
      cout << i << "  Простые числа с 1-100\n"; 
  } 

  return 0; 
}

что же вы так код не читабельно форматируете
замена этого

Код:

    for(j=2; j <= i/2; j++) 
     
      if((i%j) == 0) prime = false; 

на вот это

Код:

    for(j=2; j <= i/2; j++) 
        if((i%j) == 0)
        {
               prime = false; 
               break;
      }

ускорит выполнение программы

[Антон (LogRus)]

а вот так еще быстрей

Код:

int _tmain(int argc, _TCHAR* argv[])
{
	const int max_num = 100;
	vector<int> simples;
	simples.push_back(2);
	for(int i=simples.front(); i<max_num; ++i)
	{
		bool simple = true;
		for(vector<int>::iterator it = simples.begin(),it_end = simples.end(); it != it_end; ++it)
			if((i%*it)==0)
			{
				simple = false;
				break;
			}

		if (simple)
		{
			simples.push_back(i);
			cout<<i<<"\n";
		}
	}

	for(vector<int>::iterator it = simples.begin(),it_end = simples.end(); it != it_end; ++it)
		cout<<*it<<"\n";

	return 0;
}