Unreachable code?

Hii!
My compiler says:
Unreachable code at beginning of switch statement.
Does that mean that only the first case is not reached??
Because I've deleted all statements within the first case except for the break, and yet the problem still occurs.
Any idea?

So you have code between where it says switch() and the first case label?

In other words:

Code:

---

switch (foo) {
    printf("This line will never happen.\n");
    case 1:
        //etcetera
}

---

No.the first case is in its appropriate place.

You could always post the code, so we can stop guessing and get to the point of actually telling you what's wrong.

Quote:

---

Originally Posted by Leojeen

No.the first case is in its appropriate place.

---

Then you'll have to show us.

Ok
Here is the code.

Code:

---

switch(choice1){
                                    case 1:
                                            printf("\nHow many accounts you want to add?");
                                            scanf("%d",&choice2);
                                            account_number += choice2;
                                            account temp[account_number];
                                            for (int i = 0 ; i < choice2 ; i++)
                                                temp [i] = add_account();
                                            for (int j = choice2 ; j < account_number ; j++)
                                                fread(&temp[j],sizeof(account),1,ptr);
                                            sort(temp,account_number);
                                            for(int k = 0 ; k < account_number ; k++)
                                                    fwrite(&temp[k],sizeof(account),1,ptr);
                                            break;
                                    case 2:
                                            if(!account_number){
                                                                printf("Cannot delete account.Your database is empty\n.");
                                                                exit(1);
                                                                }
                                            account temp2[account_number];
                                            do{
                                                printf("\nType the account's corresponding user name.\n");
                                                account temp;
                                                fgets(temp.user_name,MAX1,stdin);
                                                if(account_number){
                                                                  delete_account(&temp,account_number);
                                                                  account_number--;
                                                                  }
                                                else{
                                                    printf("Cannot delete. Database already empty.\n");
                                                    exit(1);
                                                    }
                                                printf("Again?(1 to remove more, another key to exit):");
                                                scanf("%d",&choice3);
                                                }while(choice3 == 1);
                                            for (int j = 0 ; j < account_number ; j++)
                                                    fread(&temp2[j],sizeof(account),1,ptr);
                                            sort(temp2 , account_number);
                                            for(int k = 0 ; k < account_number ; k++)
                                                    fwrite(&temp2[k],sizeof(account),1,ptr);
                                            break;
                                    case 3:
                                            do{
                                              if(!account_number){
                                                                  printf("Database is empty.\n");
                                                                  exit(1);
                                                                  }
                                              printf("\nType the account's corresponding user name.\n");
                                              account temp;
                                              fgets(temp.user_name,MAX1,stdin);
                                              change_password(&temp);
                                              printf("Password changed.\n");
                                              printf("again?(1 to change more passwords, any other key to exits)",&choice4);
                                              }while(choice4 == 1);
                                    case 4 :
                                            do{
                                              account temp;
                                              printf("Enter the name of the account.\n");
                                              fgets(temp.user_name,MAX2,stdin);
                                              display(&temp,ptr);
                                              printf("More details?(1 to confirm, else to exit):");
                                              scanf("%d",&choice5);
                                              }while(choice5 == 1);
                                    case 5:
                                            printf("You existed the program.\n");
                                            fclose(ptr);
                                            return 0;
                                    default:
                                            printf("Invalid Input.\n");
                                    }

---

Perhaps the switch itself is not reached?

> for (int i = 0 ; i < choice2 ; i++)
Given that this isn't valid C code (it's valid C99 code), the compiler probably turned it into this

Code:

---

switch ( choice1 ) {
    int i;
    case 1:
}

---

and then promptly complained about it.

Try something like this instead, to stop the propagation of the declaration.

Code:

---

switch ( choice1 ) {
    case 1: {
      // all your case 1 code here.
    } break;
}

---

But, the integer i has nothing to do with the case variable. it exits only when I'm looping, and that is I need it for.
and to make clearer, I removed all the code in between the case label (the first one) and the break statement..and the problem is still there.

Guys, I need little help.

Quote:

---

Originally Posted by Leojeen

Guys, I need little help.

---

We can't give you any help because the code you posted doesn't have any errors. You can make sure that the code you posted is the code you're compiling, and that you've got the line number right for the error and you're on the right switch. What IDE & compiler are you using, and what actual error message are you getting?

Ok.
I'm using Dev - C++ 4.9.9.2
and the problem corresponds to the switch line.

The errors are:

