Decoder

[rishamessi]

Decoder show values as
_
| |
|_|

this represent zero contain 6 dashes
my code is to take a string like "01" and return 8 , 6 dahses from zero and 2 from one
there is something wrong can anyone help me

# include <iostream>
# include <string>
using namespace std ;

class CheckFunction 
{
      public : 
               int newFunction(string code) ;
};

int CheckFunction::newFunction(string code)
               {
                   int count = 0 ;
                   for ( int i = 0 ; i < code.length() ; i ++ )
                   {
                       if ( code[i] == 0 )
                       {
                            count += 6 ;
                       }
                       else if ( code[i] == 1 )
                       {
                            count += 2 ;
                       }
                       else if ( code[i] == 3 )
                       {
                            count += 5 ;
                       }
                       else if ( code[i] == 4 )
                       {
                            count += 4 ;
                       }
                       else if ( code[i] == 5 )
                       {
                            count += 5 ;
                       }
                       else if ( code[i] == 6 )
                       {
                            count += 6 ;
                       }
                       else if ( code[i] == 7 )
                       {
                            count += 3 ;
                       }
                        else if ( code[i] == 8 )
                       {
                            count += 7 ;
                       }
                       else if ( code[i] == 9 )
                       {
                            count += 3 ;
                       }
                   }
                   return count ;
               }


int main ()
{
    string str ;
    cin >> str ; 
    
    CheckFunction c1 ;
    cout << c1.newFunction(str);
    
    system("pause");
    return 0 ;
}

[j3bk4iN6]

You can place the member function outside of the class by making a constructor then

<type> <class name>::<function> outside the class. with a function that long it makes sense

a better way to make an encoder is to use ceasar cypher and shift the ascii values by a number and make the char 'a' through 'z' circular.

[rishamessi]

that is not the problem
please any one help me

[j3bk4iN6]

then the decoder would use the opposite shift.

[jLNv0pDG]

#include <string>

[rishamessi]

the problem isn't solved
in every time the output is zero he couldn't read any of the if conditions please help

[rishamessi]

please urgent , help please

[j3bk4iN6]

well "01" would return 8 since 0 count +=6 and 1 count +=2.

[rishamessi]

but it return zero but i don't know the reason please help urgent please

[j3bk4iN6]

try else if for the if's that follow if and make the last one else

[rishamessi]

still zero please try it on your IDE and see it and help me ....urgent

[gcampton]

but it return zero but i don't know the reason please help urgent please

why is it when I read posts like this I feel the need to do the opposite and not help?

[gcampton]

Blah! whatever, code is a string not an array. try a const char* or a better comparison

[j3bk4iN6]

using namespace std ;

class CheckFunction 
{
      public : 
               int newFunction(char* str) ;
};

int CheckFunction::newFunction(char* str)
               {
                   int count = 0 ;
                   for ( int i = 0 ;  *(str).length()  ; i ++ )
                   {
                       if ( *(str)+i == 0 )
                       {
                            count += 6 ;
                       }
                       else if ( *(str)+i == 1 )
                       {
                            count += 2 ;
                       }
                       else if ( *(str)+i == 3 )
                       {
                            count += 5 ;
                       }
                       else if ( *(str)+i == 4 )
                       {
                            count += 4 ;
                       }
                       else if ( *(str)+i == 5 )
                       {
                            count += 5 ;
                       }
                       else if ( *(str)+i == 6 )
                       {
                            count += 6 ;
                       }
                       else if ( *(str)+i == 7 )
                       {
                            count += 3 ;
                       }
                        else if ( *(str)+i == 8 )
                       {
                            count += 7 ;
                       }
                       else if ( *(str)+i == 9 )
                       {
                            count += 3 ;
                       }
                   }
                   return count ;
               }


int main ()
{
    char* str = 0 ;
    cin >> str ; 
	
	
    
	CheckFunction c1;
    cout << c1.newFunction(str);
    

    return 0 ;
}

just to let you know this is what I got sofar but length() not working. I think the whole thing should be re-worked so its a function instead of a class.

[gcampton]

why do his homework for him? it was bad enuf he was begging....