{"id":105,"date":"2014-03-17T18:00:06","date_gmt":"2014-03-17T18:00:06","guid":{"rendered":"http:\/\/www.karstencode.com\/?page_id=105"},"modified":"2014-03-17T18:01:34","modified_gmt":"2014-03-17T18:01:34","slug":"a-review-of-the-t3-project","status":"publish","type":"page","link":"http:\/\/www.karstencode.com\/?page_id=105","title":{"rendered":"A Review of the T^3 Project"},"content":{"rendered":"

This is a personal review of my post DBC work. After DBC I took time for family obligations and this is a review exercise. \u00a0I’m going to point out the parts in which I had problems and my thought process behind some of the work. Please feel free to leave comments. Actually, I insist that you give advice.<\/p>\n

First the code:\u00a0https:\/\/github.com\/ymeynot45\/T3_trials<\/p>\n

\n
\n
\"Tic<\/a><\/dt>\n
Tic Tac Toe board<\/dd>\n<\/dl>\n<\/div>\n

I’ll admit I found it really easy to figure out how to do the work, but I had more than one problem where I forgot terms. \u00a0Those are the issues that always take the most time for me as they are the hardest to figure out what was wrong.<\/p>\n

I started with the HTML, CSS and JS. I wanted to focus on the AJAX and knowing what data I was passing back and forth before I designed the backend to handle it. \u00a0I had used AJAX for some minor things in the past but it was normally sending only one or two pieces of data.
\nSo my first problem was
\n$(“#currentboard”).val(response.currentboard);
\nThis code was to pass the current condition of the board back and forth. Simple, except when writing it I put .value as the code in sublime turned blue indicating that it was a keyword and it is also the name of the CSS location I’m trying to change. \u00a0It’s one of those things when you haven’t done it in a while and it looks right, it sounds right, and it feels right… yet, it isn’t right. \u00a0I fixed it by unhiding the field and seeing that it wasn’t changing. \u00a0Then I threw a “puts” statement in the ruby right before sending back the info. \u00a0That made it real clear that the Ruby was working but JS was broken. \u00a0An easy fix once I knew where to look.<\/p>\n

Next came an issue from the Ruby. My memory failed me as I used a .to_s like I would with an integer, on an array. The code,
\ncols = [firstcol.join, secondcol.join, thirdcol.join]
\nI forgot that when you .to_s an array it converts the brackets and quote marks in the array. \u00a0I didn’t catch the error with a .class as it was supposed to be a string. I caught it with another print\/puts line. The extra quotes didn’t standout enough for me to take note, but the escapes for each of the quote marks inside of the brackets were a big clue.<\/p>\n

Structure… When I write code I like it when it reads like a book. \u00a0The methods in the main body are all wrapped in ones that I create so it is explicit what each of them do, but I’ve never gotten a good read from my mentors as to when too much is too much. \u00a0An example:
\nif board.include? “5”
\nreturn “5”
\nthat is a simple piece of code. \u00a0It is at the beginning of the process for the AI to choose a move. \u00a0It is easy to understand that if there are a five in board return a 5, but it doesn’t tell you why. \u00a0I like using method names to walk people through the code. \u00a0I didn’t make a new method because it is such a simple piece of code. \u00a0I look forward to having more mentors and teachers show me how to make my code more legible to programmers. How not to over complicate that which is already simple.<\/p>\n

Another decision in my code was:<\/p>\n

\n
\n
\"line_check<\/a><\/dt>\n
line_check code<\/dd>\n<\/dl>\n<\/div>\n

I originally had this as two separate methods: block_check & win_check. \u00a0It include the line, but not the piece so it was practically the same method and only took one argument. \u00a0I debated combining the two as it made it more likely to break and harder to track where the problem might occur. I eventually did because I know we remove repetition to make our code better but there are occasion where I’m not sure if it applicable.<\/p>\n

The rest was easy, some planning & some trial in error. \u00a0My next project for fun and as a refresher will be to convert a Lone Wolf<\/a>\u00a0book in to a web game. \u00a0It will require basics like a login, sql storage of player & character info, a lot of AJAX when combat takes place and an improvement in my UX skills to make the HTML and CSS accessible and attractive to a larger audience. \u00a0I hope that if I work hard that I could have it done with in two weeks. \u00a0I need another DBC like project with a ticking clock. \u00a0I do my best work under the gun.<\/p>\n","protected":false},"excerpt":{"rendered":"

This is a personal review of my post DBC work. After DBC I took time for family obligations and this is a review exercise. \u00a0I’m going to point out the parts in which I had problems and my thought process behind some of the work. Please feel free to leave comments. Actually, I insist that […]<\/p>\n","protected":false},"author":1,"featured_media":0,"parent":65,"menu_order":0,"comment_status":"open","ping_status":"open","template":"","meta":{"footnotes":""},"class_list":["post-105","page","type-page","status-publish","hentry"],"_links":{"self":[{"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/pages\/105","targetHints":{"allow":["GET"]}}],"collection":[{"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/pages"}],"about":[{"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/types\/page"}],"author":[{"embeddable":true,"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"http:\/\/www.karstencode.com\/index.php?rest_route=%2Fwp%2Fv2%2Fcomments&post=105"}],"version-history":[{"count":3,"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/pages\/105\/revisions"}],"predecessor-version":[{"id":108,"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/pages\/105\/revisions\/108"}],"up":[{"embeddable":true,"href":"http:\/\/www.karstencode.com\/index.php?rest_route=\/wp\/v2\/pages\/65"}],"wp:attachment":[{"href":"http:\/\/www.karstencode.com\/index.php?rest_route=%2Fwp%2Fv2%2Fmedia&parent=105"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}