In function `int main()':

[Warning] unreachable code at beginning of switch statement

Hope that helps

So I copied that code that you posted in Dev-C++, found the declaration of your struct in another thread, commented out all the function calls that I didn't have, and I got the following:

Quote:

---

temp.c:63: warning: too many arguments for format

---

(that's the line with a printf that ends in &choice4). Nothing about unreachable code (I made sure to add -Wunreachable-code to be sure).

Ok, that may appear long, but have a look at this.
(the problem is still there).
I know there is a function that I haven't implemented yet. (delete).

Code:

---

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <cctype>
#define MAX2 20
#define MAX1 5
typedef struct{
              char user_name[MAX2],password[MAX2];
              int privilige;
        }account;
void display(account *,FILE *);
account add_account();
void delete_account(account *, int);
void change_password(account *);
void sort (account [] , int);
int main(){
          int account_number = 0,choice1,choice2,choice3,choice4,choice5;
          FILE *ptr;
          if((ptr = fopen("accounts.bin","r+")) || (ptr = fopen("accounts.bin","w"))){
                  do{
                    printf("Ready to manipulate your data.\n");
                    printf("\t\tTo add accounts, press 1.\n");
                    printf("\t\tTo delete an account, press 2.\n");
                    printf("\t\tTo change a password, press 3.\n");
                    printf("\t\tTo display details of an account, press 4.\n");
                    printf("\t\tTo exit, press 5.\n");
                    printf("\t\t\tYour choice?");
                    scanf("%d",&choice1);
                    switch(choice1){
                                    case 1:
                                            printf("\nHow many accounts you want to add?");
                                            scanf("%d",&choice2);
                                            account_number += choice2;
                                            account temp[account_number];
                                            for (int i = 0 ; i < choice2 ; i++)
                                                temp [i] = add_account();
                                            for (int j = choice2 ; j < account_number ; j++)
                                                fread(&temp[j],sizeof(account),1,ptr);
                                            sort(temp,account_number);
                                            for(int k = 0 ; k < account_number ; k++)
                                                    fwrite(&temp[k],sizeof(account),1,ptr);
                                            break;
                                    case 2:
                                            if(!account_number){
                                                                printf("Cannot delete account.Your database is empty\n.");
                                                                exit(1);
                                                                }
                                            account temp2[account_number];
                                            do{
                                                printf("\nType the account's corresponding user name.\n");
                                                account temp;
                                                fgets(temp.user_name,MAX1,stdin);
                                                if(account_number){
                                                                  delete_account(&temp,account_number);
                                                                  account_number--;
                                                                  }
                                                else{
                                                    printf("Cannot delete. Database already empty.\n");
                                                    exit(1);
                                                    }
                                                printf("Again?(1 to remove more, another key to exit):");
                                                scanf("%d",&choice3);
                                                }while(choice3 == 1);
                                            for (int j = 0 ; j < account_number ; j++)
                                                    fread(&temp2[j],sizeof(account),1,ptr);
                                            sort(temp2 , account_number);
                                            for(int k = 0 ; k < account_number ; k++)
                                                    fwrite(&temp2[k],sizeof(account),1,ptr);
                                            break;
                                    case 3:
                                            do{
                                              if(!account_number){
                                                                  printf("Database is empty.\n");
                                                                  exit(1);
                                                                  }
                                              printf("\nType the account's corresponding user name.\n");
                                              account temp;
                                              fgets(temp.user_name,MAX1,stdin);
                                              change_password(&temp);
                                              printf("Password changed.\n");
                                              printf("again?(1 to change more passwords, any other key to exits):");
                                              scanf("%d",&choice4);
                                              }while(choice4 == 1);
                                    case 4 :
                                            do{
                                              account temp;
                                              printf("Enter the name of the account.\n");
                                              fgets(temp.user_name,MAX2,stdin);
                                              display(&temp,ptr);
                                              printf("More details?(1 to confirm, else to exit):");
                                              scanf("%d",&choice5);
                                              }while(choice5 == 1);
                                    case 5:
                                            printf("You existed the program.\n");
                                            fclose(ptr);
                                            return 0;
                                    default:
                                            printf("Invalid Input.\n");
                                    }
                    }while(1);
                  }
          else{
                printf("couldn't open the file.\n");
                exit(1);
                }
          printf("\n");
          system("PAUSE");
          return 0;
    }
void display(account *x,FILE *ptr){
                            account temp;
                            strcpy(temp.user_name , "doesn't exist.");
                            strcpy(temp.password , "Doesn't exist.");
                            temp.privilige = 4;
                            int i = 0;
                            while(!feof(ptr)){
                                              fread(&temp,sizeof(account),1,ptr);
                                              if(!strcmp(temp.user_name , x->user_name)){
                                                                                        temp = *x;
                                                                                        exit(1);
                                                                                        }
                                              }
                            printf("\tThe name is: %s\n",temp.user_name);
                            printf("\tThe password is: %s\n",temp.password);
                            switch(temp.privilige){
                                                  case 1 :
                                                          printf("\tThe privilige is admin.\n");
                                                          break;
                                                  case 2:
                                                          printf("\tThe privilige is: general\n");
                                                          break;
                                                  case 3:
                                                          printf("\tThe privilige is mail-only.\n");
                                                          break;
                                                  default :
                                                          printf("Doesn't exist.\n");
                                                  }
                            }


account add_account(){
                      account x;
                      printf("\tEnter the user name:");
                      fgets(x.user_name,MAX2,stdin);
                      printf("\tenter the password");
                      int a = 0;
                      do{
                        fgets(x.password,MAX2,stdin);
                        int s = strlen(x.password);
                        for (int i = 0 ; i < s - 1 ; i++){
                              if(!isalpha(x.password[i])){
                                                          a = 1;
                                                          break;
                                                          }
                              }
                      }while(!a);
                      printf("\tEnter the privilige: (1 for admin, 2 for general, and 3 for mail-only)");
                      scanf("%d",&x.privilige);
                      while(x.privilige != 1 && x.privilige != 2 && x.privilige != 3){
                              printf("Invalid privilige.Enter again:");
                              scanf("%d",&x.privilige);
                              }
                      return x;
        }
void change_password(account *x){
                                printf("Enter the passowrd for %s(should contain at least one non_alpha char)\n",x->user_name);
                                int a = 0;
                                char temp[MAX2];
                                fgets(temp,MAX2,stdin);
                                if (!strcmp(temp,x->password)){
                                                          printf("confirmation:");
                                                          fgets(temp,MAX2,stdin);
                                                          if (!strcmp(temp,x->password)){
                                                                                        printf("Ok, now enter the new password:");
                                                                                        do{
                                                                                          fgets(temp,MAX2,stdin);
                                                                                          int s = strlen(temp);
                                                                                          for (int i = 0 ; i < s - 1 ; i++){
                                                                                              if(!isalpha(temp[i])){
                                                                                                                    a = 1;
                                                                                                                    break;
                                                                                                                    }
                                                                                              }
                                                                                          }while(!a);
                                                                                        strcpy(x->password,temp);
                                                                                        printf("The password is now changed.\n");
                                                                                        }
                                                          else{
                                                              printf("The entered password and the confirmation do not match each other.\n");
                                                              }
                                                          }
      }
void sort(account x[],int n){
                      int result;
                      int min;
                      for (int i = 0 ; i < n  - 1 ; i++){
                            min = i;
                            for (int j = i + 1 ; j < n; j++){
                                      if ((result = strcmp(x[i].user_name,x[j].user_name)) > 0)
                                                  min = j;
                                      }
                            account temp;
                            temp = x[i];
                            x[i] = x[min];
                            x[min] = temp;
                            }
    }

---

Changing cctype to ctype.h and making sure I'm in C99 mode (for gcc on the mac) gives me this:

Code:

---

temp.c: In function ‘main’:
temp.c:43: error: case label in scope of identifier with variably modified type not containing enclosing switch statement
temp.c:70: error: case label in scope of identifier with variably modified type not containing enclosing switch statement
temp.c:84: error: case label in scope of identifier with variably modified type not containing enclosing switch statement
temp.c:93: error: case label in scope of identifier with variably modified type not containing enclosing switch statement
temp.c:97: error: ‘default’ label in scope of identifier with variably modified type not containing enclosing switch statement
temp.c:44: warning: will never be executed
temp.c:45: warning: will never be executed
temp.c:48: warning: will never be executed
temp.c:50: warning: will never be executed
temp.c:54: warning: will never be executed
temp.c:58: warning: will never be executed
temp.c:61: warning: will never be executed
temp.c:64: warning: will never be executed
temp.c:65: warning: will never be executed
temp.c:64: warning: will never be executed
temp.c:66: warning: will never be executed
temp.c:68: warning: will never be executed
temp.c:67: warning: will never be executed
temp.c:72: warning: will never be executed
temp.c:73: warning: will never be executed
temp.c:76: warning: will never be executed
temp.c:87: warning: will never be executed
temp.c:94: warning: will never be executed
temp.c:98: warning: will never be executed
temp.c:106: warning: will never be executed
temp.c: In function ‘display’:
temp.c:115: warning: unused variable ‘i’

---

The compiler doesn't like it when you declare variables inside a case statement. Putting those in blocks so that the other cases can't see them only leaves the warning about line 106 (a printf statement after an exit, which won't ever happen).