kavinderrana121 Ответов: 3

Программа на языке Си для нахождения простого числа и его положения в заданном массиве

это модульная программа на языке Си для поиска простого числа a и отображения его положения в заданном массиве
я пытаюсь сделать эту программу определив функцию нет ошибки времени компиляции но эта программа терпит крах когда я запускаю эту программу
Ребята пожалуйста помогите мне я новичок в программировании

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

#include<stdio.h>
void prime_num(int num,int a[10]);
int main()
{
   int a[10],i,num;
    printf("enter the size of array");
      scanf("%d",&num);
    printf("enter the elements");
         
          for(i=0;i<num;i++)
              {
                 scanf("%d",&a[i]);
                    }
         printf("the entered array is ");
              for(i=0;i<num;i++)
           {
              printf("%d",a[i]);
                }

               prime_num(num,a[10]); //function called here

   }  //end of main function
void prime_num(int num,int a[10])
{
    int i,counter,j;
printf("all the prime numbers are");
for(i=0;i<=num;i++)
{
    counter=0;
    for(j=2;j<a[i];j++)
    {
        if((a[i]%j)==0)
        {
            counter=1;
            break;
        }


    }
    if(counter==0)
   {

    printf("the found prime no is at %d position and the number is %d",i,a[i]);
   }
}

}

3 Ответов

Рейтинг:
26

Patrice T

Цитата:
там нет ошибки времени компиляции, но эта программа падает, когда я запускаю эту программу

Что такое сообщение об ошибке, положение ошибки, каков ваш ввод ?
Вся информация, которую Вы нам не дали.

prime_num(num,a[10]);

Здесь, a[10] является ли адрес чего-то после окончания a потому что размер равен 10, а элементы-от 0 до 9.
Правильный код должен быть таким:
prime_num(num,a);


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

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

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

Рейтинг:
20

CPallini

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

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

void prime_num(int size, int a[]);

int main()
{
  int * a, i, size;

  printf("enter the size of array\n");
  scanf("%d",&size);
  printf("enter the elements\n");


  //-> dynamic memory allocation
  a = malloc( size * sizeof (int) );

  if (! a )
  {
    printf("unable to allocate memory\n");
    return -1;
  }
  //<- 
         
  for(i=0;i<size;i++)
  { 
    scanf("%d",&a[i]);
  }
                    
  printf("the entered array is\n");

  for(i=0;i<size;i++)
  { 
    printf("%d\n",a[i]);
  }

  prime_num(size, a); //function called here


  free( a ); // dynamic memory deallocation

  return 0;

}  //end of main function

void prime_num(int size,int a[])
{
  int i, j;
  printf("all the prime numbers are\n");
  for(i=0;i<size;i++)
  { 
    for(j=2;j<a[i];j++)
    { 
      if((a[i]%j)==0)
        break;
    }
    if(j == a[i])
    { 
      printf("the found prime no is at %d position and the number is %d\n",i,a[i]);
    }
  }
}

Рейтинг:
11

OriginalGriff

Начните с сортировки отступов, чтобы ваш код было легче читать, и избавления от избыточных комментариев (а это все они). Прокомментируйте то, что делает код, а не то, что представляют собой фактические инструкции, мы можем прочитать их...

int main()
{
   int a[10],i,num;
   printf("enter the size of array");
   scanf("%d",&num);
   printf("enter the elements");
   for(i=0;i<num;i++)
   {
       scanf("%d",&a[i]);
   }
   printf("the entered array is ");
   for(i=0;i<num;i++)
       {
       printf("%d",a[i]);
       }
   prime_num(num,a[10]);
   }
Затем начните работать над тем, что не так с вашим кодом.
Начните с рассмотрения того, что вы вводите: какова ценность num например? Если он больше 10, у вас есть проблема, потому что вы разрешили ввести только 10 значений. Вы должны проверить это и отклонить значения, которые не подходят (потому что если вы этого не сделаете, ваше приложение рухнет, когда ваш пользователь введет "плохое" значение).

Следующее, что нужно сделать, - это посмотреть на свой prime_num функция. Что именно вы передаете ему? Это массив или отдельный элемент?

Подсказка: что будет делать [10], если вы поставите его после массива, а именно массива из десяти элементов?