Problem with reading text from a .txt file

[Gigi]

do I need to put in the array and then try analyzing it ? I don't really have much time, because it's already end of the course and we haven't even looked at Files and Exception Handling, so I'm kind doing everything off that little problem that I did. I would reallyy appreciate if you could give me a hand with this one

[Fubarable]

do I need to put in the array and then try analyzing it

Why not give it a try? You've got nothing to lose.

[Gigi]

I don't really get, since it reads each line and then does the decryption, I don't know how to tell not to analyze it after its read each line, but wait until it reaches the *** part and ... ?

[Gigi]

.readLine() <- will always read just one line at the time, until put it the loop ? but how is this going to work ?

[Fubarable]

it only analyzes each line because you tell it to analyze after reading each line. If you don't want it to do that, then don't tell it to do that. If you only want it to analyze after it either reaches *** or it runs out of lines, then you must tell it to only analyze after reaching these points. Again, the program code follows the logic. Anyway, it's late, I'm on call at the hospital tomorrow, and I'm going to bed. Best of luck to you and to your endeavors!

[angryboy]

hey u might want to change this line.

if (sentence.equals("***"))
{
    System.out.println("Enter any key to continue: ");
    inputs.nextLine(); // CHANGED!!!!!!
}

also, there is something wrong w/ the calculation for tempIndex. you'll wana look into that.

here's the original values for tempIndex:

S:\SVN\gigi>java Problem2CeasarCipher
22
THERE WAS NOTHING SO VERY REMARKABLE IN THAT
11
YZC OTO LWTNP ESTYV TE DZ GPCJ XFNS ZFE ZQ ESP HLJ
11
EZ SPLC ESP CLMMTE DLJ EZ TEDPWQ
22
OH DEAR OH DEAR I SHALL BE LATE
Enter any key to continue:

here's what happend after I changed tempIndex = 22.

S:\SVN\gigi>java Problem2CeasarCipher
22
THERE WAS NOTHING SO VERY REMARKABLE IN THAT
22
NOR DID ALICE THINK IT SO VERY MUCH OUT OF THE WAY
22
TO HEAR THE RABBIT SAY TO ITSELF
22
OH DEAR OH DEAR I SHALL BE LATE
Enter any key to continue:

[angryboy]

i see your problem. your encoding seed is per paragraph. but your decoding seed is per line. so you need to change the loop to not change tempIndex until it hits ***.

[angryboy]

yup, just set tempIndex every paragraph instead of every sentence.
also reset values for all of your variables such as: wordLetters,arrayIndex,offset, tempIndex, tempHighest, etc... to zero.

Sorry, i'm not allow to post complete solutions or else CJSLMAN will b-slap me for it.

S:\SVN\gigi>java Problem2CeasarCipher
THERE WAS NOTHING SO VERY REMARKABLE IN THAT
NOR DID ALICE THINK IT SO VERY MUCH OUT OF THE WAY
TO HEAR THE RABBIT SAY TO ITSELF
OH DEAR OH DEAR I SHALL BE LATE
Enter any key to continue:

EITHER THE WELL WAS VERY DEEP OR SHE FELL VERY SLOWLY
FOR SHE HAD PLENTY OF TIME AS SHE WENT DOWN TO LOOK ABOUT HER
AND TO WONDER WHAT WAS GOING TO HAPPEN NEXT
FIRST SHE TRIED TO LOOK DOWN AND MAKE OUT
WHAT SHE WAS COMING TO BUT IT WAS TOO DARK TO SEE ANYTHING
Enter any key to continue:

THEN SHE LOOKED AT THE SIDES OF THE WELL
AND NOTICED THAT THEY WERE FILLED WITH CUPBOARDS AND BOOKSHELVES
HERE AND THERE SHE SAW MAPS AND PICTURES HUNG UPON PEGS
SHE TOOK DOWN A JAR FROM ONE OF THE SHELVES AS SHE PASSED
IT WAS LABELLED ORANGE MARMALADE
BUT TO HER GREAT DISAPPOINTMENT IT WAS EMPTY
SHE DID NOT LIKE TO DROP THE JAR FOR FEAR OF KILLING SOMEBODY
SO MANAGED TO PUT IT INTO ONE OF THE CUPBOARDS AS SHE FELL PAST IT

[Gigi]

I really need this done and I have no idea how to set tempIndex every paragraph instead of every sentence. Do I need to create an array that will store it ? or what ? I don't want to spend 5 hours doing this question, or you can just tell me how it's done

[angryboy]

