Ignore:
Timestamp:
Sep 4, 2018, 1:00:48 PM (6 years ago)
Author:
alloc
Message:

Code style cleanup (mostly whitespace changes, enforcing braces, using cleanup)

Location:
binary-improvements/7dtd-server-fixes/src/JSON
Files:
10 edited

Legend:

Unmodified
Added
Removed

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONArray.cs

    r315 r325  
    1 using System;
    21using System.Collections.Generic;
    32using System.Text;
    43
    5 namespace AllocsFixes.JSON
    6 {
    7         public class JSONArray : JSONNode
    8         {
    9                 private List<JSONNode> nodes = new List<JSONNode> ();
     4namespace AllocsFixes.JSON {
     5        public class JSONArray : JSONNode {
     6                private readonly List<JSONNode> nodes = new List<JSONNode> ();
    107
    118                public JSONNode this [int index] {
    … …  
    1815                }
    1916
    20                 public void Add (JSONNode node)
    21                 {
     17                public void Add (JSONNode node) {
    2218                        nodes.Add (node);
    2319                }
    2420
    25                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    26                 {
     21                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    2722                        stringBuilder.Append ("[");
    28                         if (prettyPrint)
     23                        if (prettyPrint) {
    2924                                stringBuilder.Append ('\n');
     25                        }
     26
    3027                        foreach (JSONNode n in nodes) {
    31                                 if (prettyPrint)
    32                                         stringBuilder.Append (new String ('\t', currentLevel + 1));
     28                                if (prettyPrint) {
     29                                        stringBuilder.Append (new string ('\t', currentLevel + 1));
     30                                }
     31
    3332                                n.ToString (stringBuilder, prettyPrint, currentLevel + 1);
    3433                                stringBuilder.Append (",");
    35                                 if (prettyPrint)
     34                                if (prettyPrint) {
    3635                                        stringBuilder.Append ('\n');
     36                                }
    3737                        }
    38                         if (nodes.Count > 0)
     38
     39                        if (nodes.Count > 0) {
    3940                                stringBuilder.Remove (stringBuilder.Length - (prettyPrint ? 2 : 1), 1);
    40                         if (prettyPrint)
    41                                 stringBuilder.Append (new String ('\t', currentLevel));
     41                        }
     42
     43                        if (prettyPrint) {
     44                                stringBuilder.Append (new string ('\t', currentLevel));
     45                        }
     46
    4247                        stringBuilder.Append ("]");
    4348                }
    4449
    45                 public static JSONArray Parse (string json, ref int offset)
    46                 {
     50                public static JSONArray Parse (string json, ref int offset) {
    4751                        //Log.Out ("ParseArray enter (" + offset + ")");
    4852                        JSONArray arr = new JSONArray ();
    … …  
    5862                                                        nextElemAllowed = true;
    5963                                                        offset++;
    60                                                 } else
    61                                                         throw new MalformedJSONException ("Could not parse array, found a comma without a value first");
     64                                                } else {
     65                                                        throw new MalformedJSONException (
     66                                                                "Could not parse array, found a comma without a value first");
     67                                                }
     68
    6269                                                break;
    6370                                        case ']':
    6471                                                offset++;
     72
    6573                                                //Log.Out ("JSON:Parsed Array: " + arr.ToString ());
    6674                                                return arr;
    … …  
    7280                        }
    7381                }
    74 
    7582        }
    7683}
    77 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONBoolean.cs

    r309 r325  
    1 using System;
    21using System.Text;
    32
    4 namespace AllocsFixes.JSON
    5 {
    6         public class JSONBoolean : JSONValue
    7         {
    8                 private bool value;
     3namespace AllocsFixes.JSON {
     4        public class JSONBoolean : JSONValue {
     5                private readonly bool value;
    96
    10                 public JSONBoolean (bool value)
    11                 {
     7                public JSONBoolean (bool value) {
    128                        this.value = value;
    139                }
    1410
    15                 public bool GetBool ()
    16                 {
     11                public bool GetBool () {
    1712                        return value;
    1813                }
    1914
    20                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    21                 {
     15                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    2216                        stringBuilder.Append (value ? "true" : "false");
    2317                }
    2418
    25                 public static JSONBoolean Parse (string json, ref int offset)
    26                 {
     19                public static JSONBoolean Parse (string json, ref int offset) {
    2720                        //Log.Out ("ParseBool enter (" + offset + ")");
    2821
    … …  
    3124                                offset += 4;
    3225                                return new JSONBoolean (true);
    33                         } else if (json.Substring (offset, 5).Equals ("false")) {
     26                        }
     27
     28                        if (json.Substring (offset, 5).Equals ("false")) {
    3429                                //Log.Out ("JSON:Parsed Bool: false");
    3530                                offset += 5;
    3631                                return new JSONBoolean (false);
    37                         } else {
    38                                 throw new MalformedJSONException ("No valid boolean found");
    3932                        }
     33
     34                        throw new MalformedJSONException ("No valid boolean found");
    4035                }
    41 
    4236        }
    4337}
    44 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONNode.cs

    r309 r325  
    1 using System;
    21using System.Text;
    32
    4 namespace AllocsFixes.JSON
    5 {
    6         public abstract class JSONNode
    7         {
    8                 public abstract void ToString(StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0);
     3namespace AllocsFixes.JSON {
     4        public abstract class JSONNode {
     5                public abstract void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0);
    96
    107                public override string ToString () {

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONNull.cs

    r309 r325  
    1 using System;
    21using System.Text;
    32
    4 namespace AllocsFixes.JSON
    5 {
    6         public class JSONNull : JSONValue
    7         {
    8                 public JSONNull ()
    9                 {
    10                 }
    11 
    12                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    13                 {
     3namespace AllocsFixes.JSON {
     4        public class JSONNull : JSONValue {
     5                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    146                        stringBuilder.Append ("null");
    157                }
    168
    17                 public static JSONNull Parse (string json, ref int offset)
    18                 {
     9                public static JSONNull Parse (string json, ref int offset) {
    1910                        //Log.Out ("ParseNull enter (" + offset + ")");
    2011
    … …  
    2314                                offset += 4;
    2415                                return new JSONNull ();
    25                         } else {
    26                                 throw new MalformedJSONException ("No valid null value found");
    2716                        }
     17
     18                        throw new MalformedJSONException ("No valid null value found");
    2819                }
    29 
    3020        }
    3121}
    32 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONNumber.cs

    r324 r325  
    22using System.Text;
    33
    4 namespace AllocsFixes.JSON
    5 {
    6         public class JSONNumber : JSONValue
    7         {
    8                 private double value;
     4namespace AllocsFixes.JSON {
     5        public class JSONNumber : JSONValue {
     6                private readonly double value;
    97
    10                 public JSONNumber (double value)
    11                 {
     8                public JSONNumber (double value) {
    129                        this.value = value;
    1310                }
    1411
    15                 public double GetDouble ()
    16                 {
     12                public double GetDouble () {
    1713                        return value;
    1814                }
    1915
    20                 public int GetInt ()
    21                 {
    22                         return (int)Math.Round(value);
     16                public int GetInt () {
     17                        return (int) Math.Round (value);
    2318                }
    2419
    25                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    26                 {
     20                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    2721                        stringBuilder.Append (value.ToCultureInvariantString ());
    2822                }
    2923
    30                 public static JSONNumber Parse (string json, ref int offset)
    31                 {
     24                public static JSONNumber Parse (string json, ref int offset) {
    3225                        //Log.Out ("ParseNumber enter (" + offset + ")");
    3326                        StringBuilder sbNum = new StringBuilder ();
    … …  
    3730                        while (offset < json.Length) {
    3831                                if (json [offset] >= '0' && json [offset] <= '9') {
    39                                         if (hasExp)
     32                                        if (hasExp) {
    4033                                                sbExp.Append (json [offset]);
    41                                         else
     34                                        } else {
    4235                                                sbNum.Append (json [offset]);
     36                                        }
    4337                                } else if (json [offset] == '.') {
    4438                                        if (hasExp) {
    4539                                                throw new MalformedJSONException ("Decimal separator in exponent");
     40                                        }
     41
     42                                        if (hasDec) {
     43                                                throw new MalformedJSONException ("Multiple decimal separators in number found");
     44                                        }
     45
     46                                        if (sbNum.Length == 0) {
     47                                                throw new MalformedJSONException ("No leading digits before decimal separator found");
     48                                        }
     49
     50                                        sbNum.Append ('.');
     51                                        hasDec = true;
     52                                } else if (json [offset] == '-') {
     53                                        if (hasExp) {
     54                                                if (sbExp.Length > 0) {
     55                                                        throw new MalformedJSONException ("Negative sign in exponent after digits");
     56                                                }
     57
     58                                                sbExp.Append (json [offset]);
    4659                                        } else {
    47                                                 if (hasDec)
    48                                                         throw new MalformedJSONException ("Multiple decimal separators in number found");
    49                                                 else if (sbNum.Length == 0) {
    50                                                         throw new MalformedJSONException ("No leading digits before decimal separator found");
    51                                                 } else {
    52                                                         sbNum.Append ('.');
    53                                                         hasDec = true;
     60                                                if (sbNum.Length > 0) {
     61                                                        throw new MalformedJSONException ("Negative sign in mantissa after digits");
    5462                                                }
    55                                         }
    56                                 } else  if (json [offset] == '-') {
    57                                         if (hasExp) {
    58                                                 if (sbExp.Length > 0)
    59                                                         throw new MalformedJSONException ("Negative sign in exponent after digits");
    60                                                 else
    61                                                         sbExp.Append (json [offset]);
    62                                         } else {
    63                                                 if (sbNum.Length > 0)
    64                                                         throw new MalformedJSONException ("Negative sign in mantissa after digits");
    65                                                 else
    66                                                         sbNum.Append (json [offset]);
     63
     64                                                sbNum.Append (json [offset]);
    6765                                        }
    6866                                } else if (json [offset] == 'e' || json [offset] == 'E') {
    69                                         if (hasExp)
     67                                        if (hasExp) {
    7068                                                throw new MalformedJSONException ("Multiple exponential markers in number found");
    71                                         else if (sbNum.Length == 0) {
     69                                        }
     70
     71                                        if (sbNum.Length == 0) {
    7272                                                throw new MalformedJSONException ("No leading digits before exponential marker found");
    73                                         } else {
    74                                                 sbExp = new StringBuilder ();
    75                                                 hasExp = true;
    7673                                        }
     74
     75                                        sbExp = new StringBuilder ();
     76                                        hasExp = true;
    7777                                } else if (json [offset] == '+') {
    7878                                        if (hasExp) {
    79                                                 if (sbExp.Length > 0)
     79                                                if (sbExp.Length > 0) {
    8080                                                        throw new MalformedJSONException ("Positive sign in exponent after digits");
    81                                                 else
    82                                                         sbExp.Append (json [offset]);
     81                                                }
     82
     83                                                sbExp.Append (json [offset]);
    8384                                        } else {
    8485                                                throw new MalformedJSONException ("Positive sign in mantissa found");
    … …  
    8687                                } else {
    8788                                        double number;
    88                                         if (!StringParsers.TryParseDouble(sbNum.ToString (), out number)) {
    89                                                 throw new MalformedJSONException ("Mantissa is not a valid decimal (\"" + sbNum.ToString () + "\")");
     89                                        if (!StringParsers.TryParseDouble (sbNum.ToString (), out number)) {
     90                                                throw new MalformedJSONException ("Mantissa is not a valid decimal (\"" + sbNum + "\")");
    9091                                        }
    9192
    … …  
    9394                                                int exp;
    9495                                                if (!int.TryParse (sbExp.ToString (), out exp)) {
    95                                                         throw new MalformedJSONException ("Exponent is not a valid integer (\"" + sbExp.ToString () + "\")");
     96                                                        throw new MalformedJSONException ("Exponent is not a valid integer (\"" + sbExp + "\")");
    9697                                                }
    9798
    … …  
    102103                                        return new JSONNumber (number);
    103104                                }
     105
    104106                                offset++;
    105107                        }
     108
    106109                        throw new MalformedJSONException ("End of JSON reached before parsing number finished");
    107110                }
    108 
    109111        }
    110112}
    111 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONObject.cs

    r315 r325  
    1 using System;
    21using System.Collections.Generic;
    32using System.Text;
    43
    5 namespace AllocsFixes.JSON
    6 {
    7         public class JSONObject : JSONNode
    8         {
    9                 private Dictionary<string, JSONNode> nodes = new Dictionary<string, JSONNode> ();
     4namespace AllocsFixes.JSON {
     5        public class JSONObject : JSONNode {
     6                private readonly Dictionary<string, JSONNode> nodes = new Dictionary<string, JSONNode> ();
    107
    118                public JSONNode this [string name] {
    … …  
    2219                }
    2320
    24                 public bool ContainsKey (string name)
    25                 {
     21                public bool ContainsKey (string name) {
    2622                        return nodes.ContainsKey (name);
    2723                }
    2824
    29                 public void Add (string name, JSONNode node)
    30                 {
     25                public void Add (string name, JSONNode node) {
    3126                        nodes.Add (name, node);
    3227                }
    3328
    34                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    35                 {
     29                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    3630                        stringBuilder.Append ("{");
    37                         if (prettyPrint)
     31                        if (prettyPrint) {
    3832                                stringBuilder.Append ('\n');
     33                        }
     34
    3935                        foreach (KeyValuePair<string, JSONNode> kvp in nodes) {
    40                                 if (prettyPrint)
    41                                         stringBuilder.Append (new String ('\t', currentLevel + 1));
    42                                 stringBuilder.Append (String.Format ("\"{0}\":", kvp.Key));
    43                                 if (prettyPrint)
     36                                if (prettyPrint) {
     37                                        stringBuilder.Append (new string ('\t', currentLevel + 1));
     38                                }
     39
     40                                stringBuilder.Append (string.Format ("\"{0}\":", kvp.Key));
     41                                if (prettyPrint) {
    4442                                        stringBuilder.Append (" ");
     43                                }
     44
    4545                                kvp.Value.ToString (stringBuilder, prettyPrint, currentLevel + 1);
    4646                                stringBuilder.Append (",");
    47                                 if (prettyPrint)
     47                                if (prettyPrint) {
    4848                                        stringBuilder.Append ('\n');
     49                                }
    4950                        }
    50                         if (nodes.Count > 0)
     51
     52                        if (nodes.Count > 0) {
    5153                                stringBuilder.Remove (stringBuilder.Length - (prettyPrint ? 2 : 1), 1);
    52                         if (prettyPrint)
    53                                 stringBuilder.Append (new String ('\t', currentLevel));
     54                        }
     55
     56                        if (prettyPrint) {
     57                                stringBuilder.Append (new string ('\t', currentLevel));
     58                        }
     59
    5460                        stringBuilder.Append ("}");
    5561                }
    5662
    57                 public static JSONObject Parse (string json, ref int offset)
    58                 {
     63                public static JSONObject Parse (string json, ref int offset) {
    5964                        //Log.Out ("ParseObject enter (" + offset + ")");
    6065                        JSONObject obj = new JSONObject ();
    … …  
    7075                                                        Parser.SkipWhitespace (json, ref offset);
    7176                                                        if (json [offset] != ':') {
    72                                                                 throw new MalformedJSONException ("Could not parse object, missing colon (\":\") after key");
     77                                                                throw new MalformedJSONException (
     78                                                                        "Could not parse object, missing colon (\":\") after key");
    7379                                                        }
     80
    7481                                                        offset++;
    7582                                                        JSONNode val = Parser.ParseInternal (json, ref offset);
    … …  
    7784                                                        nextElemAllowed = false;
    7885                                                } else {
    79                                                         throw new MalformedJSONException ("Could not parse object, found new key without a separating comma");
     86                                                        throw new MalformedJSONException (
     87                                                                "Could not parse object, found new key without a separating comma");
    8088                                                }
     89
    8190                                                break;
    8291                                        case ',':
    … …  
    8493                                                        nextElemAllowed = true;
    8594                                                        offset++;
    86                                                 } else
    87                                                         throw new MalformedJSONException ("Could not parse object, found a comma without a key/value pair first");
     95                                                } else {
     96                                                        throw new MalformedJSONException (
     97                                                                "Could not parse object, found a comma without a key/value pair first");
     98                                                }
     99
    88100                                                break;
    89101                                        case '}':
    90102                                                offset++;
     103
    91104                                                //Log.Out ("JSON:Parsed Object: " + obj.ToString ());
    92105                                                return obj;
    … …  
    94107                        }
    95108                }
    96 
    97109        }
    98110}
    99 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONString.cs

    r309 r325  
    1 using System;
    21using System.Text;
    32
    4 namespace AllocsFixes.JSON
    5 {
    6         public class JSONString : JSONValue
    7         {
    8                 private string value;
     3namespace AllocsFixes.JSON {
     4        public class JSONString : JSONValue {
     5                private readonly string value;
    96
    10                 public JSONString (string value)
    11                 {
     7                public JSONString (string value) {
    128                        this.value = value;
    139                }
    1410
    15                 public string GetString ()
    16                 {
     11                public string GetString () {
    1712                        return value;
    1813                }
    1914
    20                 public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0)
    21                 {
     15                public override void ToString (StringBuilder stringBuilder, bool prettyPrint = false, int currentLevel = 0) {
    2216                        if (value == null || value.Length == 0) {
    2317                                stringBuilder.Append ("\"\"");
    … …  
    2721                        int len = value.Length;
    2822
    29                         stringBuilder.EnsureCapacity (stringBuilder.Length + 2*len);
    30                         String t;
     23                        stringBuilder.EnsureCapacity (stringBuilder.Length + 2 * len);
    3124
    3225                        stringBuilder.Append ('"');
    … …  
    3629                                        case '\\':
    3730                                        case '"':
     31
    3832//                                      case '/':
    3933                                                stringBuilder.Append ('\\');
    … …  
    5852                                                if (c < ' ') {
    5953                                                        stringBuilder.Append ("\\u");
    60                                                         stringBuilder.Append (((int)c).ToString ("X4"));
     54                                                        stringBuilder.Append (((int) c).ToString ("X4"));
    6155                                                } else {
    6256                                                        stringBuilder.Append (c);
    6357                                                }
     58
    6459                                                break;
    6560                                }
    … …  
    6964                }
    7065
    71                 public static JSONString Parse (string json, ref int offset)
    72                 {
     66                public static JSONString Parse (string json, ref int offset) {
    7367                        //Log.Out ("ParseString enter (" + offset + ")");
    7468                        StringBuilder sb = new StringBuilder ();
    … …  
    10397                                                                break;
    10498                                                }
     99
    105100                                                offset++;
    106101                                                break;
    107102                                        case '"':
    108103                                                offset++;
     104
    109105                                                //Log.Out ("JSON:Parsed String: " + sb.ToString ());
    110106                                                return new JSONString (sb.ToString ());
    … …  
    115111                                }
    116112                        }
     113
    117114                        throw new MalformedJSONException ("End of JSON reached before parsing string finished");
    118115                }
    119 
    120116        }
    121117}
    122 

  • binary-improvements/7dtd-server-fixes/src/JSON/JSONValue.cs

    r279 r325  
    1 using System;
    2 
    3 namespace AllocsFixes.JSON
    4 {
     1namespace AllocsFixes.JSON {
    52        public abstract class JSONValue : JSONNode {
    63        }
    74}
    8 

  • binary-improvements/7dtd-server-fixes/src/JSON/MalformedJSONException.cs

    r188 r325  
    22using System.Runtime.Serialization;
    33
    4 namespace AllocsFixes.JSON
    5 {
    6         public class MalformedJSONException : ApplicationException
    7         {
    8                 public MalformedJSONException ()
    9                 {
     4namespace AllocsFixes.JSON {
     5        public class MalformedJSONException : ApplicationException {
     6                public MalformedJSONException () {
    107                }
    118
    12                 public MalformedJSONException (string message) : base(message)
    13                 {
     9                public MalformedJSONException (string message) : base (message) {
    1410                }
    1511
    16                 public MalformedJSONException (string message, System.Exception inner) : base(message, inner)
    17                 {
     12                public MalformedJSONException (string message, Exception inner) : base (message, inner) {
    1813                }
    19  
    20                 protected MalformedJSONException (SerializationInfo info, StreamingContext context) : base(info, context)
    21                 {
     14
     15                protected MalformedJSONException (SerializationInfo info, StreamingContext context) : base (info, context) {
    2216                }
    2317        }
    2418}
    25 

  • binary-improvements/7dtd-server-fixes/src/JSON/Parser.cs

    r188 r325  
    1 using System;
    2 using System.Text;
    3 
    4 namespace AllocsFixes.JSON
    5 {
    6         public class Parser
    7         {
    8 
    9                 public static JSONNode Parse (string json)
    10                 {
     1namespace AllocsFixes.JSON {
     2        public class Parser {
     3                public static JSONNode Parse (string json) {
    114                        int offset = 0;
    125                        return ParseInternal (json, ref offset);
    136                }
    147
    15                 public static JSONNode ParseInternal (string json, ref int offset)
    16                 {
     8                public static JSONNode ParseInternal (string json, ref int offset) {
    179                        SkipWhitespace (json, ref offset);
     10
    1811                        //Log.Out ("ParseInternal (" + offset + "): Decide on: '" + json [offset] + "'");
    1912                        switch (json [offset]) {
    … …  
    3427                }
    3528
    36                 public static void SkipWhitespace (string json, ref int offset)
    37                 {
     29                public static void SkipWhitespace (string json, ref int offset) {
    3830                        //Log.Out ("SkipWhitespace (" + offset + "): '" + json [offset] + "'");
    3931                        while (offset < json.Length) {
    … …  
    4941                                }
    5042                        }
     43
    5144                        throw new MalformedJSONException ("End of JSON reached before parsing finished");
    5245                }
    53 
    54 
    5546        }
    5647}
    57 
Note: See TracChangeset for help on using the changeset viewer.