مالخطأ في برنامجي

رتب اولا الكود اخي ليسهل قرائته

#include<stdio.h>
#include<conio.h>
main()
{
	int min,i,j;
	int t[4]={12,15,36,28};
	for( i=0;i<4;i++ )
	{
		min=t;
		for( j=1;i<3;j++ )
		{
			 if(min>t[j])
			{
				min=t[j];
				t[j]=t;
				t=min;
			}
		}
	} 
	for( i=0;i<4;i++ )
	{
		printf("t[%d]=%d",t);     
	}
	getch();          
}

حسناً، الخطأ الذي يمنع برنامجك من العمل هو:

for( j=1;i<3;j++ )

لأنك تقوم بزيادة j إلى اللانهاية حيث أنك تعتمد في شرط استمرار الحلقة على كون i < 3، وهذا سيظل محققاً دائماً لأنك لا تقوم بزيادة قيمة i داخل هذه الحلقة.

يوجد أيضاً في الكود بعض الأخطاء المنطقية، قمت بالتعديل المناسب بإذن الله:

#include<iostream>
using namespace std;

int main()
{
        int i, j;
	char a;
        int t[4]= {47,15,36,28};        
	for(i= 0; i < 3; i++)
        {
                for(j= i + 1; j < 4; j++)
                {
                         if(t > t[j])
                        {
				int temp;
                                temp= t[j];
                                t[j]= t;
                                t= temp;
                        }
                }
        } 
        for(i= 0; i < 4; i++)
        {
                cout<< t<< endl;
        }
	cin>> a;        
	return 0;          
}

لاحظ أنك لا تحتاج إلى متغير min، فقط تحتاج لمتغير مساعد temp أثناء تبديل القيم.

ملاحظة: استعملت مكتبة iostream وكائناتها لأنني لا أجيد التعامل مع المكتبات التي كنت تستعلمها، فليس من الضرورة أن تقوم بتغيير هذه الأمور.

بالتوفيق :)

Abboodd كتب:

حسناً، الخطأ الذي يمنع برنامجك من العمل هو:

for( j=1;i<3;j++ )

لأنك تقوم بزيادة j إلى اللانهاية حيث أنك تعتمد في شرط استمرار الحلقة على كون i < 3، وهذا سيظل محققاً دائماً لأنك لا تقوم بزيادة قيمة i داخل هذه الحلقة.

يوجد أيضاً في الكود بعض الأخطاء المنطقية، قمت بالتعديل المناسب بإذن الله:

#include<iostream>
using namespace std;

int main()
{
        int i, j;
	char a;
        int t[4]= {47,15,36,28};        
	for(i= 0; i < 3; i++)
        {
                for(j= i + 1; j < 4; j++)
                {
                         if(t > t[j])
                        {
				int temp;
                                temp= t[j];
                                t[j]= t;
                                t= temp;
                        }
                }
        } 
        for(i= 0; i < 4; i++)
        {
                cout<< t<< endl;
        }
	cin>> a;        
	return 0;          
}

لاحظ أنك لا تحتاج إلى متغير min، فقط تحتاج لمتغير مساعد temp أثناء تبديل القيم.

ملاحظة: استعملت مكتبة iostream وكائناتها لأنني لا أجيد التعامل مع المكتبات التي كنت تستعلمها، فليس من الضرورة أن تقوم بتغيير هذه الأمور.

بالتوفيق :)

بارك الله فيك يا أخي على الاجابة

لكني أقصد بالمتغير min أني أعتبر أصغر عنصر في الجدول موجود في الخانة الأولى ثم أبدأ بالمقارنة مع الخانات الاخرى فإن وجدت مثلا أن الخانة 2 يوجد فيها عدد أقل من الخانه الأولى أقوم بالتغير بين الخانة الأولى والثانية وهكذا فهل أستطيع عمل ذالك بهذا الكود

:) :) :)

نعم لا مشكلة أعد المتغير min كما كان، لكني لا أرى له فائدة حقيقية هنا.

بالتوفيق :)

شكرااا على مساعدتي ولقد تمكنت بفضلكم من اتمام برنامجي واصبح بهذا الشكل

#include<stdio.h>
#include<conio.h>
main()
{int min,i,j;

int t[4]={12,15,36,28};

   for( i=0;i<3;i++)
{         
       min=t;
       for( j=i+1;j<4;j++ )
            {
                  if(min>t[j])
                      {
                           min=t[j];
                           t[j]=t;
                            t=min;
                                      }
                                            }
                                                }      
  for( i=0;i<4;i++ )
{
        printf("t[%d]=%d \n",i,t);     
}     
   getch();        
        }

:) :) :lol: