« first day (3981 days earlier)      last day (30 days later) » 

12:06 AM
@sigod -- the posted code is broken and showing some fundamental misunderstandings; Code Review is for code that is working. — ad absurdum 52 secs ago
REFRESH! There are 6390 unanswered questions (91.7661 answered)
 
 
8 hours later…
8:44 AM
It feels a bit needlessly complicated and you aren't actually using private encapsulation. Unfortunately SO tends to frown at design and big picture issues. If you could re-post this at Codidact Software Development under the code review category, I'll take a closer look at it and post an answer. — Lundin 16 secs ago
 
 
2 hours later…
10:18 AM
0
Q: Text based tic-tac-toe in bash

sbottingotaI wrote this two player tic-tac-toe program in bash. To make a move you enter a number between 0 and 8 which corresponds to the square: 0|1|2 3|4|5 6|7|8 This is my code: #!/usr/bin/env bash declare -r BOARD_SIZE=9 declare -r DEFAULT_SQUARE_VALUE=- declare -ra PLAYERS=(O X) draw_board() { ...

 
10:58 AM
Do you have any specific question about the code? If you seek feedback with a working application, please move your question to codereview.stackexchange.comNico Haase 41 secs ago
 
 
4 hours later…
3:27 PM
If you want someone to review your code and make suggestions, post to Code Review, not Stack Overflow. Stack Overflow is for code that doesn't work the way you expect. — Doug Stevenson 15 secs ago
 
0
Q: How to shrink a react function code with firebase set/update operations

Andrian I have a function which does firebase set, update operations but it looks messy. I want to shrink it but also make sure that it works the same. Any ideas? const handleUserSelect = async (user: Users) => { if (!currentUser) return; const combinedId = currentUser.uid > user.uid ...

 
3:59 PM
0
Q: My C++ Implementation of the Observer Pattern

Archie PayneI have written my own basic implementation of the Observer pattern. Please code review it as you feel. This is a one file implementation, and any feedback no matter how small is appreciated. One sticky point is I was unsure on the implementation of RemoveObserver, so this may not be optimal. Co...

 
4:22 PM
0
Q: C - Force a macro argument to be a string literal

HarithThe requirement is to define a macro which takes a single argument with the following contraints: Is a character pointer Must be non-null Must be immutable Must be 0-terminated My first try at it was to surround it with empty string literals (I got this from Modern C): #define SV(str) (sizeof("

 
Monking
 
4:45 PM
0
Q: Improving code efficiency

SHOVALWrite an efficient function in the Python language called SmartSearch, this function will receive a sorted arr array of integers of size n containing at the beginning m "significant numbers" and the rest of the numbers are "fictitious numbers" whose value is 9999. The function also receives a num...

 
5:15 PM
Monking
 
5:26 PM
Monking
 
 
1 hour later…
6:51 PM
For help improving working code, please see codereview.stackexchange.com. Be sure to read their help before posting. — T.J. Crowder 15 secs ago
 
7:46 PM
0
Q: Simplify complexity

user282070I came across this question which asks to create a function which will return true/false based on the passed array containing all the letters to make up the passed word. Each letter from array can only be used once. Here's my attempt: const checkArray = (word, arr) => { let good = true wo...

 
 
1 hour later…
9:05 PM
possible answer invalidation by Sᴀᴍ Onᴇᴌᴀ on question by Razvan Zamfir: codereview.stackexchange.com/posts/210483/revisions
possible answer invalidation by Sᴀᴍ Onᴇᴌᴀ on question by Razvan Zamfir: codereview.stackexchange.com/posts/210483/revisions
possible answer invalidation by Sᴀᴍ Onᴇᴌᴀ on question by Razvan Zamfir: codereview.stackexchange.com/posts/210483/revisions
possible answer invalidation by Sᴀᴍ Onᴇᴌᴀ on question by Razvan Zamfir: codereview.stackexchange.com/posts/210483/revisions
 

« first day (3981 days earlier)      last day (30 days later) »