کمک در مورد سی پلاس پلاس

arkco

سلام
می خوام یک برنامه بنویسم که 20 تا عدد رو از ورودی بگیره و اگه عددی با عددهای قبلی وارد شده برابر بود پیغام بده که این عدد قبلا وارد شده و بعد یک عدد دیگه بگیره باز اگه عدد وارد شده مساوی با عددهای قبلی باشه پیغام بده و یه عدد دیگه بگیره .
یعنی این که به هیچ وجه نشه دو عدد یک جور به برنامه داد. اگه کسی بلده برام بنویسه یا حداقل راهنمایی کنه که من به این برنامه نیاز فوری دارم.
ضمنا می خوام برنامه در سی پلاس پلاس نوشته بشه.
دستتون درد نکنه.

Hamid

سلام؛
کار سختی نیست، اعداد رو در یک آرایه بریز و یک شمارنده در نظر بگیر.
هر بار اگر یک عدد رو با موفقیت گرفتی، یکی به شمارنده اضافه کن.
هر عددی که می‌گیری، آرایه رو از 0 تا شمارنده، جستجو کن و ببین که مساوی با اون عدد ورودی پیدا می‌کنی یا خیر. اگر پیدا شد، وارد آرایه نکن، وگرنه، به آرایه اضافه کن.
همین Wink

Gytyonline

سلام به همه
همون طوری که حمید جان گفتن برنامه سختی نیستش و توضیحاشم کامل بودش بازم اگه نتونستی یا متوجه نشدی بگو تا برات برنامه رو یا حداقل قسمتی از اون رو برات بنویسم

arkco

Hamid و ali048 عزیز خیلی ممنونم که مرا راهنمایی کردید.
ali048 اگه می شه این برنامه را برام بنویس.پیشاپیش از شما تشکر می کنم.

Gytyonline

سلام arkco جان
سعی کن همیشه کد های برنامه تو خودت بنویسی هر چند غلط باشه بعدا اون کد رو بزار اگه غلط باشه و اصلاح می کنیم برات اگه نه هم کمکت می کنیم البته اگه میخوای خودت م یاد بگیری برنامه نویسی رو اگه نه هم که ....

اینم کد برنامه ببین کار می کنه ( با سی هستش تبدیلش به سی پلاس کاری نداره)

Hamid

ali048, جان،
الگوریتم رو درست پیاده‌سازی کردی، اما چند تا قسمت از برنامه بهینه نیست.
1- این یکی از بزرگ‌ترین مشکلات برنامه‌نویس‌هاست:
هیچ وقت توی یک حلقه، متغیر رو تعریف نکن!
چرا j رو وسط حلقه while تعریف کردی؟ این یعنی اینکه j حداقل بیست بار تعریف می‌شه! در حالی که متغیر آمده که از تعریف مجدد جلوگیری کنه. یک بار تعریف، چند بار مقداردهی.
j رو هم کنار همون i تعریف کن.

2- شرط if(flag==0 رو باید بعد از حلقه for بذاری.
ضمناً flag رو هم باید انتهای حلقه مجدداً 0 کنی وگرنه کلاً برنامه با مشکل مواجه می‌شه.

3- بهتره قبل از break یک دستور بنویسی که مثلا: "این عدد تکراری است" وگرنه کاربر فکر می‌کنه باید عدد بعدی رو وارد کنه.

به هر حال، فکر می‌کنم بهتر باشه کدها رو به این صورت تغییر بدیم:

Gytyonline

سلام حمید جان
واقعا ممنون به خاطر راهنمایی ها و نکته های عالی که بهم گوش زد کردی ، در مورد نکته اول به شخصه فکر میکردم اگه متغییر داخل حلقه for تعریف بشه بهینه تر هستش چون وقتی for تموم بشه فضای اشغال شده به حافظه بر می گرده ولی به این نکته یی که گفتی تا حالا اصلا توجه نکرده بودم . بازم مرسی
به خاطر همین شعار سایتم رو گذاشتم : " ياد بگير و به ديگران ياد بده " چون توی یاد گرفتن 1سری چیزا رو یاد میگیره ادم توی یاد دادن 1سری چیزای جدید تر و ناب تری رو Razz

arkco

سلام دوستان
من خودم این برنامه رو این طوری نوشتم.چک کنید ببینید درسته یا نه

Hamid

arkco, جان؛
اول یک نکته بگم: فراموش نکن که همیشه الگوریتم یک برنامه بیشتر از کد ارزش داره.
شما اگر الگوریتم یک برنامه رو بدونید می‌تونید خیلی راحت اون رو پیاده‌سازی کنید.

خوب، حالا شما همیشه اول در مورد الگوریتم برنامه‌ت توضیح بده و بعد، کد اون رو قرار بده.
الان ما هی بالا و پایین می‌ریم، متوجه نمی‌شیم که شما چی کار کردی.
اگر ممکنه روند عملکرد (الگوریتم) برنامه‌ت رو توضیح بده... Wink

Gytyonline

سلام arkco
اول از همه این کدت رو من فقط ظاهرش رو درست کردم تا قابل خوندن بشه چون اون طوری فکنم خودتم سرت گیج میرفت اگه می خواستی دوباره بخونیش Mr. Green