no no, not every paragraph. only set tempIndex to the max value of the first line of each paragraph. don't calculate tempIndex for the 2nd lines and so on.

here's the logic:

if input == ***, wait for user input, and reset variables.
else if sentence == firstSentence, then calculate for tempIndex and decode.
else just decode.
output decoded data.

[Gigi]

no no, not every paragraph. only set tempIndex to the max value of the first line of each paragraph. don't calculate tempIndex for the 2nd lines and so on.

here's the logic:

if input == ***, wait for user input, and reset variables.
else if sentence == firstSentence, then calculate for tempIndex and decode.
else just decode.
output decoded data.

I don't undestand this line:
else if sentence == firstSentence,
what is firstSentence ? and how it goes
can you just type the code here ?

[Gigi]

else if sentence == firstSentence, then calculate for tempIndex and decode. What is firstSentence that you comparing to ? is this case, how would you distinguish sentences in the file(which one is the first ?) because the programs read it one after another, it doesn't know which one is the first

[angryboy]

i just made it up. but you can use some cheap code like this:

boolean firstSentence = false
if ( sentence == ***) then firstSentence = true.
if (firstSentence) then calc tempIndex and decode.
else just decode.

[Gigi]

import java.util.Scanner;
//import java.util.ArrayList;
import java.io.*;

public class Problem2CeasarCipher
{
    public static void main(String[] args) throws IOException
    {
     BufferedReader input = new BufferedReader(new FileReader("DATA21.txt")); 
     Scanner inputs = new Scanner(System.in); 
    
     char[] wordLetters;
     int[] letterCounts = new int[26];
     int arrayIndex;
     int offset;
     int tempIndex = 0;
     int tempHighest = 0;
     String sentence = "";   
     //ArrayList decodedText = new ArrayList();
    boolean firstSentence = false;

    
while ((sentence = input.readLine()) != null)
{    
  
   
  if (sentence.equals("***"))
        {
        firstSentence = true;
        System.out.println("Enter any key to continue: ");
        inputs.nextLine();
       
                  arrayIndex=0;
                  offset =0;   
                  tempIndex =0;
                   tempHighest = 0;
                   wordLetters = null;

       }

 
else if (firstSentence)
  {  
   
       String copySentence = sentence;
       sentence = sentence.replaceAll(" ", "");
       wordLetters = sentence.toCharArray();
     
      for(int letter = 0; letter < wordLetters.length; letter++)
     {
        offset = wordLetters[letter] - 65;
        letterCounts[offset] +=1;
     }
     
     for(int i = 0; i <= 25; i++)
      {
       if (letterCounts[i] > tempHighest)
       { 
         tempHighest = letterCounts[i];
         tempIndex = i;
        }
      } 
 
         testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    
         System.out.println(encoding.decode(copySentence));
    }     
     else
        {
          testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    
          //System.out.println(encoding.decode(copySentence));
        }
    
     }  
   }    
}

doesn't really work

output:

Enter any key to continue: 
 [DrJava Input Box]
EITHER THE WELL WAS VERY DEEP OR SHE FELL VERY SLOWLY
FOR SHE HAD PLENTY OF TIME AS SHE WENT DOWN TO LOOK ABOUT HER
AND TO WONDER WHAT WAS GOING TO HAPPEN NEXT
FIRST SHE TRIED TO LOOK DOWN AND MAKE OUT
WHAT SHE WAS COMING TO BUT IT WAS TOO DARK TO SEE ANYTHING
Enter any key to continue: 
 [DrJava Input Box]
FTQZ ETQ XAAWQP MF FTQ EUPQE AR FTQ IQXX
MZP ZAFUOQP FTMF FTQK IQDQ RUXXQP IUFT OGBNAMDPE MZP NAAWETQXHQE
TQDQ MZP FTQDQ ETQ EMI YMBE MZP BUOFGDQE TGZS GBAZ BQSE
ETQ FAAW PAIZ M VMD RDAY AZQ AR FTQ ETQXHQE ME ETQ BMEEQP
UF IME XMNQXXQP ADMZSQ YMDYMXMPQ
NGF FA TQD SDQMF PUEMBBAUZFYQZF UF IME QYBFK
ETQ PUP ZAF XUWQ FA PDAB FTQ VMD RAD RQMD AR WUXXUZS EAYQNAPK
EA YMZMSQP FA BGF UF UZFA AZQ AR FTQ OGBNAMDPE ME ETQ RQXX BMEF UF

this program is getting annoying...

[Fubarable]

