Skip to content

Code Jam 4 Review Comments #2

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
zachkirschner wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Code Jam 4 Review Comments #2

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions CodeJam4/Program.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,8 @@ public static void PrintContent(HtmlDocument doc)

//Getting Content from each subheading.

if (subTitleContentNodes != null)
if (subTitleContentNodes != null) // Some of the headlines that contains apostrophes show an error in the middle of the word (ex. "What's the most valuable NFL franchise?"
// Finding a way to be able to show the apostrophe properly should be an easy fix
{
foreach (var subTitleContentNode in subTitleContentNodes)
{
Expand All @@ -88,7 +89,8 @@ public static void PrintContent(HtmlDocument doc)

for (int i = 0; i < subTitles.Count; i++)
{
Console.ForegroundColor = ConsoleColor.Green;
// for some of the articles, the contents don't show past the article title, might have to be reviewed with more test cases to see why that is
Console.ForegroundColor = ConsoleColor.Green;
Console.WriteLine(subTitles[i], Color.Aquamarine); // prints the subtitle
Console.ResetColor();
List<string> contents = subTitleContents[i];
Expand Down Expand Up @@ -120,7 +122,7 @@ static void Main(string[] args)
if (ans.Equals('Y'))
{
vrajMode = true;
System.Console.BackgroundColor = ConsoleColor.Red;
System.Console.BackgroundColor = ConsoleColor.Red;
}

Colorful.Console.WriteLine("Enter the url of a NBC live coverage news article that you want to create an outline of:", Color.Fuchsia);
Expand Down