Re: Ranking/Score keeping

I've noticed that no one has replied to your post. Here are some things that you could do that would make it easier for someone to help without us having to spend several hours:

You didn't use strict; so I suspect that you didn't run the program using perl -w either. This might generate some meaningful messages from what you think is "correct" code. Using strict will force you to scope your variables with my, our, or local.
Your code relies on giving a good deal of input which most people are unfamiliar with. You might consider eliminating the STDIN portions and just use pre-initialized variables (scalars, arrays, hashes).
We don't have any idea what the calculations should be, so we can't tell if the precedence of your operators are correct or not. You can remedy this by providing us with input (as explained above) and then showing us what the correct results would be so that we can compare it to what we get when your code is run.

You might want to modify your code and then reply to your original post. It may be that in following the advice given above that you might solve the problem yourself. Either way it would cut the time needed for any of us to attempt a solution. Good luck,

Jim

"If a man does his best, what else is there?" -- General George S. Patton

Comment on Re: Ranking/Score keeping

Replies are listed 'Best First'.
Re: Re: Ranking/Score keeping by Anonymous Monk on Oct 19, 2001 at 23:40 UTC
Thank you for your comment. I ran the code with 'use strict' and '-w' (including a 'use vars' statement at the top) and it ran and compiled and whatnot successfully. I probably should have clarified, but some problems that I noticed witht the output was this: (1)The first output works correctly, except a blank line is added at the top (could it make an error in my logic?) (2)THe second(and third and etc.) time it is not correct. Namely, when I enter the data and give the team number, if I have given the team numbers as 1 2 3 and 4, and I enter 1, it does not prompt me for the names(this is correct, because I would already have them) BUT for 2 3 and 4 it incorrectly prompts me for their names; also, the output is erroneous. Namely the last part of it (the number of matches) stays at 1, i.e., does not increase to 2 (3, 4, etc.) as it should. Also it doesn't average the score, or remember the names. Sample input for the program would be this, if it helps: 11 . 2 . 3 . 2 . 92 . 0 . 0 . 0 . 0 . 0 . 1 . name1 name2 . 2 . name3 name4 . 3 . name5 name6 . 4 . name7 name8 . <END OF INPUT> The score would be 732 (as it correctly displays); but if you enter the SAME data again, it works incorrectly (and if you change the 11, let us say, to a 10, you can see that the scores don't average either). As for the "calculations" I don't think you need to know them, becuase (I may not have made it clear) but they DO work. . . it is the actual ranking that does not. I have tested them (in the original C++ they were written in, albeit) and they work fine. If any more info is needed, just ask.	[reply]
Re: Re: Re: Ranking/Score keeping by jlongino (Parson) on Oct 20, 2001 at 02:32 UTC
The program died when attempting to open the ranks.txt file (line 69). You should modify your code so that if the file doesn't exist (hence it is a /New File/) you create and initialize it accordingly. Once I've entered the required input the program ends and does not print any output to the screen. It writes to the rank.txt file: `--------------------------------------- 1:244:244:andy ben:1 <> 2:244:244:cathy doug:1 <> 3:244:244:elaine fred:1 <> 4:244:244:gail harry:1 ---------------------------------------` [download] I'm not sure where you came up with 732, it is not displayed anywhere that I can see. Just because something was coded in C++ doesn't mean that it will work correctly after you convert it to Perl. If your plan is to read in each team's data and then do calculations and write the info back out to the file, you should use logic along these lines: `if rank.txt exists { if rank.txt is not empty { read each team's info and store in an array/hash } } if user wants to add new team data { loop # once through loop gets only data for one team get* team number get* member names get* team scores/whatever append record to rank.txt endloop # * input/store to array/hash } perform calculations for array/hash print results` [download] More observations: A common mistake for programmers is to type in a complete program and then try to debug it. Your code should be developed in small chunks. For example, don't attempt to code subs to perform calculations when you haven't even debugged/verified your input routines yet. Your chunk size may vary depending on the complexity of a given code segment. Use print liberally when verifying code correctness. You can always comment them out later. Start by commenting out your all of your subroutine calls. As you successfully debug one chunk, uncomment the next. --Jim	[reply] [d/l] [select]