You need to not do your analysis when reading a sentence unless either *** is reached or the end of the text is reached. Right now, it appears that you're doing the exact opposite.

Myself, I try to walk through the logic of my program by stepping through the program on paper thinking "now what will it do when it reads the first line" then looping and stepping through it again until *** is reached. If you do this for one of your "paragraphs" of data, you'll see what your mistakes are.

[angryboy]

keep trying, you'll get it. well the code was badly written to begin with... also, boolean firstSentence neeeds to be inside the while loop.

[Gigi]

import java.util.Scanner;
//import java.util.ArrayList;
import java.io.*;

public class Problem2CeasarCipher
{
    public static void main(String[] args) throws IOException
    {
     BufferedReader input = new BufferedReader(new FileReader("DATA21.txt")); 
     Scanner inputs = new Scanner(System.in); 
    
     char[] wordLetters;
     int[] letterCounts = new int[26];
     int arrayIndex;
     int offset;
     int tempIndex = 0;
     int tempHighest = 0;
     String sentence = "";   
     //ArrayList decodedText = new ArrayList();
    
    
while ((sentence = input.readLine()) != null)
{    
    boolean firstSentence = false;
   
  if (sentence.equals("***"))
        {
        firstSentence = true;
        System.out.println("Enter any key to continue: ");
        inputs.nextLine();
       
                  arrayIndex=0;
                  offset =0;   
                  tempIndex =0;
                   tempHighest = 0;
                   wordLetters = null;

       }

 
else if (firstSentence)
  {  
   testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    
          //System.out.println(encoding.decode(copySentence));
       

    }     
     else
        {
         
         String copySentence = sentence;
       sentence = sentence.replaceAll(" ", "");
       wordLetters = sentence.toCharArray();
     
      for(int letter = 0; letter < wordLetters.length; letter++)
     {
        offset = wordLetters[letter] - 65;
        letterCounts[offset] +=1;
     }
     
     for(int i = 0; i <= 25; i++)
      {
       if (letterCounts[i] > tempHighest)
       { 
         tempHighest = letterCounts[i];
         tempIndex = i;
        }
      } 
 
         testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    

        }
    
     }  
   }    
}

Enter any key to continue: 
 [DrJava Input Box]
Enter any key to continue:

I can't get it seriosly, I'm just too weak for this and I'm running out of time, I still have to study for the exam, can you just tell me what EXACTLY needs to be changed? because I can do this all day long without any progress ?

[angryboy]

i can't because i rewrote the whole thing completly different from yours. but it looks like u need to move some stuff out of the else block. try moving this line out.

testProblem2CeasarCipher...

then try moving other stuff out too. gtg.

[Gigi]

import java.util.Scanner;
//import java.util.ArrayList;
import java.io.*;

public class Problem2CeasarCipher
{
    public static void main(String[] args) throws IOException
    {
     BufferedReader input = new BufferedReader(new FileReader("DATA21.txt")); 
     Scanner inputs = new Scanner(System.in); 
    
     char[] wordLetters;
     int[] letterCounts = new int[26];
     int arrayIndex;
     int offset;
     int tempIndex = 0;
     int tempHighest = 0;
     String sentence = "";   
     //ArrayList decodedText = new ArrayList();
    
    
while ((sentence = input.readLine()) != null)
{    
    boolean firstSentence = false;
   
  if (sentence.equals("***"))
        {
        firstSentence = true;
        System.out.println("Enter any key to continue: ");
        inputs.nextLine();
       
                  arrayIndex=0;
                  offset =0;   
                  tempIndex =0;
                   tempHighest = 0;
                   wordLetters = null;

       }

 
else if (firstSentence)
  {  
         String copySentence = sentence;
       sentence = sentence.replaceAll(" ", "");
       wordLetters = sentence.toCharArray();
     
      for(int letter = 0; letter < wordLetters.length; letter++)
     {
        offset = wordLetters[letter] - 65;
        letterCounts[offset] +=1;
     }
     
     for(int i = 0; i <= 25; i++)
      {
       if (letterCounts[i] > tempHighest)
       { 
         tempHighest = letterCounts[i];
         tempIndex = i;
        }
      } 
 
         testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    
     System.out.println(encoding.decode(copySentence));
  


    }     
     else
        {
           testProblem2CeasarCipher encoding = new testProblem2CeasarCipher(tempIndex);    
          //System.out.println(encoding.decode(copySentence));
 
        }
    
     }  
   }    
}

I just can't figure out what is wrong ?

[Gigi]

anyone ???