Printable View
سلام دوستان يه برنامه نوشتم اصلا ارور نداره ولي چند تا اشتباه ازم مي گيره پيغام خطا اينه
نقل قول:
size of the type is unknown or zero
يكي از خط هايي هم كه اين خطا رو داره اينهنقل قول:
(int pass_search(int N,char user_n[],userpass[100]
ممنون مي شم اگه بگيد كجاشو بايد تغيير بدم تا درست شه:11::11:
شما userpassرو از چه نوعي تعريف کردين ؟ مشخص نيست که .
يه چيزي هم که هست اينه که اندازه آرايه تون معلوم نيست . فقط در صورتي مي تونين يه آرايه بدون سايز تعريف کنين که در موقع تعريف مقداردهي اوليه صورت بگيره .
از نوع كاراكترchar تعريفش كردمنقل قول:
اين شروع يه تابع است به ما گفتن وقتي مي خواين تابع رو تعريف كنيد اگه يه بعدي بود تغيير مقدار نديم واگه دو بعدي بود به يكيشون مقدار بدين!!!
براي اين خط هم همين پيغام رو مي ده
نقل قول:
for(i=0;user_n[j]!='\0'&&userpass[i][j]!='\0';++j
از هیچ نوعی تعریفش نکردی. sin2x کاملا درست میگه. header تابع با متغیر معمولی فرق داره. به صورت زیر نوشتم وکامپایل کردم error نداد.
[HTML]
int pass_search(int N, char user_n[], char userpass[100])
[/HTML]
نظور شما از تابع دوبعدی احتمالا آرایه دوبعدی هست. آرایه ای که سایز نداشته باشه pointer هست. برای توابع معمولا از pointer استفاده میکنند چون pass کردن آرایه بزرگ هم کارایی خوبی نداره و هم میتونه باعث "Stack overflow" بشه. آرایه دوبعدی هم که گفتن به یکیش مقدار بدبد میشه آرایه ای از pointer ها.
منظورتون رو از pass نمي فهمم من يه تابع فرعي به اين نام تعريف كردم و userpass هم يك رشته هستشنقل قول:
يعني من اگه مثل شما بنويسم باز خاصيت رشته بودن رو داره؟؟
پس ايراد دومين مثالي كه نوشتم چيه همين خارو براش مي ده
منظورشون از pass ، پاس کردنه . يعني انتقال يا قرار دادن آرگومان ها در پارامتر هاي تابع و اين فقط يه تشابه اسمي بين تابع شما و کلمه ايشونه . راست مي گن . مي تونين براي رشته اي که قراره به تابع پاس بشه از نوع * char بگيرين و براي اينکه مطمئن بشين تو تابع فراخوانده شده تغيير نمي کنه مي تونين يه const هم بغلش بذارين .نقل قول:
حق با شما بود مشكل رو حل كردم فقط موند يه چيز ديگه حالا توي همون خط ارور زير رو مي ده
اون خط رو اينجوري عوض كردمنقل قول:
Error mahsa.cpp 36:)expected
اينو چي كارش كنم؟؟نقل قول:
(int pass_search(int N,char user_n[],userpass[100][66
تو کدوم خط ؟ ببخشيد ميشه کدهاتون رو داخل تگهاي html قرار بدين بهتر خونده بشه ؟ الان فکر کنم شما
[html]
;)
[/html]
رو تايپ کردين ولي اينجا شکلک افتاده .
آره همونيه كه شما مي گيد
بله userpass در اینجا رشته است. شما هم میخواستین رشته تعریف کنید اما یک char برای هردو متغیرها نوشتید. این کار رو میشه تو بدنه برنامه کرد اما نه در header تابع.نقل قول:
بهترين کار اين بود که برنامتون رو به طور کامل اين جا قرار ميدادين .
مثلا اين آرايه کاراکتري دو بعدي قراره چيکار بکنه ؟ چرا دو بعدي ؟
چه جوري مي تونم برنامم رو توي كد html قرار بدم؟
چه جوري مي تونم برنامم تو html كد وارد كنم؟
چه جوري ميتونم برنامم تو html كد قرار بدم ؟
کدهاتون رو قبل از قرار دادن در سايت تو NotePad کپي کنين . حالا اول کدهاتون کلمه html رو داخل دو تا [] قرار بدين و بعد آخر کدهاتون هم [html/] قرار بدين و بعد همشو کپي کنين تو سايت .
براي مثال اين فايلو ببينين :
کد:http://uploadbazar.com/518example.txt
[html]
#include<iostream.h>
#include<conio.h>
char read_user_pass(int N)
{
int i;
char user_pass[100][66];
for(i=0;i<N;++i)
{
cout<<"Enter username[ "<<i+1<<" ]:";
cin>>user_pass[i];
cout<<"Enter password[ "<<i+1<<" ]:";
cin>>user_pass[i+33];
}
return user_pass[100][66];
}
int user_search(int N,char user_n[],char userpass[100][66])
{
int i,j,k=0;
for(i=0;i<N;++i)
{
for(i=0;user_n[j]!='\0'&&userpass[i][j]!='\0';++j)
if(user_n[j]==userpass[i][j])
k=1;
else
{
k=0;
break;
}
if(user_n[j]!='\0'&&userpass[i][j]=='\0')
k=0;
if(user_n[j]=='\0'&&userpass[i][j]!='\0')
k=0;
}
return k;
}
int pass_serch(int N,char pass_n[],userpass[100][66])
{
int i,j,k=0,m=0;
for(i=0;i<N;++i)
{
for(i=0;pass_n[j]!='\0'&&userpass[i+33][j]!='\0';++j)
if(pass_n[j]==userpass[i+33][j])
k=1;
else
{
k=0;
break;
}
if(pass_n[j]!='\0'&&userpass[i+33][j]=='\0')
k=0;
if(user_n[j]=='\0'&&userpass[i][j]!='\0')
k=0;
}
return k;
}
int main()
{
int N,i,k,r=0;
char userpass[100][66],user_n[100],pass_n[100];
cout<<"enter number of user&&pass:";
cin>>N;
userpass=red_user_pass(N);
for(i=0; ;++i)
{
cout<<"Enter your username:";
cin>>user_n;
k=user_serch(N,user_n,userpass);
if(k=0)
cout<<"user is wrong.\n";
else
{
cout<<"Enter password:";
braek;
}
}
for(i=0; ;++i)
{
cin>>user_n;
k=pass_serch(N,user_n,userpass);
if(k=0)
{
cout<<"user is wrong.\n";
cout<<"Enter password:";
++r;
}
else
break;
}
if(r<=3)
cout<<"welcom";
else
cout<<"welcom 1 person try to enter your user";
getch();
return 0;
}
[/html]
اولين بارمه كه برنامه رو به صورت تابع نوشتم خوب بلد نيستم به احتمال زياد توي همين اشكال گرفته
اين برنامه N يوزر و پس رو مي گيره بعد يه يوزر مي خواد اگه درست بود ژسورد رو مي خواد در غير اين صورت پيغام خطا مي ده اكه پسورد اشتباه بود دوباره پيغام خطا مي ده و اگه سه بار يا بيشتر پس اشتباه وارد شد مي گه كه قبلا سعي شده وارد پسورد شن
اشتباهاتون اکثرا از روی بی دقتی بوده
مثلا در خط
شما نوع متغیر userpass رو تعیین نکردینکد:
int pass_search(int N,char pass_n[],userpass[100][66]
وقتی قبل از userpass کلمه char رو اضافه می کنید یه عالمه خطای دیگه هم پیدا میشه
خب حالا خطا های جدید
توی خط
برنامه متغیر user_n رو نمیشناسهکد:if(user_n[j]=='\0' && userpass[i][j]!='\0')
من دس تو جریان برنامه شما نیستم
ولی اگر این متغیر رو توی قسمت های دیگه تعریف کردین و اینجا می خواین ازش استفاده کنید مسلما نمی شناسش واسه رفع این مشکل یا باید متغیر رو به صورت آرگمان ورودی به تابع پاس بدین یا از اشاره گر ها استفاده کنید و یا اینکه متغیر رو سراسری تعریف کنی( یعنی خارج از تابع main و بقیه تابع ها اون بالا تعریفش کنی)
بعد از اون توی خط
خطا دارین که حدس می زنم اشکال تایپی دارین و باید به جای red بنویسین readکد:userpass=red_user_pass(N);
خطای بعدی توی خط همون خطه که احتمالا از یه آرایه یا همچین چیزی استفاده کردین و بعدش رو مشخص نکردین
خطای بعدی توی خط
هست که بازم خطای تایپی دارینکد:k=user_search(N,user_n,userpass)
user_search هست اسم تابع شما
خطای بعدی از یه دستور break گرفته شده
که بازم تایپ اون اشتباس شما نوشتین braek
خطا های برنامتون همینا بود تا اینجا بر طرف کنید
شاید بعدش بازم خطا پیدا بشه و یا حتی خطا های منطقی
به همون ترتیبی که نوشتم خطا ها رو رفع کنید تا مشکلی پیش نیاد:20:
خطاهاتون اکثرا منطقي هستن تا دستوري .
مثلا تو تابع read_user_pass شما فقط دارين کاراکتر [66][100] رو برميگردونين که به احتمال زياد حاوي يه garbage هست . در حالي که قصد شما فکر کنم برگرداندن کل آرايه هست .
کد شمارو باز نویسی کردم البته فقط برای ایده گرفتن شما. کلی اشتباه syntax داشتید مثل if k=0 که باید به صورت if k==0 نوشته میشد. منطق برنامتون هم اشتباه بود. مثلا pass_search دنبال username هم میگشت. به هر حال اصلاحش کردم و درست هم کار میکنه البته با استفاده از struct و pointer ها. کامنت هم گذاشتم که متوجه تغییرات بشید.
[html]
#include<iostream>
// HINT: I have compiled the code under linux, uncomment this line when you use Windows
//#include<conio.h>
using namespace std;
const int uname_len=33;
const int pname_len=33;
struct users
{
char username[uname_len];
char password[pname_len];
};
users* read_user_pass(int N)
{
int i;
users *user_pass;
user_pass=new users[N];
for (i=0; i<N; ++i)
{
cout << "Enter username[ " << i+1 << " ]:";
cin >> user_pass[i].username;
cout << "Enter password[ " << i+1 << " ]:";
cin >> user_pass[i].password;
}
return user_pass;
}
int user_search(int N, char user_n[uname_len], const users *userpass)
{
int i, k=0;
for (i=0; i<N; ++i)
{
if (strcmp(user_n, userpass[i].username)==0) // a match has founded
{
k=1;
break;
}
else
k=0;
// HINT: Why check pass here? Why "break" when no match found? Shouldn't it be reversed?
/*
for (i=0; (user_n[j]!='\0' && userpass[i][j]!='\0'); ++j)
if (user_n[j]==userpass[i][j])
k=1;
else
{
k=0;
break;
}
if (user_n[j]!='\0' && userpass[i][j]=='\0')
k=0;
if (user_n[j]=='\0' && userpass[i][j]!='\0')
k=0;
*/
}
return k;
}
int pass_serch(int N, char pass_n[pname_len], const users *userpass)
{
int i, k=0;
for (i=0; i<N; ++i)
{
if (strcmp(pass_n, userpass[i].password)==0) // a match has founded
{
k=1;
break;
}
else
k=0;
/*
for (i=0; (pass_n[j]!='\0' && userpass[i+33][j]!='\0'); ++j)
if (pass_n[j]==userpass[i+33][j])
k=1;
else
{
k=0;
break;
}
if (pass_n[j]!='\0' && userpass[i+33][j]=='\0')
k=0;
*/
// HINT: What is it? Why search for "user_n" when it is not even defined?
//if (user_n[j]=='\0' && userpass[i][j]!='\0')
// k=0;
}
return k;
}
int main()
{
int N, k, r=0;
users *userpass;
char user_n[uname_len], pass_n[pname_len];
cout << "Enter number of users:";
cin >> N;
// HINT: I have added this line to prevent entering zero users
if (N==0)
return 0;
userpass=read_user_pass(N);
while (1)
{
cout << "Enter your username: ";
cin >> user_n;
k=user_search(N, user_n, userpass);
if (k==0)
{
cout << "User is wrong.\n";
return 0; // when the username is wrong, there is no
// need to ask for password, so terminate
}
else
break; // user was found, go for password
}
r=0;
while (r<4) // each user is allowed to try max 3 times
// to enter his/her password
{
cout << "Enter your password: ";
cin >> pass_n;
k=pass_serch(N, pass_n, userpass);
if (k==0)
{
cout << "Password is wrong. You have " << r+1 << " out of 3 times to try.\n";
++r;
}
else
break;
}
if (r<=3)
cout << "Welcome to our system. You have logged in successfully.\n";
else
cout << "You have exceeded maximum numbers to enter you password without any success.\n";
// HINT: I have compiled the code under linux, uncomment this line when you use Windows
//getch();
delete[] userpass;
return 0;
}
[/html]
از راهنمايي خيلي مفيدتون خييييييييييلي ممنونم
من برنامه همون طور كه گفتين تغييرش دادم شد:
[html]
#include<iostream.h>
#include<conio.h>
char read_user_pass(int N)
{
int i;
char user_pass[100][66];
for(i=0;i<N;++i)
{
cout<<"Enter username[ "<<i+1<<" ]:";
cin>>user_pass[i];
cout<<"Enter password[ "<<i+1<<" ]:";
cin>>user_pass[i+33];
}
return user_pass[100][66];
}
int user_search(int N,char user_n[],char userpass[100][66])
{
int i,j,k=0;
for(i=0;i<N;++i)
{
for(i=0;user_n[j]!='\0'&&userpass[i][j]!='\0';++j)
if(user_n[j]==userpass[i][j])
k=1;
else
{
k=0;
break;
}
if(user_n[j]!='\0'&&userpass[i][j]=='\0')
k=0;
if(user_n[j]=='\0'&&userpass[i][j]!='\0')
k=0;
}
return k;
}
int pass_search(int N,char pass_n[],char userpass[100][66])
{
int i,j,k=0;
for(i=0;i<N;++i)
{
for(i=0;pass_n[j]!='\0'&&userpass[i+33][j]!='\0';++j)
if(pass_n[j]==userpass[i+33][j])
k=1;
else
{
k=0;
break;
}
if(pass_n[j]!='\0'&&userpass[i+33][j]=='\0')
k=0;
if(pass_n[j]=='\0'&&userpass[i][j]!='\0')
k=0;
}
return k;
}
int main()
{
int N,i,k,r=0,m;
char userpass[100][66],user_n[100],pass_n[100];
cout<<"enter number of user&&pass:";
cin>>N;
userpass[100][66]=read_user_pass(N);
for(i=0; ;++i)
{
cout<<"Enter your username:";
cin>>user_n;
k=user_search(N,user_n,userpass);
if(k==0)
cout<<"user is wrong.\n";
else
{
cout<<"Enter password:";
break;
}
}
for(i=0; ;++i)
{
cin>>user_n;
m=pass_search(N,user_n,userpass);
if(m==0)
{
cout<<"user is wrong.\n";
cout<<"Enter password:";
++r;
}
else
break;
}
if(r<=3)
cout<<"welcom";
else
cout<<"welcom 1 person try to enter your user";
getch();
return 0;
}
[/html]
مشكل حل شد اما پس از اجراي برنامه توي يك قسمتش گير مي كنه چه جوري برطرفش كنم؟
کدوم برنامه گير مي کنه ؟ من همين الان برنامه DaneshD رو تست کردم اصلا مشکلي نداره .
چه جوري بنويسم كه درست شه؟؟نقل قول:
برنامه ي خودم پس از اينكه مشكل برطرف شدنقل قول:
برنامه ي DaneshD رو هم نمي تونم اجرا كنم خطا مي ده؟؟
مشكل اينجا است كه ما دستورات constو فرم نوشتن
using namespace std;رو تا حالا نداشتيم و نمي تونيم تو برنامه هايي كه مي نويسيم استفاده كنيم اصلا نمي فهمم اون برنامه اي كه دوست عزيزمون لطف كردن رو چه جوري آغاز كردن!
شما مي تونين از روش دوستمون DaneshD استفاده کنين که يه ساختمان تعريف کردن و username و password رو تو اون گذاشتن و بعد يه struct برگردوندن .
شما براي iostream يه h. بذارين و using namespace std رو هم حذف کنين .نقل قول:
ايشون سعي کردن تا حد ممکن برنامه رو استاندارد بنويسن . شما اگه نمي خواين از const استفاده کنين مي تونين اون اعلان هاي const رو حذف کنين و هر جا تو برنامه uname_len و pname_len ديدين بجاش 33 بذارين .
مشكل هم همينجاست ما نمي تونيم از اين شيوه اسفاده كنيم راستش رو بخوايد اولين باره اسم تعريف ساختمان رو شنيدم!!نقل قول:
بچه ها با یه تغییرات کوچولو تو کدشون فک کنم بشه بدون pointer و struct و اینام نوشتا
بايد بشه و گرنه استادمون اين تمرين رو بهمون نمي داد :41:اگه مي تونيد كمكم كنيدنقل قول:
سلام دوستان من این برنامه رو به این ترتیب تغیرش دادم اجرا می شه ولی نتیجه ی مطلوب رو نمی ده>
[html]
#include<iostream.h>
#include<conio.h>
char read_user(int N)
{
int i;
char user[200][60];
for(i=0;i<N;++i)
{
cout<<"enter user["<<i+1<<"]:";
cin>>user[i];
}
return user[i][60];
}
int search_user(char user[][60],char user_new[],int N)
{
int k=-1,i,j;
for(i=0;i<N;++i)
{
for(j=0;user[i][j]!='\0'&&user_new[j]!='\0';++j)
if(user_new[j]!=user[i][j])
{
k=0;
break;
}
else
k=1;
if(user[i][j]=='\0'&&user_new[j]=='\0')
{
k=1;
break;
}
else
k=0;
}
return k;
}
char read_pass(int N)
{
int i;
char pass[200][60];
for(i=0;i<N;++i)
{
cout<<"Enter pass["<<i+1<<"]:";
cin>>pass[i];
}
return pass[i][60];
}
int search_pass(char pass[][60],char pass_new[],int N)
{
int i,j,k=-1;
for(i=0;i<N;++i)
{
for(j=0;pass_new[j]!='\0'&&pass[i][j]!='\0';++j)
if(pass_new[j]!=pass[i][j])
{
k=0;
break;
}
else
k=1;
if(pass[i][j]=='\0'&&pass_new[j]=='\0')
{
k=1;
break;
}
else
k=0;
}
return k;
}
int main()
{
int i,N,j=0,m;
char user_new[60],pass_new[60],pass[200][60],user[200][60];
cout<<"Enter Number of data:";
cin>>N;
read_user(N);
read_pass(N);
cout<<"Enter username:";
cin>>user_new;
m=search_user(user,user_new,N);
for(i=0; ;++i)
if(m==0)
{
cout<<"Enter your user:";
cin>>user_new;
m=search_user(user,user_new,N);
}
else
{
cout<<"enter your password:";
cin>>pass_new;
break;
}
for(i=0; ;++i)
{
m=search_pass(pass,pass_new,N);
if(m==0)
{
++j;
cout<<"Enter new pass:";
cin>>pass_new;
}
if(m==1&&j<=3)
cout<<"welcom";
if(m==1&&j>3)
{
cout<<"welcom";
cout<<"one person try to open your pass";
}
}
getch();
return 0;
}
[/html]
خواهش می کنم اگه می شه بهم بگید کجا رو اشتباه کردم؟؟؟